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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabsConnection.java

Issue 2179783002: customtabs: Histograms for special sessions prerendering. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. Created 4 years, 4 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/customtabs/CustomTabsConnection.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabsConnection.java b/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabsConnection.java
index b71e2c2d6cef81763a7867617648f20445d9303c..92d81df57927bd35b64bcdaffeb0f8af8842f78c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabsConnection.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabsConnection.java
@@ -33,6 +33,7 @@ import org.chromium.base.ThreadUtils;
import org.chromium.base.VisibleForTesting;
import org.chromium.base.annotations.SuppressFBWarnings;
import org.chromium.base.library_loader.ProcessInitException;
+import org.chromium.base.metrics.RecordHistogram;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeApplication;
import org.chromium.chrome.browser.IntentHandler;
@@ -464,13 +465,19 @@ public class CustomTabsConnection {
boolean urlsMatch = TextUtils.equals(prerenderedUrl, url)
|| (ignoreFragments
&& UrlUtilities.urlsMatchIgnoringFragments(prerenderedUrl, url));
+ WebContents result = null;
if (urlsMatch && TextUtils.equals(prerenderReferrer, referrer)) {
+ result = webContents;
mPrerender = null;
- return webContents;
} else {
cancelPrerender(session);
}
- return null;
+ if (!mClientManager.usesDefaultSessionParameters(session) && webContents != null) {
+ RecordHistogram.recordBooleanHistogram(
+ "CustomTabs.NonDefaultSessionPrerenderMatched", result != null);
+ }
+
+ return result;
}
/** Returns the URL prerendered for a session, or null. */
@@ -765,6 +772,10 @@ public class CustomTabsConnection {
if (webContents == null) return false;
if (throttle) mClientManager.registerPrerenderRequest(uid, url);
mPrerender = new PrerenderedUrlParams(session, webContents, url, referrer, extras);
+
+ RecordHistogram.recordBooleanHistogram("CustomTabs.PrerenderSessionUsesDefaultParameters",
+ mClientManager.usesDefaultSessionParameters(session));
+
return true;
}

Powered by Google App Engine
This is Rietveld 408576698