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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java

Issue 1767243002: Update prerender policy for custom tabs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mmenke@'s comments Created 4 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: chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
index de527e95d83276c2035d41bd579bd2c2276ac905..1c3da1501ddfe314db3ab5d501d5d0a11b922c07 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
@@ -79,7 +79,7 @@ public class ExternalPrerenderRequestTest extends ChromeActivityTestCaseBase<Chr
* Test adding a prerender and canceling that to add a new one.
*/
public void testAddPrerenderAndCancel() throws InterruptedException {
- WebContents webContents = mHandler.addPrerender(mProfile, mGoogleUrl, "", 0, 0);
+ WebContents webContents = mHandler.addPrerender(mProfile, mGoogleUrl, "", 0, 0, false);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, mGoogleUrl, webContents));
@@ -88,7 +88,7 @@ public class ExternalPrerenderRequestTest extends ChromeActivityTestCaseBase<Chr
mProfile, mGoogleUrl, webContents));
webContents.destroy();
Thread.sleep(PRERENDER_DELAY_MS);
- webContents = mHandler.addPrerender(mProfile, mYoutubeUrl, "", 0, 0);
+ webContents = mHandler.addPrerender(mProfile, mYoutubeUrl, "", 0, 0, false);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, mYoutubeUrl, webContents));
}
@@ -102,7 +102,7 @@ public class ExternalPrerenderRequestTest extends ChromeActivityTestCaseBase<Chr
*/
public void testCancelPrerender() {
mHandler.cancelCurrentPrerender();
- WebContents webContents = mHandler.addPrerender(mProfile, mGoogleUrl, "", 0, 0);
+ WebContents webContents = mHandler.addPrerender(mProfile, mGoogleUrl, "", 0, 0, false);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, mGoogleUrl, webContents));
}
@@ -115,11 +115,11 @@ public class ExternalPrerenderRequestTest extends ChromeActivityTestCaseBase<Chr
* Test adding two prerenders without canceling the first one.
*/
public void testAddingPrerendersInaRow() throws InterruptedException {
- WebContents webContents = mHandler.addPrerender(mProfile, mGoogleUrl, "", 0, 0);
+ WebContents webContents = mHandler.addPrerender(mProfile, mGoogleUrl, "", 0, 0, false);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, mGoogleUrl, webContents));
Thread.sleep(PRERENDER_DELAY_MS);
- WebContents newWebContents = mHandler.addPrerender(mProfile, mYoutubeUrl, "", 0, 0);
+ WebContents newWebContents = mHandler.addPrerender(mProfile, mYoutubeUrl, "", 0, 0, false);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, mYoutubeUrl, newWebContents));
}

Powered by Google App Engine
This is Rietveld 408576698