TabPriorityQueue updates tabItems when UI is busy

RESOLVED FIXED in Firefox 6

Status

RESOLVED FIXED
8 years ago
3 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

Trunk
Firefox 6
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 9 obsolete attachments)

The condition is now:

if (this.isPaintingPaused() || !UI.isIdle)

and should be:

if (this.isPaintingPaused() || !UI.isIdle())

We should add a test to verify that no tabItem updates are done when resizing or moving items.
(Assignee)

Updated

8 years ago
Summary: TabPriorityQueue updates when UI is busy → TabPriorityQueue updates tabItems when UI is busy
Created attachment 524979 [details] [diff] [review]
patch v1
Attachment #524979 - Flags: feedback?(raymond)
Comment on attachment 524979 [details] [diff] [review]
patch v1

+      EventUtils.synthesizeMouse(container, 5, 5, {type: "mousedown"}, cw);
+      EventUtils.synthesizeMouse(container, 40, 20, {type: "mousemove"}, cw);
+      EventUtils.synthesizeMouse(container, 20, 20, {type: "mouseup"}, cw);

The above repeats twice, might be good to write a helper function.

f+
Attachment #524979 - Flags: feedback?(raymond) → feedback+
Created attachment 525095 [details] [diff] [review]
patch v2
Attachment #524979 - Attachment is obsolete: true
Attachment #525095 - Flags: review?(ian)
Comment on attachment 525095 [details] [diff] [review]
patch v2

That's a pretty slick test :)
Attachment #525095 - Flags: review?(ian) → review+
Created attachment 525923 [details] [diff] [review]
patch for checkin
Attachment #525095 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
Created attachment 525983 [details] [diff] [review]
patch for checkin (added waitForFocus)
Attachment #525923 - Attachment is obsolete: true

Comment 8

8 years ago
http://hg.mozilla.org/projects/cedar/rev/b1fc69fab8b2
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: fixed-in-cedar
Target Milestone: --- → Firefox 6

Comment 9

8 years ago
The test added by this patch went orange: <http://tinderbox.mozilla.org/showlog.cgi?log=Cedar/1302807869.1302809327.10302.gz> so I backed it out: <http://hg.mozilla.org/projects/cedar/rev/53339e6932c3>
Whiteboard: fixed-in-cedar
Created attachment 526118 [details] [diff] [review]
patch v3

Added another test for another issue fixed with this patch.
Attachment #525983 - Attachment is obsolete: true
Attachment #526118 - Flags: feedback?(raymond)
Comment on attachment 526118 [details] [diff] [review]
patch v3

Looks good.  Can we merge two tests file into one?
Attachment #526118 - Flags: feedback?(raymond) → feedback+
Created attachment 526179 [details] [diff] [review]
patch v4

(In reply to comment #11)
> Looks good.  Can we merge two tests file into one?

Of course, don't know why I split them :)
Attachment #526118 - Attachment is obsolete: true
Attachment #526179 - Flags: review?(ian)
Created attachment 526333 [details] [diff] [review]
patch v5
Attachment #526179 - Attachment is obsolete: true
Attachment #526179 - Flags: review?(ian)
Attachment #526333 - Flags: review?(ian)
Comment on attachment 526333 [details] [diff] [review]
patch v5

Looks good.
Attachment #526333 - Flags: review?(ian) → review+
Created attachment 527023 [details] [diff] [review]
patch for checkin
Attachment #526333 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
Keywords: checkin-needed
Whiteboard: [fixed in cedar]
Pushed:
http://hg.mozilla.org/mozilla-central/rev/028fc2dcb854
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in cedar]
You should not use setTimeout(func, value) with value>0 if you do not want your test to be flaky.
Created attachment 527343 [details] [diff] [review]
patch v6 (without setTimeout)

(In reply to comment #20)
> You should not use setTimeout(func, value) with value>0 if you do not want your
> test to be flaky.

Good to know, removed.
Attachment #527023 - Attachment is obsolete: true
Attachment #527343 - Flags: feedback?(raymond)
Comment on attachment 527343 [details] [diff] [review]
patch v6 (without setTimeout)

Looks good
Attachment #527343 - Flags: feedback?(raymond) → feedback+
(Assignee)

Updated

8 years ago
Attachment #527343 - Flags: review?(ian)
Comment on attachment 527343 [details] [diff] [review]
patch v6 (without setTimeout)

Review of attachment 527343 [details] [diff] [review]:

Very clever :)
Attachment #527343 - Flags: review?(ian) → review+
Created attachment 528338 [details] [diff] [review]
patch for checkin
Attachment #527343 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
Keywords: checkin-needed
Whiteboard: [fixed in cedar]
Pushed:
http://hg.mozilla.org/mozilla-central/rev/4e7e81ad7525

Thank you for fixing the random orange Tim :)
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in cedar]
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.