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

Unified Diff: LayoutTests/media/video-controls-fullscreen.js

Issue 217163004: Remove the fullScreenEnabled setting (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 9 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/media/video-controls-fullscreen.js
diff --git a/LayoutTests/media/video-controls-fullscreen.js b/LayoutTests/media/video-controls-fullscreen.js
index 6de12908d3e40f112f870adc326c0c2dbca261df..b1587d390331e3295be2e82e1eb9126c8d499302 100644
--- a/LayoutTests/media/video-controls-fullscreen.js
+++ b/LayoutTests/media/video-controls-fullscreen.js
@@ -1,10 +1,7 @@
"use strict";
-function fullscreen_test(controller, fullScreenEnabled)
+function fullscreen_test(controller)
{
- if (window.internals)
- window.internals.settings.setFullScreenEnabled(fullScreenEnabled);
-
async_test(function(t)
{
var v1 = document.createElement("video");
@@ -27,18 +24,12 @@ function fullscreen_test(controller, fullScreenEnabled)
// no fullscreen button for a video element with no video track
assert_button_hidden(v2);
- if (fullScreenEnabled) {
- // click the fullscreen button
- var coords = mediaControlsButtonCoordinates(v1, "fullscreen-button");
- eventSender.mouseMoveTo(coords[0], coords[1]);
- eventSender.mouseDown();
- eventSender.mouseUp();
- // wait for the fullscreenchange event
- } else {
- // no fullscreen button when fullscreen is disabled
- assert_button_hidden(v1);
- t.done();
- }
+ // click the fullscreen button
+ var coords = mediaControlsButtonCoordinates(v1, "fullscreen-button");
+ eventSender.mouseMoveTo(coords[0], coords[1]);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+ // wait for the fullscreenchange event
}));
v1.addEventListener("webkitfullscreenchange", t.step_func(function()
« no previous file with comments | « LayoutTests/media/video-controls-fullscreen.html ('k') | LayoutTests/media/video-controls-fullscreen-disabled.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698