[wpt-sync] Sync PR 18403 - Fix DCHECK when compositing additive CSS transform animations
Categories
(Core :: CSS Parsing and Computation, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 18403 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/18403
Details from upstream follow.
Stephen McGruer <smcgruer@chromium.org> wrote:
Fix DCHECK when compositing additive CSS transform animations
Previously CSSTransformNonInterpolableValue assumed that anything that was
composited on-top of would be single. However that is not true; in the simple
case where there is a replace animation before it, the replace effect will be a
pair:target.animate({ transform: [ 'scale(1)', 'scale(2)' ] }, 1000);
target.animate({ transform: [ 'rotate(0)', 'rotate(90deg)' ] },
{ duration: 1000, composite: 'add' });In the example above, the scale animation would not yet have been reduced to a
single interpolated value, so the DCHECK for is_single_ would fail. This CL
fixes the logic to perform an interpolation in that case.Bug: 979952
Change-Id: I36a3221c55aa3d98281a21a4ac93d95d61c5a1ff
Reviewed-on: https://chromium-review.googlesource.com/1750200
WPT-Export-Revision: 98d377eef0b37634ad9da4b5239303f82fdcff25
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=76daf9b8cafd29bd7fd23e99e594dddd60833d6d
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=c6c6af06a44acdb4c9919c69481e9101c6c66f74
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=4a0d0bb5286bc4244cd8c1ced65e8665ffe84691
Assignee | ||
Comment 4•5 years ago
|
||
Ran 1 tests and 1 subtests OK : 1 PASS : 1
Assignee | ||
Comment 5•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=c6ffd3ff92c51a5a7e32752a42e81f4bcc8178ad
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/33ec07b80b18 [wpt PR 18403] - Fix DCHECK when compositing additive CSS transform animations, a=testonly
Comment 7•5 years ago
|
||
bugherder |
Description
•