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

Unified Diff: LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported.html

Issue 426593010: Set fullscreenEnabled to false when fullscreen is not supported. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 5 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: LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported.html
diff --git a/LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported.html b/LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported.html
new file mode 100644
index 0000000000000000000000000000000000000000..879719b11dc466b94700a08a25b36a65e434ccf6
--- /dev/null
+++ b/LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported.html
@@ -0,0 +1,15 @@
+<body>
+<div>This tests that the <code>fullscreenEnabled</code> property is false when fullscreen
+is not supported as laid out in section 4 of the
+<a href="http://fullscreen.spec.whatwg.org/#api">Fullscreen API</a> living
+standard</div>
+<script src="full-screen-test.js"></script>
+<script>
+ window.internals.settings.setFullscreenEnabled(false);
+
+ var iframe = document.documentElement.appendChild(document.createElement('iframe'));
philipj_slow 2014/08/04 09:24:13 You don't need an iframe to test this, just testin
Ignacio Solla 2014/08/04 11:43:33 I need it, 'allowfullscreen' is not a property of
philipj_slow 2014/08/04 12:18:51 For a top-level document, webkitFullscreenEnabled
+ iframe.setAttribute('allowfullscreen', 'true');
+ testExpected('iframe.contentDocument.webkitFullscreenEnabled', false);
+ endTest();
+</script>
+</body>

Powered by Google App Engine
This is Rietveld 408576698