bluetooth: mac: add connected LE devices to chooser

device/bluetooth/test/bluetooth_test_mac.h

Index: device/bluetooth/test/bluetooth_test_mac.h
diff --git a/device/bluetooth/test/bluetooth_test_mac.h b/device/bluetooth/test/bluetooth_test_mac.h
index 0111e58ca698c660435d1f2a93afc205a8f666e9..4a554d289165358d75b8b160e27fe0c8aa9f05b9 100644
--- a/device/bluetooth/test/bluetooth_test_mac.h
+++ b/device/bluetooth/test/bluetooth_test_mac.h
@@ -76,6 +76,12 @@ class BluetoothTestMac : public BluetoothTestBase {
       BluetoothRemoteGattService* service,
       BluetoothRemoteGattCharacteristic* characteristic) override;
 
+  // Simulates a connected low energy device. Used before starting a low energy
+  // discovey session.
+  void SimulateConnectedLowEnergyDevice(

[ortuno, 2016/10/31 04:30:51]

Please add this function to bluetooth_test.h and add a more detailed comment to
it. Similar to SimulateLowEnergyDevice.

[jlebel, 2016/11/07 01:43:18]

Done.

+      int device_ordinal,
+      const BluetoothDevice::UUIDSet& service_uuids);

[ortuno, 2016/10/31 04:30:51]

I like the current pattern of just specifying a single value rather than having
to specify name, identifier, uuids, etc. but I dislike that we pass in a number.
Could you please use an enum class instead and just take the enum.

enum class ConnectedDeviceType {
  GENERIC_DEVICE,
  HEART_RATE_DEVICE
}

SimulateLowEnergyDevice(ConnectedDeviceType type);

[jlebel, 2016/11/07 01:43:18]

Done.

+
   // Callback for the bluetooth central manager mock.
   void OnFakeBluetoothDeviceConnectGattCalled();
   void OnFakeBluetoothGattDisconnect();