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

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

Issue 1710853002: android: Add a method to let Java know whether a prerender has finished loading. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Indentation. Created 4 years, 10 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/java/src/org/chromium/chrome/browser/prerender/ExternalPrerenderHandler.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/prerender/ExternalPrerenderHandler.java b/chrome/android/java/src/org/chromium/chrome/browser/prerender/ExternalPrerenderHandler.java
index 71cc2cba62a541d15760cef88281517ae3859381..25c072dab7c0278b664b7b6d78410b3e5c8c2b9b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/prerender/ExternalPrerenderHandler.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/prerender/ExternalPrerenderHandler.java
@@ -4,6 +4,7 @@
package org.chromium.chrome.browser.prerender;
+import org.chromium.base.VisibleForTesting;
import org.chromium.base.annotations.JNINamespace;
import org.chromium.chrome.browser.WebContentsFactory;
import org.chromium.chrome.browser.profiles.Profile;
@@ -82,12 +83,28 @@ public class ExternalPrerenderHandler {
return nativeHasPrerenderedUrl(profile, url, webContents);
}
+ /**
+ * Check whether a given url has been prerendering for the given profile and session id for the
+ * given web contents, and has finished loading.
+ * @param profile The profile to check for prerendering.
+ * @param url The url to check for prerender.
+ * @param webContents The {@link WebContents} for which to compare the session info.
+ * @return Whether the given url was prerendered and has finished loading.
+ */
+ @VisibleForTesting
+ public static boolean hasPrerenderedAndFinishedLoadingUrl(
+ Profile profile, String url, WebContents webContents) {
+ return nativeHasPrerenderedAndFinishedLoadingUrl(profile, url, webContents);
+ }
+
private static native long nativeInit();
private static native boolean nativeAddPrerender(
long nativeExternalPrerenderHandlerAndroid, Profile profile,
WebContents webContents, String url, String referrer, int width, int height);
private static native boolean nativeHasPrerenderedUrl(
Profile profile, String url, WebContents webContents);
+ private static native boolean nativeHasPrerenderedAndFinishedLoadingUrl(
+ Profile profile, String url, WebContents webContents);
private static native void nativeCancelCurrentPrerender(
long nativeExternalPrerenderHandlerAndroid);
}

Powered by Google App Engine
This is Rietveld 408576698