Hiroyuki Ikezoe (:hiro) Assignee
	Description • 7 years ago

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.

	Hiroyuki Ikezoe (:hiro) Assignee
	Comment 1 • 7 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4fc65bcff4853e33b5708d5bb8893f37ec6d5cf1

	Brian Birtles (:birtles)
	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

	Pulsebot
	Comment 4 • 7 years ago

Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dd6ac8761b9d
Add content property to generate pseudo element. r=birtles

	Carsten Book [:Tomcat]
	Comment 5 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/dd6ac8761b9d

	Brian Birtles (:birtles)
	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

	Hiroyuki Ikezoe (:hiro) Assignee
	Comment 7 • 7 years ago

Ah, we have the same function in test_animation_observers.sync.html.

	Hiroyuki Ikezoe (:hiro) 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!

	Pulsebot
	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

	Carsten Book [:Tomcat]
	Comment 11 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/45a389969f25