Bluetooth scanning does not stop when closing chrome://md-settings page |
||||||||||
Issue descriptionChrome Version: 58.0.3029.18 OS: Chrome OS What steps will reproduce the problem? (1) Navigate to chrome://md-settings and turn BT on (2) Play audio media over BT headphones (3) Open Bluetooth settings (audio starts to stutter - see a separate bug 700959) (4) Close chrome://md-settings window What is the expected result? Bluetooth scanning stops What happens instead? Audio continues to stutter which *suggests* that the Bluetooth scanning is not stopped.
,
Mar 21 2017
After step 4 in #0 , we can see that the Bluetooth scanning is still on in the bt_console page in shell. But i don't see any audio sluttering.
,
Mar 22 2017
#2, interesting, maybe you can comment on bug 700959, I could see it on 2 different samus devices although I've only tried one set of headphones (I've mentioned the exact setup on that other bug).
,
Mar 24 2017
,
Mar 24 2017
,
Mar 27 2017
MD settings is only a Finch experiment in M58
,
Mar 31 2017
,
Apr 7 2017
This can also be detected by observing chrome://bluetooth-internals/#adapter
,
Apr 7 2017
,
Apr 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f4de1dce7be5bc5c6fcf4615478757773dace027 commit f4de1dce7be5bc5c6fcf4615478757773dace027 Author: stevenjb <stevenjb@chromium.org> Date: Thu Apr 13 02:47:46 2017 MD Settings: Bluetooth: Fix adapter state and discovery This CL: * Adds some additional bluetooth event logging (USER events). * Fixes the toggle logic so as not to set powered=false before the adapter state is received or when the toggle state matches the adapter state. * Disables the toggle until the adapter state changes (not normally observable, but can prevent rapid toggling artifacts). * Modifies BluetoothEventRouter to stop discovery when all WebUI tabs (or all tabs/windows associated with an extension) are closed. BUG= 703694 , 703698 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2801403002 Cr-Commit-Position: refs/heads/master@{#464258} [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/browser/resources/settings/bluetooth_page/bluetooth_page.html [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/browser/resources/settings/bluetooth_page/bluetooth_page.js [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/browser/resources/settings/bluetooth_page/bluetooth_subpage.html [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/browser/resources/settings/bluetooth_page/bluetooth_subpage.js [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/test/data/webui/settings/bluetooth_page_tests.js [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/test/data/webui/settings/fake_bluetooth.js [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/chrome/test/data/webui/settings/fake_bluetooth_private.js [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/extensions/browser/api/DEPS [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/extensions/browser/api/bluetooth/bluetooth_api.cc [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/extensions/browser/api/bluetooth/bluetooth_event_router.cc [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/extensions/browser/api/bluetooth/bluetooth_event_router.h [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc [modify] https://crrev.com/f4de1dce7be5bc5c6fcf4615478757773dace027/extensions/browser/api/bluetooth/bluetooth_private_api.cc
,
Apr 13 2017
,
Apr 13 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-59; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-59 label, otherwise remove Merge-TBD label. Thanks.
,
Apr 18 2017
verified on 59.0.3071.8
,
May 31 2017
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by steve...@chromium.org
, Mar 21 2017Owner: steve...@chromium.org
Status: Assigned (was: Unconfirmed)