There is some non-trivial work to implement this on top of having requestFullscreen() return Promise, so I decide to spawn it into a separate bug from bug 1188256.

The next patch would create another subclass of FullscreenChange for handling fullscreen exit. Depends on D5988

Depends on D5989

Depends on D5990

Comment on attachment 9009506 [details] Bug 1491212 part 1 - Move PendingFullscreenRequestList to a higher place. r=smaug Olli Pettay [:smaug] has approved the revision.

Comment on attachment 9009509 [details] Bug 1491212 part 4 - Fix a wpt of exitFullscreen to check against the right Promise. r=smaug Olli Pettay [:smaug] (r- if the bug doesn't explain what the change(s) are about.) has approved the revision.

Comment on attachment 9009508 [details] Bug 1491212 part 3 - Have Document.exitFullscreen return a Promise. r=smaug Olli Pettay [:smaug] (r- if the bug doesn't explain what the change(s) are about.) has approved the revision.

Comment on attachment 9009507 [details] Bug 1491212 part 2 - Have a new FullscreenChange superclass split from FullscreenRequest. r=smaug Olli Pettay [:smaug] (r- if the bug doesn't explain what the change(s) are about.) has approved the revision.

Pushed by xquan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/12e1e7b2f4a7 part 1 - Move PendingFullscreenRequestList to a higher place. r=smaug https://hg.mozilla.org/integration/autoland/rev/824f566f9043 part 2 - Have a new FullscreenChange superclass split from FullscreenRequest. r=smaug https://hg.mozilla.org/integration/autoland/rev/32b021561f8c part 3 - Have Document.exitFullscreen return a Promise. r=smaug https://hg.mozilla.org/integration/autoland/rev/b444a3e482d8 part 4 - Fix a wpt of exitFullscreen to check against the right Promise. r=smaug

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/13041 for changes under testing/web-platform/tests

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

https://hg.mozilla.org/mozilla-central/rev/12e1e7b2f4a7 https://hg.mozilla.org/mozilla-central/rev/824f566f9043 https://hg.mozilla.org/mozilla-central/rev/32b021561f8c https://hg.mozilla.org/mozilla-central/rev/b444a3e482d8

Upstream PR merged

Can't merge web-platform-tests PR due to failing upstream checks: Github PR https://github.com/web-platform-tests/wpt/pull/13041 * Taskcluster (pull_request) (https://tools.taskcluster.net/task-group-inspector/#/aKOzgjrLRCG9H_HYX58ocQ)

Updated documentation as part of the big push on Fullscreen docs, and listed this change on Firefox 64 for developers. https://developer.mozilla.org/en-US/docs/Web/API/Document/exitFullscreen