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

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

Issue 9814030: get rid of old options pages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more fixes Created 8 years, 9 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/options_ui2.cc
diff --git a/chrome/browser/ui/webui/options2/options_ui2.cc b/chrome/browser/ui/webui/options2/options_ui2.cc
index 1d32812da3013ff8799cd6df08973d2dc589dbb5..83bc0c59536a6f2ede59d8d34c7de57df52f8ec3 100644
--- a/chrome/browser/ui/webui/options2/options_ui2.cc
+++ b/chrome/browser/ui/webui/options2/options_ui2.cc
@@ -21,8 +21,6 @@
#include "chrome/browser/browser_about_handler.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/ui/webui/options/options_sync_setup_handler.h"
-#include "chrome/browser/ui/webui/options/stop_syncing_handler.h"
#include "chrome/browser/ui/webui/options2/autofill_options_handler2.h"
#include "chrome/browser/ui/webui/options2/browser_options_handler2.h"
#include "chrome/browser/ui/webui/options2/clear_browser_data_handler2.h"
@@ -35,9 +33,11 @@
#include "chrome/browser/ui/webui/options2/import_data_handler2.h"
#include "chrome/browser/ui/webui/options2/language_options_handler2.h"
#include "chrome/browser/ui/webui/options2/manage_profile_handler2.h"
+#include "chrome/browser/ui/webui/options2/options_sync_setup_handler.h"
#include "chrome/browser/ui/webui/options2/password_manager_handler2.h"
#include "chrome/browser/ui/webui/options2/search_engine_manager_handler2.h"
#include "chrome/browser/ui/webui/options2/startup_pages_handler2.h"
+#include "chrome/browser/ui/webui/options2/stop_syncing_handler.h"
#include "chrome/browser/ui/webui/options2/web_intents_settings_handler2.h"
#include "chrome/browser/ui/webui/theme_source.h"
#include "chrome/common/jstemplate_builder.h"
@@ -161,6 +161,44 @@ std::string OptionsUIHTMLSource::GetMimeType(const std::string& path) const {
////////////////////////////////////////////////////////////////////////////////
//
+// OptionsPageUIHandler
+//
+////////////////////////////////////////////////////////////////////////////////
+
+OptionsPageUIHandler::OptionsPageUIHandler() {
+}
+
+OptionsPageUIHandler::~OptionsPageUIHandler() {
+}
+
+bool OptionsPageUIHandler::IsEnabled() {
+ return true;
+}
+
+// static
+void OptionsPageUIHandler::RegisterStrings(
+ DictionaryValue* localized_strings,
+ const OptionsStringResource* resources,
+ size_t length) {
Dan Beam 2012/03/23 03:25:31 er, why do we need |length|?
+ for (size_t i = 0; i < length; ++i) {
+ localized_strings->SetString(
+ resources[i].name, l10n_util::GetStringUTF16(resources[i].id));
+ }
+}
+
+void OptionsPageUIHandler::RegisterTitle(DictionaryValue* localized_strings,
+ const std::string& variable_name,
+ int title_id) {
+ localized_strings->SetString(variable_name,
+ l10n_util::GetStringUTF16(title_id));
+ localized_strings->SetString(variable_name + "TabTitle",
+ l10n_util::GetStringFUTF16(IDS_OPTIONS_TAB_TITLE,
+ l10n_util::GetStringUTF16(IDS_SETTINGS_TITLE),
+ l10n_util::GetStringUTF16(title_id)));
+}
+
+////////////////////////////////////////////////////////////////////////////////
+//
// OptionsUI
//
////////////////////////////////////////////////////////////////////////////////

Powered by Google App Engine
This is Rietveld 408576698