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

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

Issue 2666093002: Remove base::FundamentalValue (Closed)
Patch Set: Rebase Created 3 years, 10 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/font_settings_handler.cc
diff --git a/chrome/browser/ui/webui/options/font_settings_handler.cc b/chrome/browser/ui/webui/options/font_settings_handler.cc
index a9e028e4b671be11d5ae7921da2be282b49f1fdc..f57a800f98bb9e99ccca75de4479b69fb87095cc 100644
--- a/chrome/browser/ui/webui/options/font_settings_handler.cc
+++ b/chrome/browser/ui/webui/options/font_settings_handler.cc
@@ -208,7 +208,7 @@ void FontSettingsHandler::FontsListHasLoaded(
void FontSettingsHandler::SetUpStandardFontSample() {
base::StringValue font_value(
FontSettingsUtilities::ResolveFontList(standard_font_.GetValue()));
- base::FundamentalValue size_value(default_font_size_.GetValue());
+ base::Value size_value(default_font_size_.GetValue());
web_ui()->CallJavascriptFunctionUnsafe("FontSettings.setUpStandardFontSample",
font_value, size_value);
}
@@ -216,7 +216,7 @@ void FontSettingsHandler::SetUpStandardFontSample() {
void FontSettingsHandler::SetUpSerifFontSample() {
base::StringValue font_value(
FontSettingsUtilities::ResolveFontList(serif_font_.GetValue()));
- base::FundamentalValue size_value(default_font_size_.GetValue());
+ base::Value size_value(default_font_size_.GetValue());
web_ui()->CallJavascriptFunctionUnsafe("FontSettings.setUpSerifFontSample",
font_value, size_value);
}
@@ -224,7 +224,7 @@ void FontSettingsHandler::SetUpSerifFontSample() {
void FontSettingsHandler::SetUpSansSerifFontSample() {
base::StringValue font_value(
FontSettingsUtilities::ResolveFontList(sans_serif_font_.GetValue()));
- base::FundamentalValue size_value(default_font_size_.GetValue());
+ base::Value size_value(default_font_size_.GetValue());
web_ui()->CallJavascriptFunctionUnsafe(
"FontSettings.setUpSansSerifFontSample", font_value, size_value);
}
@@ -232,13 +232,13 @@ void FontSettingsHandler::SetUpSansSerifFontSample() {
void FontSettingsHandler::SetUpFixedFontSample() {
base::StringValue font_value(
FontSettingsUtilities::ResolveFontList(fixed_font_.GetValue()));
- base::FundamentalValue size_value(default_fixed_font_size_.GetValue());
+ base::Value size_value(default_fixed_font_size_.GetValue());
web_ui()->CallJavascriptFunctionUnsafe("FontSettings.setUpFixedFontSample",
font_value, size_value);
}
void FontSettingsHandler::SetUpMinimumFontSample() {
- base::FundamentalValue size_value(minimum_font_size_.GetValue());
+ base::Value size_value(minimum_font_size_.GetValue());
web_ui()->CallJavascriptFunctionUnsafe("FontSettings.setUpMinimumFontSample",
size_value);
}
@@ -256,7 +256,7 @@ FontSettingsHandler::GetAdvancedFontSettingsExtension() {
void FontSettingsHandler::NotifyAdvancedFontSettingsAvailability() {
web_ui()->CallJavascriptFunctionUnsafe(
"FontSettings.notifyAdvancedFontSettingsAvailability",
- base::FundamentalValue(GetAdvancedFontSettingsExtension() != NULL));
+ base::Value(GetAdvancedFontSettingsExtension() != NULL));
}
void FontSettingsHandler::HandleOpenAdvancedFontSettingsOptions(
« no previous file with comments | « chrome/browser/ui/webui/options/core_options_handler.cc ('k') | chrome/browser/ui/webui/options/import_data_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698