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

Unified Diff: chrome/browser/prerender/prerender_browsertest.cc

Issue 8095007: Defer loading of audio/video tags while prerendering. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Addressing Andrew's comments Created 9 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
« no previous file with comments | « no previous file | chrome/chrome_renderer.gypi » ('j') | chrome/renderer/prerender/prerender_helper.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_browsertest.cc
diff --git a/chrome/browser/prerender/prerender_browsertest.cc b/chrome/browser/prerender/prerender_browsertest.cc
index 96c9f7772fa9e5c291e8ba0ce6b1aeeadf236aa2..10140a6fb987d4d193c6f1eabbf9d623ae582793 100644
--- a/chrome/browser/prerender/prerender_browsertest.cc
+++ b/chrome/browser/prerender/prerender_browsertest.cc
@@ -1236,26 +1236,29 @@ IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderTaskManager) {
EXPECT_LE(num_tabs_with_prerender_page_title, 2);
}
-// Checks that prerenderers will terminate when an audio tag is encountered.
+// Checks that prerenderers will work when an audio tag is encountered.
IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderHTML5Audio) {
PrerenderTestURL("files/prerender/prerender_html5_audio.html",
- FINAL_STATUS_HTML5_MEDIA,
+ FINAL_STATUS_USED,
1);
+ NavigateToDestURL();
}
-// Checks that prerenderers will terminate when a video tag is encountered.
+// Checks that prerenderers will work when a video tag is encountered.
IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderHTML5Video) {
PrerenderTestURL("files/prerender/prerender_html5_video.html",
- FINAL_STATUS_HTML5_MEDIA,
+ FINAL_STATUS_USED,
1);
+ NavigateToDestURL();
}
-// Checks that prerenderers will terminate when a video tag is inserted via
+// Checks that prerenderers will be used when a video tag is inserted via
// javascript.
IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderHTML5VideoJs) {
PrerenderTestURL("files/prerender/prerender_html5_video_script.html",
- FINAL_STATUS_HTML5_MEDIA,
+ FINAL_STATUS_USED,
1);
+ NavigateToDestURL();
}
// Checks that scripts can retrieve the correct window size while prerendering.
« no previous file with comments | « no previous file | chrome/chrome_renderer.gypi » ('j') | chrome/renderer/prerender/prerender_helper.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698