Refactoring bluetooth API code.

chrome/browser/chromeos/bluetooth/bluetooth_adapter_dbus.cc

Index: chrome/browser/chromeos/bluetooth/bluetooth_adapter_dbus.cc
diff --git a/chrome/browser/chromeos/bluetooth/bluetooth_adapter.cc b/chrome/browser/chromeos/bluetooth/bluetooth_adapter_dbus.cc
similarity index 72%
rename from chrome/browser/chromeos/bluetooth/bluetooth_adapter.cc
rename to chrome/browser/chromeos/bluetooth/bluetooth_adapter_dbus.cc
index 62b8ecb2bd0ce5965051650af27f2b92da7b94cc..e58e22e1154e5e14ec8fd4b0f15ba6d311d49af8 100644
--- a/chrome/browser/chromeos/bluetooth/bluetooth_adapter.cc
+++ b/chrome/browser/chromeos/bluetooth/bluetooth_adapter_dbus.cc
@@ -2,18 +2,21 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/chromeos/bluetooth/bluetooth_adapter.h"
+#include "chrome/browser/chromeos/bluetooth/bluetooth_adapter_dbus.h"
+
+#include <string>
 
 #include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/values.h"
-#include "chrome/browser/chromeos/bluetooth/bluetooth_device.h"
+#include "chrome/browser/chromeos/bluetooth/bluetooth_device_dbus.h"
 #include "chromeos/dbus/bluetooth_adapter_client.h"
 #include "chromeos/dbus/bluetooth_device_client.h"
 #include "chromeos/dbus/bluetooth_manager_client.h"
 #include "chromeos/dbus/bluetooth_out_of_band_client.h"
+#include "chromeos/dbus/bluetooth_out_of_band_pairing_data.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "dbus/object_path.h"
 
