Chrome OS: Attach bluetooth UI to device discovery API

chrome/browser/ui/webui/options/chromeos/bluetooth_options_handler.cc

 // Copyright (c) 2011 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/options/chromeos/bluetooth_options_handler.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
+#include "chrome/browser/chromeos/bluetooth/bluetooth_device.h"
 #include "chrome/browser/chromeos/system/runtime_environment.h"
 #include "chrome/browser/ui/webui/options/chromeos/system_settings_provider.h"
 #include "chrome/common/chrome_switches.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace chromeos {
 
 BluetoothOptionsHandler::BluetoothOptionsHandler()
     : chromeos::CrosOptionsPageUIHandler(
         new chromeos::SystemSettingsProvider()) {
 }
 
 BluetoothOptionsHandler::~BluetoothOptionsHandler() {
-  // TODO(kevers): Shutdown bluetooth.
+  VLOG(1) << "BluetoothOptionsHandler dtor";

[kevers, 2011/10/27 16:14:43]

Do we intend to keep this comment in the final release?  Some discussion on
various forums concerned with the amount of bloat that these message contribute
to the final product.  Should we use DVLOG instead, or add a TODO to remove once
further into the development cycle?

[Vince Laviano, 2011/10/27 20:15:54]

Removed.

+
+  if (!CommandLine::ForCurrentProcess()
+      ->HasSwitch(switches::kEnableBluetooth)) {
+    return;
+  }
+
+  chromeos::BluetoothManager* bluetooth_manager =
+      chromeos::BluetoothManager::GetInstance();
+  DCHECK(bluetooth_manager);
+
+  chromeos::BluetoothAdapter* default_adapter =
+      bluetooth_manager->DefaultAdapter();
+
+  if (default_adapter != NULL) {
+    default_adapter->RemoveObserver(this);
+  }
+
+  bluetooth_manager->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("enableBluetooth",
       l10n_util::GetStringUTF16(IDS_OPTIONS_SETTINGS_BLUETOOTH_ENABLE));
@@ skipping 13 matching lines @@
 
 void BluetoothOptionsHandler::Initialize() {
   DCHECK(web_ui_);
   // Bluetooth support is a work in progress.  Supress the feature unless
   // explicitly enabled via a command line flag.
   // TODO(kevers): Test for presence of bluetooth hardware.
   if (!CommandLine::ForCurrentProcess()
       ->HasSwitch(switches::kEnableBluetooth)) {
     return;
   }
+  VLOG(1) << "Initialize";

[kevers, 2011/10/27 16:14:43]

Message appears to be too generic.

[Vince Laviano, 2011/10/27 20:15:54]

Removed.

+
   web_ui_->CallJavascriptFunction(
       "options.SystemOptions.showBluetoothSettings");
-  // TODO(kevers): Initialize bluetooth.
+
+  chromeos::BluetoothManager* bluetooth_manager =
+      chromeos::BluetoothManager::GetInstance();
+  DCHECK(bluetooth_manager);
+  bluetooth_manager->AddObserver(this);
+
+  chromeos::BluetoothAdapter* default_adapter =
+      bluetooth_manager->DefaultAdapter();
+  DefaultAdapterChanged(default_adapter);
 }
 
 void BluetoothOptionsHandler::RegisterMessages() {
   DCHECK(web_ui_);
   web_ui_->RegisterMessageCallback("bluetoothEnableChange",
       base::Bind(&BluetoothOptionsHandler::EnableChangeCallback,
                  base::Unretained(this)));
   web_ui_->RegisterMessageCallback("findBluetoothDevices",
       base::Bind(&BluetoothOptionsHandler::FindDevicesCallback,
                  base::Unretained(this)));
   web_ui_->RegisterMessageCallback("updateBluetoothDevice",
       base::Bind(&BluetoothOptionsHandler::UpdateDeviceCallback,
                  base::Unretained(this)));
 }
 
 void BluetoothOptionsHandler::EnableChangeCallback(
     const ListValue* args) {
   // TODO(kevers): Call Bluetooth API to enable or disable.
 }
 
 void BluetoothOptionsHandler::FindDevicesCallback(
     const ListValue* args) {
   // We only initiate a scan if we're running on Chrome OS. Otherwise, we
   // generate a fake device list.
   if (!chromeos::system::runtime_environment::IsRunningOnChromeOS()) {
     GenerateFakeDeviceList();
     return;
   }
-  // TODO(kevers): Fetch real Bluetooth devices.
+
+  chromeos::BluetoothManager* bluetooth_manager =
+      chromeos::BluetoothManager::GetInstance();
+  DCHECK(bluetooth_manager);
+
+  chromeos::BluetoothAdapter* default_adapter =
+      bluetooth_manager->DefaultAdapter();
+
+  if ((default_adapter == NULL && !default_adapter_id_.empty()) ||
+      (default_adapter != NULL &&
+       default_adapter_id_ != default_adapter->Id())) {
+    LOG(WARNING) << "unexpected default adapter change from \""

[kevers, 2011/10/27 16:14:43]

Seems to be a developing trend to migrate away from LOG(WARNING), replacing with
VLOG.  Consider mapping VLOG(1) -> VLOG(2) and LOG(WARNING) -> VLOG(1).

[Vince Laviano, 2011/10/27 20:15:54]

Done.

+                 << default_adapter_id_ << "\" to \"" << default_adapter->Id()
+                 << "\"";
+    DefaultAdapterChanged(default_adapter);
+  }
+
+  if (default_adapter == NULL) {
+    LOG(WARNING) << "FindDevicesCallback: no default adapter";
+    return;
+  }
+
+  default_adapter->StartDiscovery();
 }
 
 void BluetoothOptionsHandler::UpdateDeviceCallback(
     const ListValue* args) {
   // TODO(kevers): Trigger connect/disconnect.
 }
 
 void BluetoothOptionsHandler::DeviceNotification(
     const DictionaryValue& device) {
   web_ui_->CallJavascriptFunction(
       "options.SystemOptions.addBluetoothDevice", device);
 }
 
+void BluetoothOptionsHandler::DefaultAdapterChanged(
+    chromeos::BluetoothAdapter* adapter) {
+  std::string old_default_adapter_id = default_adapter_id_;
+
+  if (adapter == NULL) {
+    default_adapter_id_.clear();
+    LOG(INFO) << "DefaultAdapterChanged: no default bluetooth adapter";

[kevers, 2011/10/27 16:14:43]

VLOG(2)?

[Vince Laviano, 2011/10/27 20:15:54]

Done.

+  } else {
+    default_adapter_id_ = adapter->Id();
+    LOG(INFO) << "DefaultAdapterChanged: " << default_adapter_id_;
+  }
+
+  if (default_adapter_id_ == old_default_adapter_id) {
+    return;
+  }
+
+  if (adapter != NULL) {
+    adapter->AddObserver(this);
+  }
+
+  // TODO(vlaviano): Respond to adapter change.
+}
+
+void BluetoothOptionsHandler::DiscoveryStarted(const std::string& adapter_id) {
+  VLOG(1) << "Discovery started on " << adapter_id;
+}
+
+void BluetoothOptionsHandler::DiscoveryEnded(const std::string& adapter_id) {
+  VLOG(1) << "Discovery ended on " << adapter_id;
+  web_ui_->CallJavascriptFunction(
+      "options.SystemOptions.notifyBluetoothSearchComplete");
+
+  // Stop the discovery session.
+  // TODO(vlaviano): We may want to expose DeviceDisappeared, remove the
+  // "Find devices" button, and let the discovery session continue throughout
+  // the time that the page is visible rather than just doing a single discovery
+  // cycle in response to a button click.
+  chromeos::BluetoothManager* bluetooth_manager =
+      chromeos::BluetoothManager::GetInstance();
+  DCHECK(bluetooth_manager);
+
+  chromeos::BluetoothAdapter* default_adapter =
+      bluetooth_manager->DefaultAdapter();
+
+  if ((default_adapter == NULL && !default_adapter_id_.empty()) ||

[kevers, 2011/10/27 16:14:43]

Lines 200-207 appear to duplicate lines 128-135.  Please create a method for
adapter validation.

[Vince Laviano, 2011/10/27 20:15:54]

Done.

+      (default_adapter != NULL &&
+       default_adapter_id_ != default_adapter->Id())) {
+    LOG(WARNING) << "unexpected default adapter change from \""
+                 << default_adapter_id_ << "\" to \"" << default_adapter->Id()
+                 << "\"";
+    DefaultAdapterChanged(default_adapter);
+  }
+
+  if (default_adapter == NULL) {
+    LOG(WARNING) << "DiscoveryEnded: no default adapter";
+    return;
+  }
+
+  default_adapter->StopDiscovery();
+}
+
+void BluetoothOptionsHandler::DeviceFound(const std::string& adapter_id,
+                                          chromeos::BluetoothDevice* device) {
+  VLOG(1) << "Device found on " << adapter_id;
+  DCHECK(device);
+
+  // TODO(vlaviano): eliminate inconsistencies between the javascript rep and
+  // BluetoothDevice so that we can use BluetoothDevice::AsDictionary() here.
+  DictionaryValue device_js_rep;
+  device_js_rep.SetString("deviceName", device->GetName());
+  device_js_rep.SetString("deviceId", device->GetAddress());
+  device_js_rep.SetString("deviceType", device->GetIcon());
+  if (device->IsPaired()) {
+    device_js_rep.SetString("deviceStatus", "bluetoothDeviceConnected");
+  } else {
+    device_js_rep.SetString("deviceStatus", "bluetoothDeviceNotPaired");
+  }
+
+  web_ui_->CallJavascriptFunction(
+      "options.SystemOptions.addBluetoothDevice", device_js_rep);
+}
+
 void BluetoothOptionsHandler::GenerateFakeDeviceList() {
   // TODO(kevers): Send notifications asynchronously simulating that the
   //               process of discovering bluetooth devices takes time.
   //               Fire each notification using OneShotTimer with a
   //               varying delay.
   std::string data[9] = {
     "Fake Wireless Keyboard", "01-02-03-04-05", "keyboard",
     "Fake Wireless Mouse",  "02-03-04-05-01", "mouse",
     "Fake Wireless Headset", "03-04-05-01-02", "headset"};
 
   for (int i = 0; i < 3; i++) {
     DictionaryValue device;
     device.SetString("deviceName", data[3*i]);
     device.SetString("deviceId", data[3*i+1]);
     device.SetString("deviceType", data[3*i+2]);
     device.SetString("deviceStatus", "bluetoothDeviceNotPaired");
     web_ui_->CallJavascriptFunction(
         "options.SystemOptions.addBluetoothDevice", device);
   }
   web_ui_->CallJavascriptFunction(
       "options.SystemOptions.notifyBluetoothSearchComplete");
 }
 
-}
-
+}  // namespace chromeos