Closed Bug 1367307 Opened 3 years ago Closed 3 years ago

stylo: GetAnimations() for pseudo elements does not work

Categories

(Core :: CSS Parsing and Computation, enhancement)

enhancement
Not set

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: nobody → hikezoe
Status: NEW → ASSIGNED
Blocks: 1337599
No longer blocks: 1302945
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
https://hg.mozilla.org/mozilla-central/rev/dd6ac8761b9d
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Ah, we have the same function in test_animation_observers.sync.html.
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+
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
You need to log in before you can comment on or make changes to this bug.