Resizes for single groups don't stick correctly, returning them to their userSize values

RESOLVED FIXED in Firefox 8

Status

Firefox Graveyard
Panorama
P5
normal
RESOLVED FIXED
6 years ago
a year ago

People

(Reporter: mitcho, Assigned: ttaubert)

Tracking

({polish})

Trunk
Firefox 8
polish
Dependency tree / graph
Bug Flags:
in-testsuite +

Details

(Whiteboard: [visual][polish])

Attachments

(2 attachments, 2 obsolete attachments)

Created attachment 503757 [details]
Screenshot

When a window only has a single group and this window is resized down past the usersize, you observe some odd behavior where you momentarily see the group get resized to new bounds to fit the window, but then bounce back to its original size. Something odd is going on in the resize code in this case.

This is one non-pushAway cause of bug 587140.

This bug is a followup to bug 625269. The test for 625269 includes a TODO for this bug.
(Assignee)

Updated

6 years ago
Blocks: 673196
(Assignee)

Updated

6 years ago
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
Whiteboard: [visual][polish][good first bug] → [visual][polish]
Target Milestone: Future → ---
Version: unspecified → Trunk
(Assignee)

Comment 1

6 years ago
Created attachment 547592 [details] [diff] [review]
patch v1

Item.pushAway() does strange things when called with only one top-level item. There's no need to even execute all this code when there's only one top-level item (that's what the patch does).

I refactored the whole test for bug 625269 (that covers this bug, too) because it didn't measure the group size correctly and was as a whole a bit unclear.
Attachment #547592 - Flags: feedback?(raymond)
Attachment #547592 - Flags: feedback?(raymond) → feedback+
(Assignee)

Updated

6 years ago
Attachment #547592 - Flags: review?(dietrich)
Comment on attachment 547592 [details] [diff] [review]
patch v1

Review of attachment 547592 [details] [diff] [review]:
-----------------------------------------------------------------

looks good, r=me.
Attachment #547592 - Flags: review?(dietrich) → review+
(Assignee)

Comment 3

6 years ago
Created attachment 547966 [details] [diff] [review]
patch v2

Alas, browser_tabview_bug625269.js failed once on the try server. I extended it to wait until any css transition (if active) ended and check the group bounds after that.
Attachment #547592 - Attachment is obsolete: true
Attachment #547966 - Flags: review?(dietrich)
(Assignee)

Comment 4

6 years ago
Comment on attachment 547966 [details] [diff] [review]
patch v2

*Sigh* still fails on try. Investigating.
Attachment #547966 - Flags: review?(dietrich)
(Assignee)

Comment 5

6 years ago
Created attachment 548265 [details] [diff] [review]
patch v3

Reverted the changes from patch v2. There are no animations involved, silly me. The problem was that there can be multiple resize events and we need to wait until the target size has been reached.
Attachment #547966 - Attachment is obsolete: true
Attachment #548265 - Flags: review?(dietrich)
Attachment #548265 - Flags: review?(dietrich) → review+
(Assignee)

Comment 6

6 years ago
http://hg.mozilla.org/integration/fx-team/rev/42e7710675b2
Whiteboard: [visual][polish] → [visual][polish][fixed-in-fx-team]
(Assignee)

Comment 7

6 years ago
http://hg.mozilla.org/mozilla-central/rev/42e7710675b2
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [visual][polish][fixed-in-fx-team] → [visual][polish]
Target Milestone: --- → Firefox 8
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.