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

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

Issue 1767243002: Update prerender policy for custom tabs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix indentation 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/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 00665868353f82f661541644214698fe52f37307..ba59d3b9a8e949417487efaa045b28a5ea923574 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
@@ -33,13 +33,15 @@ public class ExternalPrerenderHandler {
* @param referrer The referrer for the prerender request.
* @param width The width for the content view (render widget host view) for the prerender.
* @param height The height for the content view (render widget host view) for the prerender.
+ * @param prerenderOnMobile Whether the prerender should happen if the device has a mobile
+ * connection.
* @return The {@link WebContents} that is linked to this prerender. {@code null} if
* unsuccessful.
*/
public WebContents addPrerender(Profile profile, String url, String referrer, int width,
- int height) {
+ int height, boolean prerenderOnMobile) {
WebContents webContents = WebContentsFactory.createWebContents(false, false);
- if (addPrerender(profile, webContents, url, referrer, width, height)) {
+ if (addPrerender(profile, webContents, url, referrer, width, height, prerenderOnMobile)) {
return webContents;
}
if (webContents != null) webContents.destroy();
@@ -59,9 +61,9 @@ public class ExternalPrerenderHandler {
* @return Whether the prerender was successful.
*/
public boolean addPrerender(Profile profile, WebContents webContents, String url,
- String referrer, int width, int height) {
+ String referrer, int width, int height, boolean prerenderOnMobile) {
return nativeAddPrerender(mNativeExternalPrerenderHandler, profile, webContents,
- url, referrer, width, height);
+ url, referrer, width, height, prerenderOnMobile);
}
/**
@@ -101,7 +103,8 @@ public class ExternalPrerenderHandler {
private static native long nativeInit();
private static native boolean nativeAddPrerender(
long nativeExternalPrerenderHandlerAndroid, Profile profile,
- WebContents webContents, String url, String referrer, int width, int height);
+ WebContents webContents, String url, String referrer,
+ int width, int height, boolean prerenderOnMobile);
private static native boolean nativeHasPrerenderedUrl(
Profile profile, String url, WebContents webContents);
private static native boolean nativeHasPrerenderedAndFinishedLoadingUrl(

Powered by Google App Engine
This is Rietveld 408576698