Adding support for Low Energy device discovery to BluetoothAdapterMac

device/bluetooth/bluetooth_adapter.h

Index: device/bluetooth/bluetooth_adapter.h
diff --git a/device/bluetooth/bluetooth_adapter.h b/device/bluetooth/bluetooth_adapter.h
index 7c87eb835018f15fea211b56417618f81569f2a3..30e775af802db69566c45d7ee79f42418b908b99 100644
--- a/device/bluetooth/bluetooth_adapter.h
+++ b/device/bluetooth/bluetooth_adapter.h
@@ -422,7 +422,9 @@ class DEVICE_BLUETOOTH_EXPORT BluetoothAdapter
   //      success.
   //
   // |discovery_filter| passed to AddDiscoverySession and RemoveDiscoverySession
-  // is owned by other objects and shall not be freed.
+  // is owned by other objects and shall not be freed.  When the count is
+  // greater than 0 and AddDiscoverySession (RemoveDiscoverySession) is called

[armansito, 2015/06/09 19:32:58]

I'm confused by the parentheses? Do you mean "AddDiscoverySession or
RemoveDiscoverySession is called"?

[krstnmnlsn, 2015/06/09 23:09:00]

Yep, switched.

+  // the filter being used by the underlying controller must be updated.
   //
   // These methods invoke |callback| for success and |error_callback| for
   // failures.
@@ -433,6 +435,8 @@ class DEVICE_BLUETOOTH_EXPORT BluetoothAdapter
       BluetoothDiscoveryFilter* discovery_filter,
       const base::Closure& callback,
       const ErrorCallback& error_callback) = 0;

[armansito, 2015/06/09 19:32:59]

nit: New line before the comment block (keeps things visually easier to parse).

[krstnmnlsn, 2015/06/09 23:09:01]

Done.

+  // Used to set and update the discovery filter used by the underlying
+  // bluetooth controller.

[armansito, 2015/06/09 19:32:59]

nit: s/bluetooth/Bluetooth/

[krstnmnlsn, 2015/06/09 23:09:00]

Done.

   virtual void SetDiscoveryFilter(
       scoped_ptr<BluetoothDiscoveryFilter> discovery_filter,
       const base::Closure& callback,