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

Unified Diff: chrome/browser/ui/webui/options2/chromeos/core_chromeos_options_handler2.cc

Issue 9693032: [uber page] Split up initialization of handlers from initialization of webui pages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: copyright bumps 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/chromeos/core_chromeos_options_handler2.cc
diff --git a/chrome/browser/ui/webui/options2/chromeos/core_chromeos_options_handler2.cc b/chrome/browser/ui/webui/options2/chromeos/core_chromeos_options_handler2.cc
index e45969cc5aa325ab9fc8a4609acfd4c706b8ecc9..3ce04aa56b153eb3d038864ca6f47a2d5ec12d9f 100644
--- a/chrome/browser/ui/webui/options2/chromeos/core_chromeos_options_handler2.cc
+++ b/chrome/browser/ui/webui/options2/chromeos/core_chromeos_options_handler2.cc
@@ -111,7 +111,7 @@ CoreChromeOSOptionsHandler::~CoreChromeOSOptionsHandler() {
pointer_factory_.GetWeakPtr()));
}
-void CoreChromeOSOptionsHandler::Initialize() {
+void CoreChromeOSOptionsHandler::InitializeHandler() {
proxy_prefs_.reset(PrefSetObserver::CreateProxyPrefSetObserver(
Profile::FromWebUI(web_ui())->GetPrefs(), this));
// Observe the chromeos::ProxyConfigServiceImpl for changes from the UI.
@@ -122,6 +122,11 @@ void CoreChromeOSOptionsHandler::Initialize() {
pointer_factory_.GetWeakPtr()));
}
+void CoreChromeOSOptionsHandler::InitializePage() {
+ // NOTE: Don't remove this, we're intentionally ignoring the base class'
+ // implementation of InitializePage.
+}
+
base::Value* CoreChromeOSOptionsHandler::FetchPref(
const std::string& pref_name) {
if (proxy_cros_settings_parser::IsProxyPref(pref_name)) {

Powered by Google App Engine
This is Rietveld 408576698