[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 105 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;
   }
 
   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());

[csilv, 2012/03/14 00:09:11]

seems like this should be moved to InitializePage().  InitializePage() will also
need the command line check

[Dan Beam, 2012/03/14 01:13:14]

Done.

 }
 
 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