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

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

Issue 639923003: [aw] Fullscreen tests for non-media elements. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@fullscreenNonMediaForReview
Patch Set: Add copyright notice to full_screen_video.js 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/VideoTestWebServer.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestWebServer.java b/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestWebServer.java
index 5f244f402b43f03691c5424dbeab38e922360bca..f255cf66e34722503d695359695ced9afd806e30 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestWebServer.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/util/VideoTestWebServer.java
@@ -19,7 +19,8 @@ import java.util.List;
*/
public class VideoTestWebServer {
- // VIDEO_ID must be kept in sync with the id in full_screen_video_test.html.
+ // VIDEO_ID must be kept in sync with the id in VIDEO_TEST_HTML_FILE and
+ // VIDEO_INSIDE_DIV_TEST_HTML_FILE.
public static final String VIDEO_ID = "video";
public static final String ONE_PIXEL_ONE_FRAME_WEBM_FILENAME = "one_pixel_one_frame.webm";
public static final String ONE_PIXEL_ONE_FRAME_WEBM_BASE64 =
@@ -28,9 +29,14 @@ public class VideoTestWebServer {
"dW5khkAFVl9WUDglhohAA1ZQOIOBAeBABrCBlrqBlh9DtnVAdOeBAKNAboEAAIDyCACdASqWAJYA" +
"Pk0ci0WD+IBAAJiWlu4XdQTSq2H4MW0+sMO0gz8HMRe+0jRo0aNGjRo0aNGjRo0aNGjRo0aNGjRo" +
"0aNGjRo0aNGjRo0VAAD+/729RWRzH4mOZ9/O8Dl319afX4gsgAAA";
+ private static final String VIDEO_TEST_HTML_FILE = "full_screen_video_test.html";
+ private static final String VIDEO_INSIDE_DIV_TEST_HTML_FILE =
+ "full_screen_video_inside_div_test.html";
+ private static final String VIDEO_TEST_JS_FILE = "full_screen_video.js";
private String mOnePixelOneFrameWebmURL;
private String mFullScreenVideoTestURL;
+ private String mFullScreenVideoInsideDivTestURL;
private TestWebServer mTestWebServer;
public VideoTestWebServer(Context context) throws Exception {
@@ -39,7 +45,12 @@ public class VideoTestWebServer {
mOnePixelOneFrameWebmURL = mTestWebServer.setResponseBase64("/" +
ONE_PIXEL_ONE_FRAME_WEBM_FILENAME,
ONE_PIXEL_ONE_FRAME_WEBM_BASE64, headers);
- initFullScreenVideoTest(context);
+ mFullScreenVideoTestURL = initFullScreenVideoTest(context, VIDEO_TEST_HTML_FILE);
+ mFullScreenVideoInsideDivTestURL =
+ initFullScreenVideoTest(context, VIDEO_INSIDE_DIV_TEST_HTML_FILE);
+ mTestWebServer.setResponse("/" + VIDEO_TEST_JS_FILE,
+ loadAssetData(context, VIDEO_TEST_JS_FILE),
+ getHTMLHeaders(false));
}
/**
@@ -53,6 +64,10 @@ public class VideoTestWebServer {
return mFullScreenVideoTestURL;
}
+ public String getFullScreenVideoInsideDivTestURL() {
+ return mFullScreenVideoInsideDivTestURL;
+ }
+
public TestWebServer getTestWebServer() {
return mTestWebServer;
}
@@ -65,10 +80,10 @@ public class VideoTestWebServer {
return CommonResources.getContentTypeAndCacheHeaders("text/html", disableCache);
}
- private void initFullScreenVideoTest(Context context) throws IOException {
- final String fullScreenVideoPath = "full_screen_video_test.html";
- String data = loadAssetData(context, fullScreenVideoPath);
- mFullScreenVideoTestURL = mTestWebServer.setResponse("/" + fullScreenVideoPath,
+ private String initFullScreenVideoTest(Context context, final String videoTestHtmlFile)
+ throws IOException {
+ String data = loadAssetData(context, videoTestHtmlFile);
+ return mTestWebServer.setResponse("/" + videoTestHtmlFile,
data.replace("VIDEO_FILE_URL", getOnePixelOneFrameWebmURL()),
getHTMLHeaders(false));
}

Powered by Google App Engine
This is Rietveld 408576698