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

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

Issue 280383002: Revert 269855 "Only commit cookie changes in prerenders after a ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 7 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: trunk/src/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
===================================================================
--- trunk/src/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java (revision 269877)
+++ trunk/src/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java (working copy)
@@ -21,14 +21,11 @@
* Tests for adding and removing prerenders using the {@link ExternalPrerenderHandler}
*/
public class ExternalPrerenderRequestTest extends ChromeShellTestBase {
- private static final String HOMEPAGE_URL =
- TestHttpServerClient.getUrl("chrome/test/data/android/prerender/homepage.html");
private static final String GOOGLE_URL =
TestHttpServerClient.getUrl("chrome/test/data/android/prerender/google.html");
private static final String YOUTUBE_URL =
TestHttpServerClient.getUrl("chrome/test/data/android/prerender/youtube.html");
private static final int PRERENDER_DELAY_MS = 500;
- private static final int CHECK_COOKIE_STORE_FREQUENCY_MS = 200;
private ExternalPrerenderHandler mHandler;
private Profile mProfile;
@@ -37,8 +34,7 @@
public void setUp() throws Exception {
super.setUp();
clearAppData();
- // Launch with a non-blank homepage, to trigger cookie store loading.
- launchChromeShellWithUrl(HOMEPAGE_URL);
+ launchChromeShellWithBlankPage();
assertTrue(waitForActiveShellToBeDoneLoading());
mHandler = new ExternalPrerenderHandler();
final Callable<Profile> profileCallable = new Callable<Profile>() {
@@ -48,8 +44,6 @@
}
};
mProfile = ThreadUtils.runOnUiThreadBlocking(profileCallable);
- while (!ExternalPrerenderHandler.hasCookieStoreLoaded(mProfile))
- Thread.sleep(CHECK_COOKIE_STORE_FREQUENCY_MS);
}
@MediumTest

Powered by Google App Engine
This is Rietveld 408576698