1205938 - Enable fullscreen-api-race test on e10s

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Xidorn Quan [:xidorn] UTC+11]

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

Comment 2

[Xidorn Quan [:xidorn] UTC+11]

Try push looks good: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4d8b4c64969b

Comment 3

[Xidorn Quan [:xidorn] UTC+11]

Attachment: MozReview Request: Bug 1205938 part 1 - Add waitForCondition() function to SimpleTest. r?jmaher

This function is roughly copied from browser/base/content/test/general/head.js.
It has been widely used in browser chrome mochitests, and should be fine to
have it available to all mochitests via SimpleTest.

Review commit: https://reviewboard.mozilla.org/r/50305/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50305/

Comment 4

[Xidorn Quan [:xidorn] UTC+11]

Attachment: MozReview Request: Bug 1205938 part 2 - Enable fullscreen-api-race test on e10s and OS X. r?smaug

Review commit: https://reviewboard.mozilla.org/r/50307/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50307/

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8748469 [details]
MozReview Request: Bug 1205938 part 2 - Enable fullscreen-api-race test on e10s and OS X. r?smaug

https://reviewboard.mozilla.org/r/50307/#review47155

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8748468 [details]
MozReview Request: Bug 1205938 part 1 - Add waitForCondition() function to SimpleTest. r?jmaher

https://reviewboard.mozilla.org/r/50305/#review47335

this looks like a simple move of a function into simpletest- I have no issues with it.  Can we have a unittest for this?  we have testing/tests/Harness_sanity/test_*.html to test some of the simpletest functions- it would be nice to get basic coverage- ok with a followup bug.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8710b9ecc282
https://hg.mozilla.org/integration/mozilla-inbound/rev/7144610b5ce1

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/8710b9ecc282
https://hg.mozilla.org/mozilla-central/rev/7144610b5ce1