[uber page] Split up initialization of handlers from initialization of webui pages

chrome/browser/ui/webui/options2/chromeos/bluetooth_options_handler2.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/options2/chromeos/bluetooth_options_handler2.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
@@ skipping 14 matching lines @@
 // value stored in each list element is indicated by the following constants.
 const int kUpdateDeviceAddressIndex = 0;
 const int kUpdateDeviceCommandIndex = 1;
 const int kUpdateDevicePasskeyIndex = 2;
 
 }  // namespace
 
 namespace chromeos {
 namespace options2 {
 
-BluetoothOptionsHandler::BluetoothOptionsHandler() {
+BluetoothOptionsHandler::BluetoothOptionsHandler()
+    : bluetooth_enabled_(CommandLine::ForCurrentProcess()
+                         ->HasSwitch(switches::kEnableBluetooth)) {
 }
 
 BluetoothOptionsHandler::~BluetoothOptionsHandler() {
-  if (!CommandLine::ForCurrentProcess()
-      ->HasSwitch(switches::kEnableBluetooth)) {
-    return;
-  }
-
-  adapter_->RemoveObserver(this);
+  if (bluetooth_enabled_ && adapter_.get())
+    adapter_->RemoveObserver(this);
 }
 
 void BluetoothOptionsHandler::GetLocalizedValues(
     DictionaryValue* localized_strings) {
   DCHECK(localized_strings);
   localized_strings->SetString("bluetooth",
       l10n_util::GetStringUTF16(IDS_OPTIONS_SETTINGS_SECTION_TITLE_BLUETOOTH));
   localized_strings->SetString("disableBluetooth",
       l10n_util::GetStringUTF16(IDS_OPTIONS_SETTINGS_BLUETOOTH_DISABLE));
   localized_strings->SetString("enableBluetooth",
@@ skipping 61 matching lines @@
       l10n_util::GetStringUTF16(
       IDS_OPTIONS_SETTINGS_BLUETOOTH_CONNECTION_FAILED_INCORRECT_PIN));
   localized_strings->SetString("bluetoothErrorTimeout",
       l10n_util::GetStringUTF16(
       IDS_OPTIONS_SETTINGS_BLUETOOTH_CONNECTION_FAILED_TIMEOUT));
   localized_strings->SetString("bluetoothErrorConnectionFailed",
       l10n_util::GetStringUTF16(
       IDS_OPTIONS_SETTINGS_BLUETOOTH_CONNECTION_FAILED));
 }
 
-void BluetoothOptionsHandler::Initialize() {
+void BluetoothOptionsHandler::InitializeHandler() {
   // Bluetooth support is a work in progress.  Supress the feature unless
   // explicitly enabled via a command line flag.
-  if (!CommandLine::ForCurrentProcess()
-      ->HasSwitch(switches::kEnableBluetooth)) {
-    return;
+  if (bluetooth_enabled_) {
+    adapter_.reset(BluetoothAdapter::CreateDefaultAdapter());
+    adapter_->AddObserver(this);
   }
+}
 
-  adapter_.reset(BluetoothAdapter::CreateDefaultAdapter());
-  adapter_->AddObserver(this);
-
-  // Show or hide the bluetooth settings and update the checkbox based
-  // on the current present/powered state.
-  AdapterPresentChanged(adapter_.get(), adapter_->IsPresent());
+void BluetoothOptionsHandler::InitializePage() {
+  // Show or hide the bluetooth settings and update the checkbox based on the
+  // current present/powered state.
+  if (bluetooth_enabled_)
+    AdapterPresentChanged(adapter_.get(), adapter_->IsPresent());
 }
 
 void BluetoothOptionsHandler::AdapterPresentChanged(BluetoothAdapter* adapter,
                                                     bool present) {
   DCHECK(adapter == adapter_.get());
   if (present) {
     web_ui()->CallJavascriptFunction(
         "options.BrowserOptions.showBluetoothSettings");
 
     // Update the checkbox and visibility based on the powered state of the
@@ skipping 196 matching lines @@
 void BluetoothOptionsHandler::ErrorCallback() {
   // TODO(keybuk): we don't get any form of error response from dbus::
   // yet, other than an error occurred. I'm going to fix that, then this
   // gets replaced by genuine error information from the method which we
   // can act on, rather than a debug log statement.
   DVLOG(1) << "Failed.";
 }
 
 }  // namespace options2
 }  // namespace chromeos