commit e55cc57a115e86d475cf9647654d508db9077a3e
author Garfield Tan <xutan@google.com> Wed Mar 12 18:55:06 2025
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Mar 12 18:55:06 2025
tree 8651cdc5fbc46f434708401af9f0d0b33cedb90e
parent bd119533f048022ed7ba9fd449ee399989121b9a

device: Stop notifying observers for paired devices on Android

Instead, BluetoothAdapterAndroid updates paired devices every time
GetDevices() is called. It also updates its device cache synchronously
during the process of GetDevices() so its return value contains all
paired devices prior to the call.

It is OK not to notify observers because Web Bluetooth spec doesn't
consider paired states, and Web Serial spec requires Bluetooth devices
pairing before opening the dialog. It doesn't strictly match the API
spec of BluetoothAdapter, but it only affects rare use cases where
devices whose pairing statuses change when a chooser dialog is shown.
The notification is unreliable because it stops when the user or the
system revokes the permission.

Serial chooser also needs to remove options added by OnPortAdded() when
options are initialized, because the notification is sent by
SerialDeviceEnumerator when a port is added.

Bug: 387371131
Test: Both Bluetooth and Serial choosers look OK.
Test: device_unittests
Change-Id: I15098ae6a948af34c954b40a74f3e079a5b7905d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6341269
Reviewed-by: Jack Hsieh <chengweih@chromium.org>
Reviewed-by: Matt Reynolds <mattreynolds@chromium.org>
Commit-Queue: Garfield Tan <xutan@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1431662}