New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 901681 link

Starred by 1 user
Status: Fixed
Owner:
█ hongjunchoi@chromium.org
Last visit > 30 days ago
Closed: Nov 15
Cc:
█ ortuno@chromium.org
kenrb@chromium.org
martinkr@google.com
engedy@chromium.org
ssoneff@google.com
kpaulhamus@chromium.org
jdoerrie@chromium.org
mknowles@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

Implement BluetoothDevice::IsPaired on MacOS

Project Member Reported by hongjunchoi@google.com, Nov 4 
Currently IsPaired() always returned false on mac for bluetooth low energy devices. Implement IsPaired() function to detect whether a device with a given identifier is already paired with the system. 

This feature is required to support Bluetooth security keys in WebAuthentication API.
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 15 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0

commit 4c30279e0259e1ba76a7c5cd3d5881db4412a5d0
Author: Hongjun Choi <hongjunchoi@google.com>
Date: Thu Nov 15 12:30:43 2018

Implement BluetoothLowEnergyDeviceMac::IsPaired

Implement BluetoothLowEnergyDeviceMac::IsPaired()[1] by watching system
property list file and storing set of device identifiers and device
address which can be used to construct IOBluetoothDevice[2] object as
specified in the design doc[3].

[1]: https://cs.chromium.org/chromium/src/device/bluetooth/bluetooth_device.h?type=cs&q=IsPaired&sq=package:chromium&g=0&l=262
[2]: https://developer.apple.com/documentation/iobluetooth/iobluetoothdevice?language=objc
[3]: go/ispaired-macos-ble

Bug:  901681 
Change-Id: I606f706db0f1557af099809b54c46da3308d0db2
Reviewed-on: https://chromium-review.googlesource.com/c/1307243
Commit-Queue: Jun Choi <hongjunchoi@chromium.org>
Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608335}
[modify] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/BUILD.gn
[modify] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_adapter_mac.h
[modify] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_adapter_mac.mm
[modify] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_adapter_mac_unittest.mm
[modify] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_low_energy_device_mac.h
[modify] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_low_energy_device_mac.mm
[add] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_low_energy_device_watcher_mac.h
[add] https://crrev.com/4c30279e0259e1ba76a7c5cd3d5881db4412a5d0/device/bluetooth/bluetooth_low_energy_device_watcher_mac.mm

Comment 2 by hongjunchoi@chromium.org, Nov 15

Status: Fixed (was: Started)
►

Sign in to add a comment