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

Unified Diff: chrome/browser/ui/webui/options2/font_settings_handler2.cc

Issue 9071026: Copy changes in r115955 from options to options2 (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 12 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/options2/font_settings_handler2.cc
diff --git a/chrome/browser/ui/webui/options2/font_settings_handler2.cc b/chrome/browser/ui/webui/options2/font_settings_handler2.cc
index 7875f7ad25b47a768ea4e956329bfce39063a006..56b5fc7c38b0158eb9d57802618fce9c29642c18 100644
--- a/chrome/browser/ui/webui/options2/font_settings_handler2.cc
+++ b/chrome/browser/ui/webui/options2/font_settings_handler2.cc
@@ -81,15 +81,19 @@ void FontSettingsHandler::RegisterMessages() {
FontSettingsUtilities::ValidateSavedFonts(pref_service);
// Register for preferences that we need to observe manually.
- standard_font_.Init(prefs::kWebKitStandardFontFamily, pref_service, this);
- serif_font_.Init(prefs::kWebKitSerifFontFamily, pref_service, this);
- sans_serif_font_.Init(prefs::kWebKitSansSerifFontFamily, pref_service, this);
- fixed_font_.Init(prefs::kWebKitFixedFontFamily, pref_service, this);
- font_encoding_.Init(prefs::kDefaultCharset, pref_service, this);
- default_font_size_.Init(prefs::kWebKitDefaultFontSize, pref_service, this);
- default_fixed_font_size_.Init(prefs::kWebKitDefaultFixedFontSize,
+ standard_font_.Init(prefs::kWebKitGlobalStandardFontFamily,
+ pref_service, this);
+ serif_font_.Init(prefs::kWebKitGlobalSerifFontFamily, pref_service, this);
+ sans_serif_font_.Init(prefs::kWebKitGlobalSansSerifFontFamily,
+ pref_service, this);
+ fixed_font_.Init(prefs::kWebKitGlobalFixedFontFamily, pref_service, this);
+ font_encoding_.Init(prefs::kGlobalDefaultCharset, pref_service, this);
+ default_font_size_.Init(prefs::kWebKitGlobalDefaultFontSize,
+ pref_service, this);
+ default_fixed_font_size_.Init(prefs::kWebKitGlobalDefaultFixedFontSize,
pref_service, this);
- minimum_font_size_.Init(prefs::kWebKitMinimumFontSize, pref_service, this);
+ minimum_font_size_.Init(prefs::kWebKitGlobalMinimumFontSize,
+ pref_service, this);
web_ui()->RegisterMessageCallback("fetchFontsData",
base::Bind(&FontSettingsHandler::HandleFetchFontsData,
@@ -150,20 +154,20 @@ void FontSettingsHandler::Observe(int type,
const content::NotificationDetails& details) {
if (type == chrome::NOTIFICATION_PREF_CHANGED) {
std::string* pref_name = content::Details<std::string>(details).ptr();
- if (*pref_name == prefs::kWebKitStandardFontFamily) {
+ if (*pref_name == prefs::kWebKitGlobalStandardFontFamily) {
SetUpStandardFontSample();
- } else if (*pref_name == prefs::kWebKitSerifFontFamily) {
+ } else if (*pref_name == prefs::kWebKitGlobalSerifFontFamily) {
SetUpSerifFontSample();
- } else if (*pref_name == prefs::kWebKitSansSerifFontFamily) {
+ } else if (*pref_name == prefs::kWebKitGlobalSansSerifFontFamily) {
SetUpSansSerifFontSample();
- } else if (*pref_name == prefs::kWebKitFixedFontFamily ||
- *pref_name == prefs::kWebKitDefaultFixedFontSize) {
+ } else if (*pref_name == prefs::kWebKitGlobalFixedFontFamily ||
+ *pref_name == prefs::kWebKitGlobalDefaultFixedFontSize) {
SetUpFixedFontSample();
- } else if (*pref_name == prefs::kWebKitDefaultFontSize) {
+ } else if (*pref_name == prefs::kWebKitGlobalDefaultFontSize) {
SetUpStandardFontSample();
SetUpSerifFontSample();
SetUpSansSerifFontSample();
- } else if (*pref_name == prefs::kWebKitMinimumFontSize) {
+ } else if (*pref_name == prefs::kWebKitGlobalMinimumFontSize) {
SetUpMinimumFontSample();
}
}

Powered by Google App Engine
This is Rietveld 408576698