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

Side by Side 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: Address review comments Created 6 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <body>
2 <div>This tests that the <code>fullscreenSupported</code> property is false when fullscreen
philipj_slow 2014/08/04 12:23:01 Oops, this should say webkitFullscreenEnabled I gu
Ignacio Solla 2014/08/04 12:31:58 Done.
3 is not supported as laid out in section 4 of the
4 <a href="http://fullscreen.spec.whatwg.org/#api">Fullscreen API</a> living
5 standard</div>
6 <script src="full-screen-test.js"></script>
7 <script>
8 window.internals.settings.setFullscreenSupported(false);
9 var iframe = document.documentElement.appendChild(document.createElement('if rame'));
10 iframe.setAttribute('allowfullscreen', 'true');
11
12 testExpected('iframe.contentDocument.webkitFullscreenEnabled', false);
13 endTest();
14 </script>
15 </body>
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/fullscreen/full-screen-not-enabled-when-unsupported-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698