Closed Bug 583219 Opened 9 years ago Closed 9 years ago

Transitions get stuck when downloadable fonts load or page zoom changes

Categories

(Core :: CSS Parsing and Computation, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla5

People

(Reporter: paul, Unassigned)

References

()

Details

Attachments

(3 files)

Attached image screenshot
Load this demo:
http://demos.hacks.mozilla.org/openweb/LONDONPROJECT/

Click on the Firefox logo before the page is completely loaded: transitions start but don't finish.

Same behavior if I zoom/unzoom during the transitions.
Was this a recent regression or has it been a problem for a while?
I don't know. Just run into the same problem in another demo.
How can I help?
Attached patch patchSplinter Review
I started to try to hack restyling the root frame into the restyle tracker (which would allow simplifying everything here between the SetProcessingRestyles() calls to the posting of an appropriate restyle event for the root frame), but it seemed like too much work.

I still need to write an automated test.
Attachment #522794 - Flags: review?(bzbarsky)
Summary: Transitions "stuck" if done during page load or page zoom → Transitions get stuck when downloadable fonts load or page zoom changes
Comment on attachment 522794 [details] [diff] [review]
patch

r=me
Attachment #522794 - Flags: review?(bzbarsky) → review+
Comment on attachment 522830 [details] [diff] [review]
test

r=me
Attachment #522830 - Flags: review?(bzbarsky) → review+
http://hg.mozilla.org/mozilla-central/rev/a04d86c6539e
http://hg.mozilla.org/mozilla-central/rev/3badbf020354

(Oops, I guess the commit message for the first patch wasn't really right anymore, though.)
Status: NEW → RESOLVED
Closed: 9 years ago
OS: Windows 7 → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
Could you document your SpecialPowers API additions at https://developer.mozilla.org/en/SpecialPowers ?
You need to log in before you can comment on or make changes to this bug.