Closed
Bug 1584430
Opened 5 years ago
Closed 5 years ago
[wpt-sync] Sync PR 19353 - [ScrollTimeline] Remove CompositingReason::kScrollTimelineTarget
Categories
(Testing :: web-platform-tests, task, P4)
Testing
web-platform-tests
Tracking
(firefox71 fixed)
RESOLVED
FIXED
mozilla71
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 19353 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/19353
Details from upstream follow.
Yi Gu <yigu@chromium.org> wrote:
[ScrollTimeline] Remove CompositingReason::kScrollTimelineTarget
Since we have supported main-thread AnimationWorklet, there is no need
to promote the scroll-source automatically for ScrollTimeline.Changes to make:
- Attaching / detaching animation to ScrollTimeline no longer need to
trigger compositing update nor paint property update.- Animations with scroll timeline should fall back to main thread
when the scroll source is not composited.- ScrollTimeline::HasActiveScrollTimeline which used to help with
making compositing decisions can be removed. However, an alternative
should be implemented because it also optimized the compositing state
change updates [1].Bug:1008720
Change-Id: I07dbefeb4e16e8be832b354f0a68bfd7cd0f6ee6
Reviewed-on: https://chromium-review.googlesource.com/1825720
WPT-Export-Revision: 6743a785db4e21069378f7dc31a31c443c498af6
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=576a432a1cc633f6262d5470124f5a0302c53372
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=bd88231881a1282054761261631585f66af4a24a
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=24cde2896eb83126f43e62b610f5567b88331b3b
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=f20d3799842da96e422a4f111f8235c655cffcdc
Assignee | ||
Comment 5•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=65073af8c90a146d133f74baef150050b0828f31
Assignee | ||
Comment 6•5 years ago
|
||
Ran 2 tests TIMEOUT: 2
Assignee | ||
Comment 7•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=61e8028954256379159a19d63c5e584a46fdb427
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/bb30c80321ba [wpt PR 19353] - [ScrollTimeline] Remove CompositingReason::kScrollTimelineTarget, a=testonly
Comment 9•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in
before you can comment on or make changes to this bug.
Description
•