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

Unified Diff: components/google/core/browser/google_search_metrics.h

Issue 342053002: Add UMA metrics for Android Chrome Google Search. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: '' Created 6 years, 6 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: components/google/core/browser/google_search_metrics.h
diff --git a/components/google/core/browser/google_search_metrics.h b/components/google/core/browser/google_search_metrics.h
index 678bcd4115ec31f1bbc3ab71f120bed1a541c156..d6d1211b13d6f35e0227f8baa407cc3233de9628 100644
--- a/components/google/core/browser/google_search_metrics.h
+++ b/components/google/core/browser/google_search_metrics.h
@@ -5,6 +5,8 @@
#ifndef COMPONENTS_GOOGLE_CORE_BROWSER_GOOGLE_SEARCH_METRICS_H_
#define COMPONENTS_GOOGLE_CORE_BROWSER_GOOGLE_SEARCH_METRICS_H_
+#include "build/build_config.h"
+
// A thin helper class used by parties interested in reporting Google search
// metrics (mostly counts of searches from different access points). This class
// partly exists to make testing easier.
@@ -31,6 +33,13 @@ class GoogleSearchMetrics {
// Record a single Google search from source |ap|.
virtual void RecordGoogleSearch(AccessPoint ap) const;
+
+#if defined(OS_ANDROID)
+ // Record a single Android Google search from source |ap|. |prerender_enabled|
+ // is set to true when prerendering is enabled via settings.
+ virtual void RecordAndroidGoogleSearch(AccessPoint ap,
Peter Kasting 2014/06/19 21:20:33 Why is this virtual?
kmadhusu 2014/06/19 22:52:49 For testing. I am working on the tests.
+ bool prerender_enabled) const;
+#endif
};
#endif // COMPONENTS_GOOGLE_CORE_BROWSER_GOOGLE_SEARCH_METRICS_H_

Powered by Google App Engine
This is Rietveld 408576698