Closed Bug 1486894 Opened 7 years ago Closed 7 years ago

[wpt-sync] Sync PR 12728 - Fix fullscreen WPT tests assuming fullscreenElement changes each time.

Categories

(Core :: DOM: Core & HTML, enhancement, P4)

enhancement

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 12728 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/12728 Details from upstream follow. Dave Tapuska <dtapuska@chromium.org> wrote: > Fix fullscreen WPT tests assuming fullscreenElement changes each time. > > Since fullscreenchange event is sent by the rAF event loop it is possible > that the fullscreenElement might change and not be observed at the time > you read the event. Fix the tests so they are robust. > > Fix the ordering of which fullscreen elements are handled in pending > frames. Otherwise we might end up with incorrect ordering. > > BUG=874444 > > Change-Id: I00c228e0aea7794564e522a3883573ea359235f1 > Reviewed-on: https://chromium-review.googlesource.com/1194726 > WPT-Export-Revision: b2928ce20a7a01021a775a5e55bb64077dbe97d5
Component: web-platform-tests → DOM
Product: Testing → Core
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/7e8bbca1f17d [wpt PR 12728] - Fix fullscreen WPT tests assuming fullscreenElement changes each time., a=testonly
Result changes from PR not available.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.