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

Unified Diff: third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/element-ready-check-fullscreen-iframe-child-manual.html

Issue 2547023002: Import wpt@3c8896ae408c8fd594979da7c99970029e7856a7 (Closed)
Patch Set: Modify TestExpectations or download new baselines for tests. Created 4 years 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/imported/wpt/fullscreen/api/element-ready-check-fullscreen-iframe-child-manual.html
diff --git a/third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/element-ready-check-fullscreen-iframe-child-manual.html b/third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/element-ready-check-fullscreen-iframe-child-manual.html
index 598fcfecf95db393d2d7bb7e2c781064dc7b15e0..08febbd4f967c79be8285308f61ed082b8bd9c82 100644
--- a/third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/element-ready-check-fullscreen-iframe-child-manual.html
+++ b/third_party/WebKit/LayoutTests/imported/wpt/fullscreen/api/element-ready-check-fullscreen-iframe-child-manual.html
@@ -8,21 +8,26 @@
<script>
// Verify that an iframe can itself go fullscreen, and that this doesn't
// influence the iframe ancestor test of the element ready check.
-async_test(function(t)
-{
- var iframe = document.querySelector("iframe");
- document.onfullscreenchange = t.step_func(function()
- {
- assert_equals(document.fullscreenElement, iframe, "fullscreen element");
- var div = document.createElement("div");
- // This adds the div to the iframe element itself, not to the iframe's
- // contentDocument. It's done here because the HTML parser treats the
- // content of iframe as a text node.
- iframe.appendChild(div);
- trusted_request(div, iframe.contentDocument.body);
- document.onfullscreenchange = t.unreached_func("second fullscreenchange event");
- document.onfullscreenerror = t.step_func_done();
+async_test(t => {
+ var iframe = document.querySelector("iframe");
+
+ document.onfullscreenchange = t.step_func(() => {
+ assert_equals(document.fullscreenElement, iframe, "fullscreen element");
+
+ // This adds the div to the iframe element itself, not to the iframe's
+ // contentDocument. It's done here because the HTML parser treats the
+ // content of iframe as a text node.
+ var div = document.createElement("div");
+ iframe.appendChild(div);
+
+ document.onfullscreenchange = t.step_func_done(() => {
+ assert_equals(document.fullscreenElement, div);
});
- trusted_request(iframe);
+ document.onfullscreenerror = t.unreached_func("fullscreenerror event");
+
+ trusted_request(div, iframe.contentDocument.body);
+ });
+
+ trusted_request(iframe);
});
</script>

Powered by Google App Engine
This is Rietveld 408576698