bluetooth: mac: add connected LE devices to chooser

content/browser/bluetooth/bluetooth_device_chooser_controller.cc

 // Copyright 2016 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.
 
 #include "content/browser/bluetooth/bluetooth_device_chooser_controller.h"
 
 #include <set>
 #include <string>
 #include <unordered_set>
 
@@ skipping 318 matching lines @@
   }
 
   if (!chooser_->CanAskForScanningPermission()) {
     VLOG(1) << "Closing immediately because Chooser cannot obtain permission.";
     OnBluetoothChooserEvent(BluetoothChooser::Event::DENIED_PERMISSION,
                             "" /* device_address */);
     return;
   }
 
   PopulateConnectedDevices();
+  std::unique_ptr<device::BluetoothDiscoveryFilter> discovery_filter =

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

Let's not change web bluetooth yet. A change in web bluetooth requires new
layout tests and mocks which I think is too much for this patch.

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

Are you suggesting to add this code in a next patch?

+      ComputeScanFilter(options_->filters);
+  for (const std::pair<device::BluetoothDevice*,
+                       device::BluetoothDevice::UUIDSet>& pair :
+       adapter_->RetrieveGattConnectedDevicesWithDiscoveryFilter(
+           discovery_filter.get())) {
+    AddFilteredDevice(*pair.first);
+  }
   if (!chooser_.get()) {
     // If the dialog's closing, no need to do any of the rest of this.
     return;
   }
 
   if (!adapter_->IsPowered()) {
     chooser_->SetAdapterPresence(
         BluetoothChooser::AdapterPresence::POWERED_OFF);
     return;
   }
@@ skipping 163 matching lines @@
 
 void BluetoothDeviceChooserController::PostErrorCallback(
     blink::mojom::WebBluetoothResult error) {
   if (!base::ThreadTaskRunnerHandle::Get()->PostTask(
           FROM_HERE, base::Bind(error_callback_, error))) {
     LOG(WARNING) << "No TaskRunner.";
   }
 }
 
 }  // namespace content