bluetooth: Fix logic mistake in DeviceChanged

content/browser/bluetooth/web_bluetooth_service_impl.cc

Index: content/browser/bluetooth/web_bluetooth_service_impl.cc
diff --git a/content/browser/bluetooth/web_bluetooth_service_impl.cc b/content/browser/bluetooth/web_bluetooth_service_impl.cc
index f2fa91880cb70c87cd37ae5ebe7ee73796bf0017..8e06fc7fa4505a11d53473e3a2436eba1f405533 100644
--- a/content/browser/bluetooth/web_bluetooth_service_impl.cc
+++ b/content/browser/bluetooth/web_bluetooth_service_impl.cc
@@ -217,7 +217,7 @@ void WebBluetoothServiceImpl::DeviceAdded(device::BluetoothAdapter* adapter,
 void WebBluetoothServiceImpl::DeviceChanged(device::BluetoothAdapter* adapter,
                                             device::BluetoothDevice* device) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  if (!device->IsGattConnected() || !device->IsConnected()) {
+  if (!device->IsGattConnected() && !device->IsConnected()) {

[Jeffrey Yasskin, 2016/05/28 02:48:00]

Should we only check one of these? All of our connections are GATT-based, so
maybe just IsGattConnected()?

[Jeffrey Yasskin, 2016/05/28 03:49:52]

Does it make sense to implement it to forward to IsConnected() there, and then
have this call site do the long-term thing?

[ortuno, 2016/05/31 16:33:21]

Done. I thought it was going to be much harder so I went for the one line change
here, since we have to request a merge for this. But it turns out implementing
IsGattConnected was also a one line change :)

     std::string device_id =
         connected_devices_->CloseConnectionToDeviceWithAddress(
             device->GetAddress());