Delayed adding BluetoothEventRouter as BluetoothAdapter observer until when the

device/bluetooth/bluetooth_adapter_chromeos.cc

Index: device/bluetooth/bluetooth_adapter_chromeos.cc
diff --git a/device/bluetooth/bluetooth_adapter_chromeos.cc b/device/bluetooth/bluetooth_adapter_chromeos.cc
index 36699e556843ed732046c26631f5f137f0b57736..a355f40b7dc413c608b8dbed2bdae3e3b88c0b7e 100644
--- a/device/bluetooth/bluetooth_adapter_chromeos.cc
+++ b/device/bluetooth/bluetooth_adapter_chromeos.cc
@@ -51,7 +51,8 @@ BluetoothAdapterChromeOs::~BluetoothAdapterChromeOs() {
 void BluetoothAdapterChromeOs::AddObserver(
     BluetoothAdapter::Observer* observer) {
   DCHECK(observer);
-  observers_.AddObserver(observer);
+  if (!observers_.HasObserver(observer))

[miket_OOO, 2012/11/06 02:04:09]

Aha. Would it make sense to move this check into GetMutableAdapter() and
preserve the expectation that a given observer can be added only once? One might
make the argument that it's the caller's job, not the AddObserver(), to make
sure not to add twice. It's not a huge deal, but it feels like an inconsistency.

[youngki, 2012/11/06 16:18:54]

Done. Reverted this file and this check is now done by bluetooth_event_router.

+    observers_.AddObserver(observer);
 }
 
 void BluetoothAdapterChromeOs::RemoveObserver(