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

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

Issue 45693002: Add ExternalPrerenderRequestHandler and related API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Nit fixes and fixing the test Created 7 years, 1 month 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
new file mode 100644
index 0000000000000000000000000000000000000000..43bbaf4bd66d6fd15c8c9272b273a319d1c045a2
--- /dev/null
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
@@ -0,0 +1,63 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.chrome.browser.prerender;
+
+import android.test.suitebuilder.annotation.SmallTest;
+
+import org.chromium.base.ThreadUtils;
+import org.chromium.base.test.util.Feature;
+import org.chromium.chrome.browser.profiles.Profile;
+import org.chromium.chrome.test.util.TestHttpServerClient;
+import org.chromium.chrome.testshell.ChromiumTestShellTestBase;
+
+import java.util.concurrent.Callable;
+import java.util.concurrent.ExecutionException;
+
+/**
+ * Tests for adding and removing prerenders using the {@link ExternalPrerenderHandler}
+ */
+public class ExternalPrerenderRequestTest extends ChromiumTestShellTestBase {
+ private static final String GOOGLE_URL =
+ TestHttpServerClient.getUrl("clank/test/data/android/prerender/google.html");
+ private ExternalPrerenderHandler mHandler;
+ @Override
+ public void setUp() throws Exception {
+ super.setUp();
+ clearAppData();
+ launchChromiumTestShellWithBlankPage();
+ assertTrue(waitForActiveShellToBeDoneLoading());
+ mHandler = new ExternalPrerenderHandler();
+ }
+
+ @SmallTest
+ @Feature({"Prerender"})
+ public void testAddPrerender() throws ExecutionException {
cbentzel 2013/11/01 20:23:29 I think you need to add more tests... - Multiple
Yusuf 2013/11/04 22:27:49 Done.
+ final Callable<Profile> profileCallable = new Callable<Profile>() {
+ @Override
+ public Profile call() throws Exception {
+ return Profile.getLastUsedProfile();
+ }
+ };
+ final Profile profile = ThreadUtils.runOnUiThreadBlocking(profileCallable);
+ final Callable<Integer> addCallable = new Callable<Integer>() {
+ @Override
+ public Integer call() throws Exception {
+ return mHandler.addPrerender(profile, GOOGLE_URL, "", 0, 0);
+ }
+ };
+ int webContentsPtr = ThreadUtils.runOnUiThreadBlocking(addCallable);
+ assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
+ profile, GOOGLE_URL, webContentsPtr));
+ final Runnable cancelRunnable = new Runnable() {
+ @Override
+ public void run() {
+ mHandler.cancelCurrentPrerender();
+ }
+ };
+ ThreadUtils.runOnUiThreadBlocking(cancelRunnable);
+ assertFalse(ExternalPrerenderHandler.hasPrerenderedUrl(
+ profile, GOOGLE_URL, webContentsPtr));
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698