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

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

Issue 9317114: Settings: Clean up browser_options page according to style guide. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review fixes. Created 8 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/options2/browser_options_handler2.cc
diff --git a/chrome/browser/ui/webui/options2/browser_options_handler2.cc b/chrome/browser/ui/webui/options2/browser_options_handler2.cc
index 35ad33d5e0e0a0c50b21b49f4ad2c381cf5f2e6d..bbda7e1a0011cb61ec03cbbc8040e9530012b272 100644
--- a/chrome/browser/ui/webui/options2/browser_options_handler2.cc
+++ b/chrome/browser/ui/webui/options2/browser_options_handler2.cc
@@ -111,7 +111,7 @@ void BrowserOptionsHandler::GetLocalizedValues(
DCHECK(localized_strings);
static OptionsStringResource resources[] = {
- { "advancedOptionsButtonTitle", IDS_OPTIONS_ADVANCED_BUTTON_TITLE },
+ { "advancedSettings", IDS_OPTIONS_ADVANCED_BUTTON_TITLE },
{ "autologinEnabled", IDS_OPTIONS_PASSWORDS_AUTOLOGIN },
{ "browsingData", IDS_OPTIONS_BROWSING_DATA_GROUP_NAME }, // needed?
{ "changeHomePage", IDS_OPTIONS_CHANGE_HOME_PAGE },
@@ -139,6 +139,7 @@ void BrowserOptionsHandler::GetLocalizedValues(
{ "sessionRestoreOverlayMessage", IDS_SESSION_RESTORE_OVERLAY_MESSAGE },
{ "sessionRestoreOverlayTitle", IDS_SESSION_RESTORE_OVERLAY_TITLE },
{ "startupRestoreLastSession", IDS_OPTIONS_STARTUP_RESTORE_LAST_SESSION },
+ { "settingsTitle", IDS_SETTINGS_TITLE },
{ "startupSetPages", IDS_OPTIONS2_STARTUP_SET_PAGES },
{ "startupShowDefaultAndNewTab",
IDS_OPTIONS_STARTUP_SHOW_DEFAULT_AND_NEWTAB},
@@ -181,7 +182,6 @@ void BrowserOptionsHandler::GetLocalizedValues(
};
RegisterStrings(localized_strings, resources, arraysize(resources));
- RegisterTitle(localized_strings, "browserPage", IDS_SETTINGS_TITLE);
localized_strings->SetString(
"syncOverview",

Powered by Google App Engine
This is Rietveld 408576698