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

Unified Diff: chrome/browser/android/metrics/launch_metrics.cc

Issue 2006023002: Log some information about the state of the homepage on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update comments Created 4 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: chrome/browser/android/metrics/launch_metrics.cc
diff --git a/chrome/browser/android/metrics/launch_metrics.cc b/chrome/browser/android/metrics/launch_metrics.cc
index 77003c611977a592b7e507fa3f7aa8deaa6f7adf..6c634ba200b7716e321f41acfec9c8572f288df6 100644
--- a/chrome/browser/android/metrics/launch_metrics.cc
+++ b/chrome/browser/android/metrics/launch_metrics.cc
@@ -12,6 +12,7 @@
#include "chrome/browser/banners/app_banner_settings_helper.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/engagement/site_engagement_service.h"
+#include "chrome/browser/prefs/pref_metrics_service.h"
#include "chrome/browser/profiles/profile.h"
#include "components/rappor/rappor_utils.h"
#include "content/public/browser/web_contents.h"
@@ -87,4 +88,17 @@ static void RecordLaunch(JNIEnv* env,
rappor_metric_action, url);
}
+static void RecordHomePageLaunchMetrics(
+ JNIEnv* env,
+ const JavaParamRef<jclass>& caller,
+ jboolean show_home_button,
+ jboolean homepage_is_ntp,
+ const JavaParamRef<jstring>& jhomepage_url) {
+ GURL homepage_url(base::android::ConvertJavaStringToUTF8(env, jhomepage_url));
+ PrefMetricsService::RecordHomePageLaunchMetrics(
+ show_home_button,
+ homepage_is_ntp,
+ homepage_url);
+}
+
}; // namespace metrics

Powered by Google App Engine
This is Rietveld 408576698