now the test hits assertion even without flip the pref, bug 1365844
This one has lots of intermittent failures even without my patch and without flip the pref, Henry is fixing some in his bug 1355746. I'll wait to see if his patch could help the intermittent here.
Per Heny, bug 1355746 is not a blocker for this bug. Henry, please elaborate it. Thanks!
(In reply to Ethan Tseng [:ethan] from comment #4) > Per Heny, bug 1355746 is not a blocker for this bug. Henry, please > elaborate it. Thanks! The relevant patch in bug 1355746 is to fix test_fullscreen-api.html intermittent failure caused by a fullscreen focus issue on Linux so I am not expecting that patch to fix test_fullscreen-api-race.html
Created attachment 8894822 [details] [diff] [review] Patch.
Comment on attachment 8894822 [details] [diff] [review] Patch. // SimpleTest.waitForFocus will fire only when we use data: URI. sounds wrong. But if we don't want to change data: usage here, this is fine, but without that comment.
Created attachment 8895180 [details] [diff] [review] Patch v2. updated comments
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/f39222925a5f rewrite dom/html/test/test_fullscreen-api-race.html. r=smaug