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

Unified Diff: android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestUtil.java

Issue 1025933002: Convert force video overlay to preference (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review Created 5 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: android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestUtil.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestUtil.java b/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestUtil.java
index 4ec4e40052098d7ed914582dd8b731a78480f294..fca2208fa0b3cf26ba65490fa2d124a846a8c67b 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestUtil.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestUtil.java
@@ -21,7 +21,7 @@ public class VideoTestUtil {
* @throws Throwable throw exception if timeout.
*/
public static boolean runVideoTest(final AwTestBase testCase, final boolean requiredUserGesture,
- long waitTime) throws Throwable {
+ final boolean forceVideoOverlay, long waitTime) throws Throwable {
final JavascriptEventObserver observer = new JavascriptEventObserver();
TestAwContentsClient client = new TestAwContentsClient();
final AwContents awContents =
@@ -32,6 +32,7 @@ public class VideoTestUtil {
AwSettings awSettings = awContents.getSettings();
awSettings.setJavaScriptEnabled(true);
awSettings.setMediaPlaybackRequiresUserGesture(requiredUserGesture);
+ awSettings.setForceVideoOverlayForTests(forceVideoOverlay);
observer.register(awContents.getContentViewCore(), "javaObserver");
}
});

Powered by Google App Engine
This is Rietveld 408576698