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

Side by Side Diff: LayoutTests/fullscreen/full-screen-remove-sibling.html

Issue 462253003: Remove early returns in fullscreen tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase 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 | Annotate | Revision Log
OLDNEW
1 <body> 1 <body>
2 <script src="full-screen-test.js"></script> 2 <script src="full-screen-test.js"></script>
3 <div id="one"> 3 <div id="one">
4 <div id="fullscreen"></div> 4 <div id="fullscreen"></div>
5 <div id="two"><span></span></div> 5 <div id="two"><span></span></div>
6 </div> 6 </div>
7 7
8 <script> 8 <script>
9 // Bail out early if the full screen API is not enabled or is missing:
10 if (Element.prototype.webkitRequestFullScreen == undefined) {
11 logResult(false, "Element.prototype.webkitRequestFullScreen == undefined");
12 endTest();
13 } else {
14 var callback; 9 var callback;
15 waitForEvent(document, 'webkitfullscreenchange', function() { 10 waitForEvent(document, 'webkitfullscreenchange', function() {
16 var one = document.getElementById('one'); 11 var one = document.getElementById('one');
17 var two = document.getElementById('two'); 12 var two = document.getElementById('two');
18 one.removeChild(two); 13 one.removeChild(two);
19 endTest(); 14 endTest();
20 }); 15 });
21 16
22 function goFullScreen() { 17 function goFullScreen() {
23 document.getElementById('fullscreen').webkitRequestFullScreen(); 18 document.getElementById('fullscreen').webkitRequestFullScreen();
24 } 19 }
25 runWithKeyDown(goFullScreen); 20 runWithKeyDown(goFullScreen);
26 }
27 </script> 21 </script>
OLDNEW
« no previous file with comments | « LayoutTests/fullscreen/full-screen-remove-children.html ('k') | LayoutTests/fullscreen/full-screen-render-inline.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698