Closed
Bug 1367307
Opened 8 years ago
Closed 7 years ago
stylo: GetAnimations() for pseudo elements does not work
Categories
(Core :: CSS Parsing and Computation, enhancement)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: hiro, Assigned: hiro)
References
Details
Attachments
(2 files)
Some tests (e.g. css-aniamtions/test_effect-target.html) in dom/animation/test fail.
Though I haven't looked in detail, I guess we need some tweaks there what Emilio does in bug 1331047.
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → hikezoe
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8875532 [details]
Bug 1367307 - Add content property to generate pseudo element.
https://reviewboard.mozilla.org/r/146962/#review151086
Attachment #8875532 -
Flags: review?(bbirtles) → review+
Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dd6ac8761b9d
Add content property to generate pseudo element. r=birtles
Comment 5•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Comment 6•7 years ago
|
||
I suspect createPseudo needs the same treatment:
http://searchfox.org/mozilla-central/rev/a798ee4fc323f9387b7576dbed177859d29d09b7/testing/web-platform/tests/web-animations/testcommon.js#81
Assignee | ||
Comment 7•7 years ago
|
||
Ah, we have the same function in test_animation_observers.sync.html.
Comment 8•7 years ago
|
||
Attachment #8875952 -
Flags: review?(hikezoe)
Assignee | ||
Comment 9•7 years ago
|
||
Comment on attachment 8875952 [details] [diff] [review]
Add content property to create pseudo elements in web-platform-tests and devtools tests too
Thanks for catching this!
Attachment #8875952 -
Flags: review?(hikezoe) → review+
Comment 10•7 years ago
|
||
Pushed by bbirtles@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/45a389969f25
Add content property to create pseudo element in web-platform-tests and devtools tests too; r=hiro
Comment 11•7 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•