bluetooth: macOS: Support for extra didConnectPeripheral event from macOS.

device/bluetooth/bluetooth_device_unittest.cc

Index: device/bluetooth/bluetooth_device_unittest.cc
diff --git a/device/bluetooth/bluetooth_device_unittest.cc b/device/bluetooth/bluetooth_device_unittest.cc
index 73ee585e2d2739896ec47a1fd473887d789cc481..f8f11be84e9b488327f54cbd2bd5972cb200ad26 100644
--- a/device/bluetooth/bluetooth_device_unittest.cc
+++ b/device/bluetooth/bluetooth_device_unittest.cc
@@ -1027,6 +1027,7 @@ TEST_F(BluetoothTest,
     return;
   }
   InitWithFakeAdapter();
+  TestBluetoothAdapterObserver observer(adapter_);
   StartLowEnergyDiscoverySession();
   BluetoothDevice* device = SimulateLowEnergyDevice(3);
 
@@ -1039,6 +1040,8 @@ TEST_F(BluetoothTest,
   SimulateGattConnection(device);
   base::RunLoop().RunUntilIdle();
 
+  EXPECT_EQ(1, gatt_discovery_attempts_);

[jlebel, 2017/05/02 19:54:46]

For Android, gatt_discovery_attempts_ is 2. So each time there is a gatt
connection event from the system, there is a gatt discovery.

Do we want the same on macOS? That would mean a full discovery cycle for macOS.

[ortuno, 2017/05/02 23:36:45]

D: That's definitely a bug in Android. Just open an issue about it, add a #if
defined(OS_ANDROID) and mention the issue.

[jlebel, 2017/05/03 21:16:19]

Done.

+  EXPECT_EQ(2, observer.device_changed_count());
   EXPECT_EQ(1, gatt_connection_attempts_);
   EXPECT_EQ(1, callback_count_);
   EXPECT_EQ(0, error_callback_count_);