Closed Bug 1665941 Opened 4 years ago Closed 4 years ago

Add more extensive cross-origin tests for fullscreen

Categories

(Core :: DOM: Navigation, task, P2)

task

Tracking

()

RESOLVED FIXED
83 Branch
Fission Milestone M6c
Tracking Status
firefox83 --- fixed

People

(Reporter: smacleod, Assigned: smacleod)

References

Details

Attachments

(3 files)

No description provided.

In order to more thoroughly test fullscreen with Fission, this change
introduces a new WPT with a more complicated tree of cross-origin
iframes. This is a manual test since WPT can't fake user interaction
in a cross-origin iframe yet. Once this is supported it should be
possible to call requestFullscreen() in automation and convert
this test.

Pushed by smacleod@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b4865afae8fa add WPT for requestFullscreen in a tree of cross-origin iframes. r=smaug DONTBUILD
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/25652 for changes under testing/web-platform/tests
Keywords: leave-open
Upstream PR merged by moz-wptsync-bot

In order to more thoroughly test fullscreen with Fission, this change
introduces a new mochitest with a more complicated tree of
cross-origin iframes. The test ensures we're firing the proper events
in the correct documents, as well as setting and unsetting
document.fullscreenElement.

This expands the current requestFullscreen() mochitest to check the
result of exiting fullscreen using the ESC key, since it follows a
separate code path from document.exitFullscreen().

Depends on D90928

Pushed by smacleod@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fae2017a1aa7 add mochitest for requestFullscreen in a tree of cross-origin iframes. r=smaug https://hg.mozilla.org/integration/autoland/rev/6d45a17995e2 expand cross-origin requestFullscreen mochitest to test ESC exit. r=smaug
Keywords: leave-open
Severity: -- → S3
Fission Milestone: --- → M6c
Priority: -- → P2
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
Regressions: 1667286
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: