Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(159)

Unified Diff: content/browser/bluetooth/bluetooth_device_chooser_controller.cc

Issue 2339253002: bluetooth: mac: add connected LE devices to chooser (Closed)
Patch Set: Implementing RetrieveGattConnectedDevicesWithDiscoveryFilter() with tests Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | device/bluetooth/bluetooth_adapter.h » ('j') | device/bluetooth/bluetooth_adapter.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/bluetooth/bluetooth_device_chooser_controller.cc
diff --git a/content/browser/bluetooth/bluetooth_device_chooser_controller.cc b/content/browser/bluetooth/bluetooth_device_chooser_controller.cc
index 95315cdf512aedaaef0d8f4415258688de7d0eda..3807b44a0d2765e8fa7de9f3fbb4314f2beedba9 100644
--- a/content/browser/bluetooth/bluetooth_device_chooser_controller.cc
+++ b/content/browser/bluetooth/bluetooth_device_chooser_controller.cc
@@ -336,6 +336,14 @@ void BluetoothDeviceChooserController::GetDevice(
}
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 we
jlebel 2016/11/07 01:43:17 Are you suggesting to add this code in a next patc
+ 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;
« no previous file with comments | « no previous file | device/bluetooth/bluetooth_adapter.h » ('j') | device/bluetooth/bluetooth_adapter.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698