Bluetooth: Send UI notifications when the connecting status changes.

device/bluetooth/bluetooth_adapter_chromeos.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 "device/bluetooth/bluetooth_adapter_chromeos.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 227 matching lines @@
 void BluetoothAdapterChromeOS::PoweredChanged(bool powered) {
   if (powered == powered_)
     return;
 
   powered_ = powered;
 
   FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
                     AdapterPoweredChanged(this, powered_));
 }
 
+void BluetoothAdapterChromeOS::DeviceChanged(BluetoothDeviceChromeOS* device) {
+  CHECK(device->adapter_ == this);
+
+  FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+                    DeviceChanged(this, device));

[keybuk, 2013/04/18 00:52:24]

There are other instances of this code in BluetoothAdapterChromeOS - they should
be changed to call this function instead

[deymo, 2013/04/18 01:38:40]

In a deprecated implementation? It is worth the refactor?

[keybuk, 2013/04/18 15:54:17]

Yes.

[deymo, 2013/04/19 01:40:30]

Done.

+}
+
 void BluetoothAdapterChromeOS::OnStartDiscovery(
     const base::Closure& callback,
     const ErrorCallback& error_callback,
     const dbus::ObjectPath& adapter_path,
     bool success) {
   if (success) {
     if (discovering_count_++ == 0) {
       DVLOG(1) << object_path_.value() << ": started discovery.";
 
       // Clear devices found in previous discovery attempts
@@ skipping 282 matching lines @@
     DVLOG(1) << "Paired device " << device->address()
              << " is no longer visible to the adapter";
     device->SetVisible(false);
 
     FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
                       DeviceChanged(this, device));
   }
 }
 
 }  // namespace chromeos