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

Unified Diff: LayoutTests/fullscreen/full-screen-remove-ancestor-after.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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fullscreen/full-screen-remove-ancestor-after.html
diff --git a/LayoutTests/fullscreen/full-screen-remove-ancestor-after.html b/LayoutTests/fullscreen/full-screen-remove-ancestor-after.html
index 0484ed499b3bce1cefe862746785e2548384e6ba..050f8bba0e5e4070b4ad2d5b2cce3c9c10b4b0bf 100644
--- a/LayoutTests/fullscreen/full-screen-remove-ancestor-after.html
+++ b/LayoutTests/fullscreen/full-screen-remove-ancestor-after.html
@@ -2,44 +2,38 @@
<script>
var runPixelTests = true;
var init = function() {
- // Bail out early if the full screen API is not enabled or is missing:
- if (Element.prototype.webkitRequestFullScreen == undefined) {
- logResult(false, "Element.prototype.webkitRequestFullScreen == undefined");
- endTest();
- } else {
- var callback;
- var fullscreenChanged = function(event)
- {
- if (callback)
- callback(event)
- };
- document.onwebkitfullscreenchange = fullscreenChanged;
+ var callback;
+ var fullscreenChanged = function(event)
+ {
+ if (callback)
+ callback(event)
+ };
+ document.onwebkitfullscreenchange = fullscreenChanged;
- var one = document.getElementById('one');
- var two = document.getElementById('two');
- var three = document.getElementById('three');
+ var one = document.getElementById('one');
+ var two = document.getElementById('two');
+ var three = document.getElementById('three');
- var threeEnteredFullScreen = function(event) {
- if (document.webkitCurrentFullScreenElement == three) {
- callback = threeExitedFullScreen;
- one.removeChild(two);
- }
- };
+ var threeEnteredFullScreen = function(event) {
+ if (document.webkitCurrentFullScreenElement == three) {
+ callback = threeExitedFullScreen;
+ one.removeChild(two);
+ }
+ };
- var threeExitedFullScreen = function(event) {
- if (document.webkitCurrentFullScreenElement == null) {
- callback = null;
- endTest();
- }
- };
+ var threeExitedFullScreen = function(event) {
+ if (document.webkitCurrentFullScreenElement == null) {
+ callback = null;
+ endTest();
+ }
+ };
- callback = threeEnteredFullScreen;
+ callback = threeEnteredFullScreen;
- function goFullScreen() {
- document.getElementById('three').webkitRequestFullScreen();
- }
- runWithKeyDown(goFullScreen);
+ function goFullScreen() {
+ document.getElementById('three').webkitRequestFullScreen();
}
+ runWithKeyDown(goFullScreen);
};
</script>
<script src="full-screen-test.js"></script>

Powered by Google App Engine
This is Rietveld 408576698