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

Unified Diff: chrome/browser/metrics/metrics_log.cc

Issue 10544114: Omnibox: In UMA logs, record typed_count of URLS. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Grammar in comment. Created 8 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: chrome/browser/metrics/metrics_log.cc
diff --git a/chrome/browser/metrics/metrics_log.cc b/chrome/browser/metrics/metrics_log.cc
index 42cd25e9ecde7cc471c63bd284112cca5b0156d7..4f4f614d74935635d4da651e58f300c94bb383bc 100644
--- a/chrome/browser/metrics/metrics_log.cc
+++ b/chrome/browser/metrics/metrics_log.cc
@@ -943,6 +943,8 @@ void MetricsLog::RecordOmniboxOpenedURL(const AutocompleteLog& log) {
suggestion->set_provider(i->provider->AsOmniboxEventProviderType());
suggestion->set_result_type(AsOmniboxEventResultType(i->type));
suggestion->set_relevance(i->relevance);
+ if (i->typed_count != -1)
+ suggestion->set_typed_count(i->typed_count);
suggestion->set_is_starred(i->starred);
}
for (ProvidersInfo::const_iterator i(log.providers_info.begin());
« no previous file with comments | « chrome/browser/autocomplete/history_url_provider.cc ('k') | chrome/common/metrics/proto/omnibox_event.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698