Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(116)

Unified Diff: third_party/WebKit/LayoutTests/fullscreen/api/element-request-fullscreen-two-iframes.html

Issue 2855843002: Reland "Sync requestFullscreen() and exitFullscreen() algorithms with the spec" (Closed)
Patch Set: Reland "Sync requestFullscreen() and exitFullscreen() algorithms with the spec" Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fullscreen/api/element-request-fullscreen-two-iframes.html
diff --git a/third_party/WebKit/LayoutTests/fullscreen/api/element-request-fullscreen-two-iframes.html b/third_party/WebKit/LayoutTests/fullscreen/api/element-request-fullscreen-two-iframes.html
index 8d6923cc90594263249f1d28acc4ae1a7b0543bc..a58a669c0c9afaf9158332eb8398e679f2491213 100644
--- a/third_party/WebKit/LayoutTests/fullscreen/api/element-request-fullscreen-two-iframes.html
+++ b/third_party/WebKit/LayoutTests/fullscreen/api/element-request-fullscreen-two-iframes.html
@@ -18,15 +18,15 @@ async_test(t => {
// Expect first a fullscreenchange event for the second (!) request, then a
// fullscreenerror event for the first request. TODO(foolip): Remove the
// Fullscreen hierarchy restrictions. https://crbug.com/627792
- a.contentDocument.onfullscreenerror = t.step_func(() => {
- b.contentDocument.onfullscreenchange = t.step_func_done(() => {
- assert_equals(document.fullscreenElement, b, 'fullscreenElement');
- assert_equals(a.contentDocument.fullscreenElement, null, 'fullscreenElement in iframe a');
- assert_equals(b.contentDocument.fullscreenElement, b.contentDocument.body, 'fullscreenElement in iframe b');
+ a.contentDocument.onfullscreenchange = t.step_func_done(() => {
+ assert_equals(document.fullscreenElement, a, 'fullscreenElement');
+ assert_equals(a.contentDocument.fullscreenElement, a.contentDocument.body, 'fullscreenElement in iframe a');
+ b.contentDocument.onfullscreenerror = t.step_func(() => {
+ assert_equals(b.contentDocument.fullscreenElement, null, 'fullscreenElement in iframe b');
});
});
- a.contentDocument.onfullscreenchange = t.unreached_func('fullscreenchange event in iframe a');
- b.contentDocument.onfullscreenerror = t.unreached_func('fullscreenerror event in iframe b');
+ a.contentDocument.onfullscreenerror = t.unreached_func('fullscreenerror event in iframe a');
+ b.contentDocument.onfullscreenchange = t.unreached_func('fullscreenchange event in iframe b');
trusted_click(t, () => {
b.contentDocument.body.requestFullscreen();

Powered by Google App Engine
This is Rietveld 408576698