Closed
Bug 1766046
Opened 2 years ago
Closed 2 years ago
Simplify animation-name implementation.
Categories
(Core :: CSS Parsing and Computation, task)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
101 Branch
Tracking | Status | |
---|---|---|
firefox101 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•2 years ago
|
||
Make the representation the same between Gecko and Servo code. This will
enable further clean-ups in the future.
Make serialization be correct, serializing as identifier unless it's an
invalid one (in which case we serialize as a string).
This changes our stringification behavior in the specified style, but
now it will match the computed style and be more
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dfec425c53fe cleanup animation-name. r=boris
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/33769 for changes under testing/web-platform/tests
Comment 4•2 years ago
|
||
Backed out for causing wpt failures at animation-timeline-computed.html.
Backout link: https://hg.mozilla.org/integration/autoland/rev/716059caf59ef1fedbec1a534ac542c8d010535b
Failure log:
https://treeherder.mozilla.org/logviewer?job_id=375639592&repo=autoland&lineNumber=12777
https://treeherder.mozilla.org/logviewer?job_id=375635999&repo=autoland&lineNumber=4444
Flags: needinfo?(emilio)
Updated•2 years ago
|
Attachment #9273480 -
Attachment description: Bug 1766046 - cleanup animation-name. r=boris,hiro,#layout-reviewers,#style → Bug 1766046 - cleanup animation-name. r=boris
Upstream PR was closed without merging
Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/742cc0583a14 cleanup animation-name. r=boris
Comment 7•2 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox101:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
Upstream PR merged by moz-wptsync-bot
You need to log in
before you can comment on or make changes to this bug.
Description
•