diff options
author | joi@chromium.org <joi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-14 13:23:38 +0000 |
---|---|---|
committer | joi@chromium.org <joi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-14 13:23:38 +0000 |
commit | 7f6f44c9e44707612415d4c85cfe9924aeacd917 (patch) | |
tree | 167b6d29691a127ef3ab1ada5b6f9c774d99363d /chrome/browser/themes | |
parent | 7af0889d30c9a27b916aa49d4bfea7ea92465072 (diff) | |
download | chromium_src-7f6f44c9e44707612415d4c85cfe9924aeacd917.zip chromium_src-7f6f44c9e44707612415d4c85cfe9924aeacd917.tar.gz chromium_src-7f6f44c9e44707612415d4c85cfe9924aeacd917.tar.bz2 |
Split UserMetrics into API vs. implementation. Move API to content/public.
TBR=davemoore@chromium.org
BUG=98716
Review URL: http://codereview.chromium.org/8919017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114416 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/themes')
-rw-r--r-- | chrome/browser/themes/theme_service.cc | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/chrome/browser/themes/theme_service.cc b/chrome/browser/themes/theme_service.cc index c4a62916..e5eb407 100644 --- a/chrome/browser/themes/theme_service.cc +++ b/chrome/browser/themes/theme_service.cc @@ -14,8 +14,8 @@ #include "chrome/common/chrome_constants.h" #include "chrome/common/chrome_notification_types.h" #include "chrome/common/pref_names.h" -#include "content/browser/user_metrics.h" #include "content/public/browser/notification_service.h" +#include "content/public/browser/user_metrics.h" #include "grit/theme_resources.h" #include "grit/theme_resources_standard.h" #include "grit/ui_resources.h" @@ -26,6 +26,7 @@ #endif using content::BrowserThread; +using content::UserMetricsAction; // Strings used in alignment properties. const char* ThemeService::kAlignmentTop = "top"; @@ -327,7 +328,7 @@ void ThemeService::SetTheme(const Extension* extension) { SaveThemeID(extension->id()); NotifyThemeChanged(); - UserMetrics::RecordAction(UserMetricsAction("Themes_Installed")); + content::RecordAction(UserMetricsAction("Themes_Installed")); } void ThemeService::RemoveUnusedThemes() { @@ -352,7 +353,7 @@ void ThemeService::RemoveUnusedThemes() { void ThemeService::UseDefaultTheme() { ClearAllThemeData(); NotifyThemeChanged(); - UserMetrics::RecordAction(UserMetricsAction("Themes_Reset")); + content::RecordAction(UserMetricsAction("Themes_Reset")); } void ThemeService::SetNativeTheme() { @@ -585,7 +586,7 @@ void ThemeService::LoadThemePrefs() { } if (loaded_pack) { - UserMetrics::RecordAction(UserMetricsAction("Themes.Loaded")); + content::RecordAction(UserMetricsAction("Themes.Loaded")); } else { // TODO(erg): We need to pop up a dialog informing the user that their // theme is being migrated. @@ -596,11 +597,11 @@ void ThemeService::LoadThemePrefs() { if (extension) { DLOG(ERROR) << "Migrating theme"; BuildFromExtension(extension); - UserMetrics::RecordAction(UserMetricsAction("Themes.Migrated")); + content::RecordAction(UserMetricsAction("Themes.Migrated")); } else { DLOG(ERROR) << "Theme is mysteriously gone."; ClearAllThemeData(); - UserMetrics::RecordAction(UserMetricsAction("Themes.Gone")); + content::RecordAction(UserMetricsAction("Themes.Gone")); } } } |