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

Side by Side Diff: chrome/browser/ui/webui/options/language_options_handler_common.cc

Issue 8919017: Split UserMetrics into API vs. implementation. Move API to content/public. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move to content namespace. Update usages. Update extract_actions tool. Created 9 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/webui/options/language_options_handler_common.h" 5 #include "chrome/browser/ui/webui/options/language_options_handler_common.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
11 11
12 #include "base/basictypes.h" 12 #include "base/basictypes.h"
13 #include "base/bind.h" 13 #include "base/bind.h"
14 #include "base/command_line.h" 14 #include "base/command_line.h"
15 #include "base/stringprintf.h" 15 #include "base/stringprintf.h"
16 #include "base/utf_string_conversions.h" 16 #include "base/utf_string_conversions.h"
17 #include "base/values.h" 17 #include "base/values.h"
18 #include "chrome/browser/browser_process.h" 18 #include "chrome/browser/browser_process.h"
19 #include "chrome/browser/prefs/pref_service.h" 19 #include "chrome/browser/prefs/pref_service.h"
20 #include "chrome/browser/ui/browser_list.h" 20 #include "chrome/browser/ui/browser_list.h"
21 #include "chrome/common/chrome_switches.h" 21 #include "chrome/common/chrome_switches.h"
22 #include "chrome/common/pref_names.h" 22 #include "chrome/common/pref_names.h"
23 #include "chrome/common/spellcheck_common.h" 23 #include "chrome/common/spellcheck_common.h"
24 #include "content/browser/user_metrics.h" 24 #include "content/public/browser/user_metrics.h"
25 #include "grit/chromium_strings.h" 25 #include "grit/chromium_strings.h"
26 #include "grit/generated_resources.h" 26 #include "grit/generated_resources.h"
27 #include "ui/base/l10n/l10n_util.h" 27 #include "ui/base/l10n/l10n_util.h"
28 28
29 using content::UserMetricsAction;
30
29 LanguageOptionsHandlerCommon::LanguageOptionsHandlerCommon() { 31 LanguageOptionsHandlerCommon::LanguageOptionsHandlerCommon() {
30 } 32 }
31 33
32 LanguageOptionsHandlerCommon::~LanguageOptionsHandlerCommon() { 34 LanguageOptionsHandlerCommon::~LanguageOptionsHandlerCommon() {
33 } 35 }
34 36
35 void LanguageOptionsHandlerCommon::GetLocalizedValues( 37 void LanguageOptionsHandlerCommon::GetLocalizedValues(
36 DictionaryValue* localized_strings) { 38 DictionaryValue* localized_strings) {
37 DCHECK(localized_strings); 39 DCHECK(localized_strings);
38 string16 product_name = GetProductName(); 40 string16 product_name = GetProductName();
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 std::vector<std::string> spell_check_languages; 134 std::vector<std::string> spell_check_languages;
133 SpellCheckCommon::SpellCheckLanguages(&spell_check_languages); 135 SpellCheckCommon::SpellCheckLanguages(&spell_check_languages);
134 for (size_t i = 0; i < spell_check_languages.size(); ++i) { 136 for (size_t i = 0; i < spell_check_languages.size(); ++i) {
135 dictionary->SetBoolean(spell_check_languages[i], true); 137 dictionary->SetBoolean(spell_check_languages[i], true);
136 } 138 }
137 return dictionary; 139 return dictionary;
138 } 140 }
139 141
140 void LanguageOptionsHandlerCommon::LanguageOptionsOpenCallback( 142 void LanguageOptionsHandlerCommon::LanguageOptionsOpenCallback(
141 const ListValue* args) { 143 const ListValue* args) {
142 UserMetrics::RecordAction(UserMetricsAction("LanguageOptions_Open")); 144 content::RecordAction(UserMetricsAction("LanguageOptions_Open"));
143 } 145 }
144 146
145 void LanguageOptionsHandlerCommon::UiLanguageChangeCallback( 147 void LanguageOptionsHandlerCommon::UiLanguageChangeCallback(
146 const ListValue* args) { 148 const ListValue* args) {
147 const std::string language_code = UTF16ToASCII(ExtractStringValue(args)); 149 const std::string language_code = UTF16ToASCII(ExtractStringValue(args));
148 CHECK(!language_code.empty()); 150 CHECK(!language_code.empty());
149 const std::string action = base::StringPrintf( 151 const std::string action = base::StringPrintf(
150 "LanguageOptions_UiLanguageChange_%s", language_code.c_str()); 152 "LanguageOptions_UiLanguageChange_%s", language_code.c_str());
151 UserMetrics::RecordComputedAction(action); 153 content::RecordComputedAction(action);
152 SetApplicationLocale(language_code); 154 SetApplicationLocale(language_code);
153 web_ui_->CallJavascriptFunction("options.LanguageOptions.uiLanguageSaved"); 155 web_ui_->CallJavascriptFunction("options.LanguageOptions.uiLanguageSaved");
154 } 156 }
155 157
156 void LanguageOptionsHandlerCommon::SpellCheckLanguageChangeCallback( 158 void LanguageOptionsHandlerCommon::SpellCheckLanguageChangeCallback(
157 const ListValue* args) { 159 const ListValue* args) {
158 const std::string language_code = UTF16ToASCII(ExtractStringValue(args)); 160 const std::string language_code = UTF16ToASCII(ExtractStringValue(args));
159 CHECK(!language_code.empty()); 161 CHECK(!language_code.empty());
160 const std::string action = base::StringPrintf( 162 const std::string action = base::StringPrintf(
161 "LanguageOptions_SpellCheckLanguageChange_%s", language_code.c_str()); 163 "LanguageOptions_SpellCheckLanguageChange_%s", language_code.c_str());
162 UserMetrics::RecordComputedAction(action); 164 content::RecordComputedAction(action);
163 } 165 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698