bluetooth: mac: add connected LE devices to chooser

device/bluetooth/bluetooth_adapter.h

 // 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.
 
 #ifndef DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_H_
 
 #include <stdint.h>
 
 #include <list>
 #include <memory>
 #include <set>
 #include <string>
+#include <unordered_map>
 #include <utility>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/containers/scoped_ptr_hash_map.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
 #include "device/bluetooth/bluetooth_advertisement.h"
@@ skipping 286 matching lines @@
   // |discoverable| is true, then it will be discoverable by other Bluetooth
   // devices. On successfully changing the adapter's discoverability, |callback|
   // will be called. On failure, |error_callback| will be called.
   virtual void SetDiscoverable(bool discoverable,
                                const base::Closure& callback,
                                const ErrorCallback& error_callback) = 0;
 
   // Indicates whether the adapter is currently discovering new devices.
   virtual bool IsDiscovering() const = 0;
 
+  // This function get all the connected peripherals into |devices_|.

[scheib, 2016/10/28 20:35:50]

Let's fix up comments some. First, GetDevices() is currently documented as
returning all connected and paired devices. We need to explain on that method
why that may not be true and that this method is needed.

This method should then explain that it returns connected devices and makes them
known to GetDevices. e.g. here something like:

Returns currently connected peripherals and adds them to known devices returned
by GetDevices().

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

Done.

+  // TODO(crbug.com/653032): Needs to be implemented for Android, ChromeOS and
+  // Windows.
+  virtual std::unordered_map<BluetoothDevice*, BluetoothDevice::UUIDSet>
+  RetrieveGattConnectedDevicesWithDiscoveryFilter(
+      const BluetoothDiscoveryFilter* discovery_filter);
+
   // Requests the adapter to start a new discovery session. On success, a new
   // instance of BluetoothDiscoverySession will be returned to the caller via
   // |callback| and the adapter will be discovering nearby Bluetooth devices.
   // The returned BluetoothDiscoverySession is owned by the caller and it's the
   // owner's responsibility to properly clean it up and stop the session when
   // device discovery is no longer needed.
   //
   // If clients desire device discovery to run, they should always call this
   // method and never make it conditional on the value of IsDiscovering(), as
   // another client might cause discovery to stop unexpectedly. Hence, clients
@@ skipping 285 matching lines @@
   std::set<BluetoothDiscoverySession*> discovery_sessions_;
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<BluetoothAdapter> weak_ptr_factory_;
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_H_