chrome: bluetooth: hook up DeviceCreated/Removed signals

chrome/browser/chromeos/dbus/bluetooth_adapter_client.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_DBUS_BLUETOOTH_ADAPTER_CLIENT_H_
 #define CHROME_BROWSER_CHROMEOS_DBUS_BLUETOOTH_ADAPTER_CLIENT_H_
 #pragma once
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/observer_list.h"
 #include "base/values.h"
 
 namespace dbus {
 class Bus;
 }  // namespace dbus
 
 namespace chromeos {
 
 // BluetoothAdapterClient is used to communicate with a bluetooth Adapter
 // interface.
 class BluetoothAdapterClient {
  public:
   // Interface for observing changes from a local bluetooth adapter.
   class Observer {
    public:
     virtual ~Observer() {}
 
+    // Called when a new known device has been created.
+    virtual void DeviceCreated(const std::string& adapter_path,
+                               const std::string& object_path) {}

[mukesh agrawal, 2012/02/01 00:48:42]

i'm somewhat surprised that these have default implementations. (i would have
expected Observer to be an abstract base.)

but it is consistent with the existing code, so i'll leave it up to you whether
or not to change.

[satorux1, 2012/02/01 17:45:18]

I think it's not so uncommon for observers to have default implementations, as
not all observers are interested in all events (i.e. with default
implementations, they can only implement events they are interested in).

+
+    // Called when a previously known device is removed.
+    virtual void DeviceRemoved(const std::string& adapter_path,
+                               const std::string& object_path) {}
+
     // Called when the adapter's Discovering property changes.
     virtual void DiscoveringPropertyChanged(const std::string& object_path,
                                             bool discovering) {}
 
     // Called when a new remote device has been discovered.
     // |device_properties| should be copied if needed.
     virtual void DeviceFound(const std::string& object_path,
                              const std::string& address,
                              const DictionaryValue& device_properties) {}
 
@@ skipping 24 matching lines @@
  protected:
   BluetoothAdapterClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BluetoothAdapterClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_DBUS_BLUETOOTH_ADAPTER_CLIENT_H_