Closed Bug 1201330 Opened 9 years ago Closed 9 years ago

If a CSS property is changed from a "scroll" handler, the property should be considered as animating (LayerActivity) until the scroll frame has become inactive

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox43 --- affected
firefox45 --- fixed

People

(Reporter: mstange, Assigned: mstange)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

For example, if a scroll handler updates the transform property of an element, we should keep that element layerized until the scrolled scroll frame becomes inactive.
Blocks: 1201336
This helps on many sites with parallax effects. Most of them modify background-position from a scroll handler. Keeping the background position layer activity alive between scrolls means there's no lag when a new scroll is started after the page has rested for a while.
Bug 1201330 - Refactor LayerActivity property management. r?roc
Attachment #8679530 - Flags: review?(roc)
Bug 1201330 - Keep scroll handler induced layer activity active until the scroll frame becomes inactive. r?roc
Attachment #8679531 - Flags: review?(roc)
Attached file testcase (deleted) —
Assignee: nobody → mstange
Status: NEW → ASSIGNED
OS: Unspecified → All
Hardware: Unspecified → All
Comment on attachment 8679530 [details] MozReview Request: Bug 1201330 - Refactor LayerActivity property management. r?roc https://reviewboard.mozilla.org/r/23461/#review20949
Attachment #8679530 - Flags: review?(roc) → review+
Comment on attachment 8679531 [details] MozReview Request: Bug 1201330 - Keep scroll handler induced layer activity active until the scroll frame becomes inactive. r?roc https://reviewboard.mozilla.org/r/23463/#review20963
Attachment #8679531 - Flags: review?(roc) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/72da2ed9b1e7db209601769d5fc4461c0490963f Bug 1201330 - Refactor LayerActivity property management. r=roc https://hg.mozilla.org/integration/mozilla-inbound/rev/d5d68c0258f3d225bce32140e13a7d40f0f29a2f Bug 1201330 - Keep scroll handler induced layer activity active until the scroll frame becomes inactive. r=roc
Flags: needinfo?(mstange)
https://hg.mozilla.org/integration/mozilla-inbound/rev/61217d41be19074ced225fcb09c281168ac7a6cd Bug 1201330 - Refactor LayerActivity property management. r=roc https://hg.mozilla.org/integration/mozilla-inbound/rev/1f3ea4c34877706356d43a1834228b1a7f91f18d Bug 1201330 - Keep scroll handler induced layer activity active until the scroll frame becomes inactive. r=roc
I had to add another nsWeakFrame, for the current scroll frame, because a scroll frame can be destroyed while it's firing a scroll event. The test that failed did that because it was listening for the scroll event in a window it had opened, and during the scroll event handler it closed that window.
Flags: needinfo?(mstange)
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
Depends on: 1240690
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: