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

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

Issue 639413002: [aw] Reuse WebContents PowerSaveBlocker in fullscreen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@testsForPowerSaver
Patch Set: Nits Created 6 years, 2 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 529d730a2e5502b416bcb30e70b1cc3d71649650..4ec4e40052098d7ed914582dd8b731a78480f294 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
@@ -35,17 +35,17 @@ public class VideoTestUtil {
observer.register(awContents.getContentViewCore(), "javaObserver");
}
});
- VideoTestWebServer webServer = new VideoTestWebServer(testCase.getActivity());
+ VideoTestWebServer webServer = new VideoTestWebServer();
try {
- String data = "<html><head><script>" +
- "addEventListener('DOMContentLoaded', function() { " +
- " document.getElementById('video').addEventListener('play', function() { " +
- " javaObserver.notifyJava(); " +
- " }, false); " +
- "}, false); " +
- "</script></head><body>" +
- "<video id='video' autoplay control src='" +
- webServer.getOnePixelOneFrameWebmURL() + "' /> </body></html>";
+ String data = "<html><head><script>"
+ + "addEventListener('DOMContentLoaded', function() { "
+ + " document.getElementById('video').addEventListener('play', function() { "
+ + " javaObserver.notifyJava(); "
+ + " }, false); "
+ + "}, false); "
+ + "</script></head><body>"
+ + "<video id='video' autoplay control src='"
+ + webServer.getOnePixelOneFrameWebmURL() + "' /> </body></html>";
testCase.loadDataAsync(awContents, data, "text/html", false);
return observer.waitForEvent(waitTime);
} finally {

Powered by Google App Engine
This is Rietveld 408576698