@@ -23,17 +26,17 @@ namespace {
 // if nobody is using it so use a WeakPtr and create the object when needed;
 // since Google C++ Style (and clang's static analyzer) forbids us having
 // exit-time destructors we use a leaky lazy instance for it.
-base::LazyInstance<base::WeakPtr<chromeos::BluetoothAdapter> >::Leaky
+base::LazyInstance<base::WeakPtr<chromeos::BluetoothAdapterDBus> >::Leaky
     default_adapter = LAZY_INSTANCE_INITIALIZER;
 
 }  // namespace
 
 namespace chromeos {
 
-BluetoothAdapter::BluetoothAdapter() : track_default_(false),
-                                       powered_(false),
-                                       discovering_(false),
-                                       weak_ptr_factory_(this) {
+BluetoothAdapterDBus::BluetoothAdapterDBus() : track_default_(false),
+                                               powered_(false),
+                                               discovering_(false),
+                                               weak_ptr_factory_(this) {
   DBusThreadManager::Get()->GetBluetoothManagerClient()->
       AddObserver(this);
   DBusThreadManager::Get()->GetBluetoothAdapterClient()->
@@ -42,7 +45,7 @@ BluetoothAdapter::BluetoothAdapter() : track_default_(false),
       AddObserver(this);
 }
 
-BluetoothAdapter::~BluetoothAdapter() {
+BluetoothAdapterDBus::~BluetoothAdapterDBus() {
   DBusThreadManager::Get()->GetBluetoothDeviceClient()->
       RemoveObserver(this);
   DBusThreadManager::Get()->GetBluetoothAdapterClient()->
@@ -53,63 +56,72 @@ BluetoothAdapter::~BluetoothAdapter() {
   STLDeleteValues(&devices_);
 }
 
-void BluetoothAdapter::AddObserver(Observer* observer) {
+void BluetoothAdapterDBus::AddObserver(BluetoothAdapter::Observer* observer) {
   DCHECK(observer);
   observers_.AddObserver(observer);
 }
 
-void BluetoothAdapter::RemoveObserver(Observer* observer) {
+void BluetoothAdapterDBus::RemoveObserver(
+    BluetoothAdapter::Observer* observer) {
   DCHECK(observer);
   observers_.RemoveObserver(observer);
 }
 
-bool BluetoothAdapter::IsPresent() const {
+const std::string& BluetoothAdapterDBus::address() const {
+  return address_;
+}
+
+const std::string& BluetoothAdapterDBus::name() const {
+  return name_;
+}
+
+bool BluetoothAdapterDBus::IsPresent() const {
   return !object_path_.value().empty() && !address_.empty();
 }
 
-bool BluetoothAdapter::IsPowered() const {
+bool BluetoothAdapterDBus::IsPowered() const {
   return powered_;
 }
 
-void BluetoothAdapter::SetPowered(bool powered,
-                                  const base::Closure& callback,
-                                  const ErrorCallback& error_callback) {
+void BluetoothAdapterDBus::SetPowered(bool powered,
+                                      const base::Closure& callback,
+                                      const ErrorCallback& error_callback) {
   DBusThreadManager::Get()->GetBluetoothAdapterClient()->
       GetProperties(object_path_)->powered.Set(
           powered,
-          base::Bind(&BluetoothAdapter::OnSetPowered,
+          base::Bind(&BluetoothAdapterDBus::OnSetPowered,
                      weak_ptr_factory_.GetWeakPtr(),
                      callback,
                      error_callback));
 }
 
-bool BluetoothAdapter::IsDiscovering() const {
+bool BluetoothAdapterDBus::IsDiscovering() const {
   return discovering_;
 }
 
-void BluetoothAdapter::SetDiscovering(bool discovering,
-                                      const base::Closure& callback,
-                                      const ErrorCallback& error_callback) {
+void BluetoothAdapterDBus::SetDiscovering(bool discovering,
+                                          const base::Closure& callback,
+                                          const ErrorCallback& error_callback) {
   if (discovering) {
     DBusThreadManager::Get()->GetBluetoothAdapterClient()->
         StartDiscovery(object_path_,
-                       base::Bind(&BluetoothAdapter::OnStartDiscovery,
+                       base::Bind(&BluetoothAdapterDBus::OnStartDiscovery,
                                   weak_ptr_factory_.GetWeakPtr(),
                                   callback,
                                   error_callback));
   } else {
     DBusThreadManager::Get()->GetBluetoothAdapterClient()->
         StopDiscovery(object_path_,
-                      base::Bind(&BluetoothAdapter::OnStopDiscovery,
+                      base::Bind(&BluetoothAdapterDBus::OnStopDiscovery,
                                  weak_ptr_factory_.GetWeakPtr(),
                                  callback,
                                  error_callback));
   }
 }
 
-BluetoothAdapter::DeviceList BluetoothAdapter::GetDevices() {
+BluetoothAdapter::DeviceList BluetoothAdapterDBus::GetDevices() {
   ConstDeviceList const_devices =
-      const_cast<const BluetoothAdapter *>(this)->GetDevices();
+      const_cast<const BluetoothAdapterDBus *>(this)->GetDevices();
 
   DeviceList devices;
   for (ConstDeviceList::const_iterator i = const_devices.begin();
@@ -119,59 +131,61 @@ BluetoothAdapter::DeviceList BluetoothAdapter::GetDevices() {
   return devices;
 }
 
-BluetoothAdapter::ConstDeviceList BluetoothAdapter::GetDevices() const {
+BluetoothAdapter::ConstDeviceList BluetoothAdapterDBus::GetDevices() const {
   ConstDeviceList devices;
-  for (DevicesMap::const_iterator iter = devices_.begin();
+  for (BluetoothAdapterDBus::DevicesMap::const_iterator iter = devices_.begin();
        iter != devices_.end(); ++iter)
     devices.push_back(iter->second);
 
   return devices;
 }
 
-BluetoothDevice* BluetoothAdapter::GetDevice(const std::string& address) {
+BluetoothDevice* BluetoothAdapterDBus::GetDevice(
+    const std::string& address) {
   return const_cast<BluetoothDevice *>(
-      const_cast<const BluetoothAdapter *>(this)->GetDevice(address));
+      const_cast<const BluetoothAdapterDBus *>(this)->GetDevice(address));
 }
 
-const BluetoothDevice* BluetoothAdapter::GetDevice(
+const BluetoothDevice* BluetoothAdapterDBus::GetDevice(
     const std::string& address) const {
-  DevicesMap::const_iterator iter = devices_.find(address);
+  BluetoothAdapterDBus::DevicesMap::const_iterator iter =
+      devices_.find(address);
   if (iter != devices_.end())
     return iter->second;
 
   return NULL;
 }
 
-void BluetoothAdapter::ReadLocalOutOfBandPairingData(
+void BluetoothAdapterDBus::ReadLocalOutOfBandPairingData(
     const BluetoothOutOfBandPairingDataCallback& callback,
     const ErrorCallback& error_callback) {
   DBusThreadManager::Get()->GetBluetoothOutOfBandClient()->
       ReadLocalData(object_path_,
-          base::Bind(&BluetoothAdapter::OnReadLocalData,
+          base::Bind(&BluetoothAdapterDBus::OnReadLocalData,
               weak_ptr_factory_.GetWeakPtr(),
               callback,
               error_callback));
 }
 
-void BluetoothAdapter::TrackDefaultAdapter() {
+void BluetoothAdapterDBus::TrackDefaultAdapter() {
   DVLOG(1) << "Tracking default adapter";
   track_default_ = true;
   DBusThreadManager::Get()->GetBluetoothManagerClient()->
-      DefaultAdapter(base::Bind(&BluetoothAdapter::AdapterCallback,
+      DefaultAdapter(base::Bind(&BluetoothAdapterDBus::AdapterCallback,
                                 weak_ptr_factory_.GetWeakPtr()));
 }
 
-void BluetoothAdapter::FindAdapter(const std::string& address) {
+void BluetoothAdapterDBus::FindAdapter(const std::string& address) {
   DVLOG(1) << "Using adapter " << address;
   track_default_ = false;
   DBusThreadManager::Get()->GetBluetoothManagerClient()->
       FindAdapter(address,
-                  base::Bind(&BluetoothAdapter::AdapterCallback,
+                  base::Bind(&BluetoothAdapterDBus::AdapterCallback,
                              weak_ptr_factory_.GetWeakPtr()));
 }
 
-void BluetoothAdapter::AdapterCallback(const dbus::ObjectPath& adapter_path,
-                                       bool success) {
+void BluetoothAdapterDBus::AdapterCallback(
+    const dbus::ObjectPath& adapter_path, bool success) {
   if (success) {
     ChangeAdapter(adapter_path);
   } else if (!object_path_.value().empty()) {
@@ -179,18 +193,20 @@ void BluetoothAdapter::AdapterCallback(const dbus::ObjectPath& adapter_path,
   }
 }
 
-void BluetoothAdapter::DefaultAdapterChanged(
+void BluetoothAdapterDBus::DefaultAdapterChanged(
     const dbus::ObjectPath& adapter_path) {
   if (track_default_)
     ChangeAdapter(adapter_path);
 }
 
-void BluetoothAdapter::AdapterRemoved(const dbus::ObjectPath& adapter_path) {
+void BluetoothAdapterDBus::AdapterRemoved(
+    const dbus::ObjectPath& adapter_path) {
   if (adapter_path == object_path_)
     RemoveAdapter();
 }
 
-void BluetoothAdapter::ChangeAdapter(const dbus::ObjectPath& adapter_path) {
+void BluetoothAdapterDBus::ChangeAdapter(
+    const dbus::ObjectPath& adapter_path) {
   if (object_path_.value().empty()) {
     DVLOG(1) << "Adapter path initialized to " << adapter_path.value();
   } else if (object_path_.value() != adapter_path.value()) {
@@ -226,7 +242,7 @@ void BluetoothAdapter::ChangeAdapter(const dbus::ObjectPath& adapter_path) {
                     AdapterPresentChanged(this, true));
 }
 
-void BluetoothAdapter::RemoveAdapter() {
+void BluetoothAdapterDBus::RemoveAdapter() {
   const bool adapter_was_present = IsPresent();
 
   DVLOG(1) << "Adapter lost.";
@@ -243,16 +259,16 @@ void BluetoothAdapter::RemoveAdapter() {
                       AdapterPresentChanged(this, false));
 }
 
-void BluetoothAdapter::OnSetPowered(const base::Closure& callback,
-                                    const ErrorCallback& error_callback,
-                                    bool success) {
+void BluetoothAdapterDBus::OnSetPowered(const base::Closure& callback,
+                                        const ErrorCallback& error_callback,
+                                        bool success) {
   if (success)
     callback.Run();
   else
     error_callback.Run();
 }
 
-void BluetoothAdapter::PoweredChanged(bool powered) {
+void BluetoothAdapterDBus::PoweredChanged(bool powered) {
   if (powered == powered_)
     return;
 
@@ -262,10 +278,11 @@ void BluetoothAdapter::PoweredChanged(bool powered) {
                     AdapterPoweredChanged(this, powered_));
 }
 
-void BluetoothAdapter::OnStartDiscovery(const base::Closure& callback,
-                                        const ErrorCallback& error_callback,
-                                        const dbus::ObjectPath& adapter_path,
-                                        bool success) {
+void BluetoothAdapterDBus::OnStartDiscovery(
+    const base::Closure& callback,
+    const ErrorCallback& error_callback,
+    const dbus::ObjectPath& adapter_path,
+    bool success) {
   if (success) {
     DVLOG(1) << object_path_.value() << ": started discovery.";
 
@@ -279,10 +296,10 @@ void BluetoothAdapter::OnStartDiscovery(const base::Closure& callback,
   }
 }
 
-void BluetoothAdapter::OnStopDiscovery(const base::Closure& callback,
-                                       const ErrorCallback& error_callback,
-                                       const dbus::ObjectPath& adapter_path,
-                                       bool success) {
+void BluetoothAdapterDBus::OnStopDiscovery(const base::Closure& callback,
+                                           const ErrorCallback& error_callback,
+                                           const dbus::ObjectPath& adapter_path,
+                                           bool success) {
   if (success) {
     DVLOG(1) << object_path_.value() << ": stopped discovery.";
     callback.Run();
@@ -294,7 +311,7 @@ void BluetoothAdapter::OnStopDiscovery(const base::Closure& callback,
   }
 }
 
-void BluetoothAdapter::DiscoveringChanged(bool discovering) {
+void BluetoothAdapterDBus::DiscoveringChanged(bool discovering) {
   if (discovering == discovering_)
     return;
 
@@ -304,7 +321,7 @@ void BluetoothAdapter::DiscoveringChanged(bool discovering) {
                     AdapterDiscoveringChanged(this, discovering_));
 }
 
-void BluetoothAdapter::OnReadLocalData(
+void BluetoothAdapterDBus::OnReadLocalData(
     const BluetoothOutOfBandPairingDataCallback& callback,
     const ErrorCallback& error_callback,
     const BluetoothOutOfBandPairingData& data,
@@ -315,7 +332,7 @@ void BluetoothAdapter::OnReadLocalData(
     error_callback.Run();
 }
 
-void BluetoothAdapter::AdapterPropertyChanged(
+void BluetoothAdapterDBus::AdapterPropertyChanged(
     const dbus::ObjectPath& adapter_path,
     const std::string& property_name) {
   if (adapter_path != object_path_)
@@ -343,13 +360,13 @@ void BluetoothAdapter::AdapterPropertyChanged(
   }
 }
 
-void BluetoothAdapter::DevicePropertyChanged(
+void BluetoothAdapterDBus::DevicePropertyChanged(
     const dbus::ObjectPath& device_path,
     const std::string& property_name) {
   UpdateDevice(device_path);
 }
 
-void BluetoothAdapter::UpdateDevice(const dbus::ObjectPath& device_path) {
+void BluetoothAdapterDBus::UpdateDevice(const dbus::ObjectPath& device_path) {
   BluetoothDeviceClient::Properties* properties =
       DBusThreadManager::Get()->GetBluetoothDeviceClient()->
       GetProperties(device_path);
@@ -364,13 +381,13 @@ void BluetoothAdapter::UpdateDevice(const dbus::ObjectPath& device_path) {
   // to properties, or the device going from discovered to connected and
   // pairing gaining an object path in the process. In any case, we want
   // to update the existing object, not create a new one.
-  DevicesMap::iterator iter = devices_.find(address);
-  BluetoothDevice* device;
+  BluetoothAdapterDBus::DevicesMap::iterator iter = devices_.find(address);
+  BluetoothDeviceDBus* device;
   const bool update_device = (iter != devices_.end());
   if (update_device) {
     device = iter->second;
   } else {
-    device = BluetoothDevice::Create(this);
+    device = BluetoothDeviceDBus::Create(this);
     devices_[address] = device;
   }
 
@@ -395,12 +412,12 @@ void BluetoothAdapter::UpdateDevice(const dbus::ObjectPath& device_path) {
   }
 }
 
-void BluetoothAdapter::ClearDevices() {
-  DevicesMap replace;
+void BluetoothAdapterDBus::ClearDevices() {
+  BluetoothAdapterDBus::DevicesMap replace;
   devices_.swap(replace);
-  for (DevicesMap::iterator iter = replace.begin();
+  for (BluetoothAdapterDBus::DevicesMap::iterator iter = replace.begin();
        iter != replace.end(); ++iter) {
-    BluetoothDevice* device = iter->second;
+    BluetoothDeviceDBus* device = iter->second;
     if (device->IsSupported() || device->IsPaired())
       FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
                         DeviceRemoved(this, device));
@@ -409,23 +426,23 @@ void BluetoothAdapter::ClearDevices() {
   }
 }
 
-void BluetoothAdapter::DeviceCreated(const dbus::ObjectPath& adapter_path,
-                                     const dbus::ObjectPath& device_path) {
+void BluetoothAdapterDBus::DeviceCreated(const dbus::ObjectPath& adapter_path,
+                                         const dbus::ObjectPath& device_path) {
   if (adapter_path != object_path_)
     return;
 
   UpdateDevice(device_path);
 }
 
-void BluetoothAdapter::DeviceRemoved(const dbus::ObjectPath& adapter_path,
-                                     const dbus::ObjectPath& device_path) {
+void BluetoothAdapterDBus::DeviceRemoved(const dbus::ObjectPath& adapter_path,
+                                         const dbus::ObjectPath& device_path) {
   if (adapter_path != object_path_)
     return;
 
-  DevicesMap::iterator iter = devices_.begin();
+  BluetoothAdapterDBus::DevicesMap::iterator iter = devices_.begin();
   while (iter != devices_.end()) {
-    BluetoothDevice* device = iter->second;
-    DevicesMap::iterator temp = iter;
+    BluetoothDeviceDBus* device = iter->second;
+    BluetoothAdapterDBus::DevicesMap::iterator temp = iter;
     ++iter;
 
     if (device->object_path_ != device_path)
@@ -459,18 +476,18 @@ void BluetoothAdapter::DeviceRemoved(const dbus::ObjectPath& adapter_path,
   }
 }
 
-void BluetoothAdapter::DevicesChanged(
+void BluetoothAdapterDBus::DevicesChanged(
     const std::vector<dbus::ObjectPath>& devices) {
   for (std::vector<dbus::ObjectPath>::const_iterator iter =
            devices.begin(); iter != devices.end(); ++iter)
     UpdateDevice(*iter);
 }
 
-void BluetoothAdapter::ClearDiscoveredDevices() {
-  DevicesMap::iterator iter = devices_.begin();
+void BluetoothAdapterDBus::ClearDiscoveredDevices() {
+  BluetoothAdapterDBus::DevicesMap::iterator iter = devices_.begin();
   while (iter != devices_.end()) {
-    BluetoothDevice* device = iter->second;
-    DevicesMap::iterator temp = iter;
+    BluetoothDeviceDBus* device = iter->second;
+    BluetoothAdapterDBus::DevicesMap::iterator temp = iter;
     ++iter;
 
     if (!device->IsPaired()) {
@@ -484,7 +501,7 @@ void BluetoothAdapter::ClearDiscoveredDevices() {
   }
 }
 
-void BluetoothAdapter::DeviceFound(
+void BluetoothAdapterDBus::DeviceFound(
     const dbus::ObjectPath& adapter_path, const std::string& address,
     const BluetoothDeviceClient::Properties& properties) {
   if (adapter_path != object_path_)
@@ -493,13 +510,13 @@ void BluetoothAdapter::DeviceFound(
   // DeviceFound can also be called to indicate that a device we've
   // paired with is now visible to the adapter during discovery, in which
   // case we want to update the existing object, not create a new one.
-  BluetoothDevice* device;
-  DevicesMap::iterator iter = devices_.find(address);
+  BluetoothDeviceDBus* device;
+  BluetoothAdapterDBus::DevicesMap::iterator iter = devices_.find(address);
   const bool update_device = (iter != devices_.end());
   if (update_device) {
     device = iter->second;
   } else {
-    device = BluetoothDevice::Create(this);
+    device = BluetoothDeviceDBus::Create(this);
     devices_[address] = device;
   }
 
@@ -520,16 +537,16 @@ void BluetoothAdapter::DeviceFound(
   }
 }
 
-void BluetoothAdapter::DeviceDisappeared(const dbus::ObjectPath& adapter_path,
-                                         const std::string& address) {
+void BluetoothAdapterDBus::DeviceDisappeared(
+    const dbus::ObjectPath& adapter_path, const std::string& address) {
   if (adapter_path != object_path_)
     return;
 
-  DevicesMap::iterator iter = devices_.find(address);
+  BluetoothAdapterDBus::DevicesMap::iterator iter = devices_.find(address);
   if (iter == devices_.end())
     return;
 
-  BluetoothDevice* device = iter->second;
+  BluetoothDeviceDBus* device = iter->second;
 
   // DeviceDisappeared can also be called to indicate that a device we've
   // paired with is no longer visible to the adapter, so don't remove
@@ -556,9 +573,9 @@ void BluetoothAdapter::DeviceDisappeared(const dbus::ObjectPath& adapter_path,
 
 
 // static
-scoped_refptr<BluetoothAdapter> BluetoothAdapter::DefaultAdapter() {
+scoped_refptr<BluetoothAdapter> BluetoothAdapterDBus::DefaultAdapter() {
   if (!default_adapter.Get().get()) {
-    BluetoothAdapter* new_adapter = new BluetoothAdapter;
+    BluetoothAdapterDBus* new_adapter = new BluetoothAdapterDBus;
     default_adapter.Get() = new_adapter->weak_ptr_factory_.GetWeakPtr();
     default_adapter.Get()->TrackDefaultAdapter();
   }
@@ -567,8 +584,9 @@ scoped_refptr<BluetoothAdapter> BluetoothAdapter::DefaultAdapter() {
 }
 
 // static
-BluetoothAdapter* BluetoothAdapter::Create(const std::string& address) {
-  BluetoothAdapter* adapter = new BluetoothAdapter;
+BluetoothAdapterDBus* BluetoothAdapterDBus::Create(
+    const std::string& address) {
+  BluetoothAdapterDBus* adapter = new BluetoothAdapterDBus;
   adapter->FindAdapter(address);
   return adapter;
 }