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

Unified Diff: chrome/browser/ui/webui/options/core_options_handler.cc

Issue 7314020: Update UMA user actions parsing, primarily to include WebUI metrics. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update regex Created 9 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/browser/ui/webui/options/core_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/core_options_handler.cc b/chrome/browser/ui/webui/options/core_options_handler.cc
index 1a3d81c50f966ec53fcb32e8bf8bac1b86b8e805..13fbed9bce1f3936b2873e69e66391776a14d43e 100644
--- a/chrome/browser/ui/webui/options/core_options_handler.cc
+++ b/chrome/browser/ui/webui/options/core_options_handler.cc
@@ -195,7 +195,7 @@ void CoreOptionsHandler::ClearPref(const std::string& pref_name,
pref_service->ScheduleSavePersistentPrefs();
if (!metric.empty())
- UserMetricsRecordAction(UserMetricsAction(metric.c_str()));
+ UserMetrics::RecordComputedAction(metric);
}
void CoreOptionsHandler::ProcessUserMetric(const Value* value,
@@ -210,7 +210,7 @@ void CoreOptionsHandler::ProcessUserMetric(const Value* value,
metric_string += bool_value ? "_Enable" : "_Disable";
}
- UserMetricsRecordAction(UserMetricsAction(metric_string.c_str()));
+ UserMetrics::RecordComputedAction(metric_string);
}
void CoreOptionsHandler::StopObservingPref(const std::string& path) {
@@ -360,7 +360,7 @@ void CoreOptionsHandler::HandleClearPref(const ListValue* args) {
void CoreOptionsHandler::HandleUserMetricsAction(const ListValue* args) {
std::string metric = UTF16ToUTF8(ExtractStringValue(args));
if (!metric.empty())
- UserMetricsRecordAction(UserMetricsAction(metric.c_str()));
+ UserMetrics::RecordComputedAction(metric);
}
void CoreOptionsHandler::UpdateClearPluginLSOData() {
« no previous file with comments | « chrome/browser/ui/webui/options/chromeos/stats_options_handler.cc ('k') | chrome/browser/ui/webui/options/options_ui.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698