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

Issue 828904 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

ARC Bluetooth repeats callbacks multiple times for the same event

Reported by steven.l...@makewonder.com, Apr 4 2018

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/604.1.38 (KHTML, like Gecko) Version/11.0 Safari/604.1.38
Platform: 10510.0.0 (Official Build) dev-channel kevin

Steps to reproduce the problem:
I've included two sets of reproduction steps.  First is a description of reproducing the problem in code.  Second is a reproduction using an app.

------------

Steps To Reproduce (in code):
(1) Scan for BLE devices using BluetoothLeScanner
(2) Find a device in ScanCallback.onScanResult
(3) Initiate a connection to the device with connectGatt()
(4) Receive notification of connection success in onConnectionStateChange()
(5) Request service discovery with gatt.discoverServices()
(6) Receive discovered services in onServicesDescivered()

Expected Results:
(1) onConnectionStateChange is called only once per connection state change

Actual Results:
(1) onConnectionStateChange is called multiple times with the same parameters

-------------

Steps To Reproduce (in app):
(1) Install attached app (or build from source https://github.com/playi/BleDebugScanner )
(2) Run app.
(3) Turn on a nearby Bluetooth LE device and make sure it is not connected to any phones, tablets or computers
(4) Wait for device to appear on the left column of the app
(5) Tap on the device
(6) Wait for device to connect and appear on the right column of the app.

What is the expected behavior?
(1) "Operation success count: 1", "Connection message count: 1" fields show that onConnectionStateChange was called once for the successful connection

What went wrong?
(1) "Operation success count" and "Connection message count" report multiple calls of onConnectionStateChange with the same parameters

Did this work before? No 

Chrome version: 67.0.3376.0  Channel: dev
OS Version: 67.0.3376.0
Flash Version: 29.0.0.125

- Apk for the app is attached
- Source for the app is at https://github.com/playi/BleDebugScanner
- The code and app work as expected on a Nexus 7
- Screenshot for Chromebook attached
 
app-debug.apk
1.1 MB Download
Screenshot 2018-03-29 at 11.27.08 AM.png
288 KB View Download
Components: Platform>ARC

Comment 2 by minz@google.com, Jun 2 2018

Owner: mcchou@chromium.org

Comment 3 by mcchou@chromium.org, Jun 11 2018

Owner: qiyuh@chromium.org
Status: Assigned (was: Unconfirmed)
@qiyuh, can you help with looking into this? Thanks.

Comment 4 by mcchou@chromium.org, Jun 11 2018

Cc: mcchou@chromium.org josephsih@chromium.org
Components: OS>Systems>Bluetooth
Labels: chro

Comment 5 by qiyuh@chromium.org, Jun 11 2018

Sounds like a bug fixed in https://chromium.googlesource.com/chromium/src/+/e14216f7b1cd035877528c5660b4d4c9d30cad97.

Hi, Steven, could you try M68 (should be dev now) to see whether the issue is still there?
Thanks I will check it out
This issue appears to be solved in M68, thank you.  

I have found that M68 contains a regression in M68 which causes bluetooth device to fail until reboot.  I do not know if this regression is related to this issue, so I recommend closing this ticket.  I will file another with the regression when I have a reproducible case.

Comment 8 by qiyuh@chromium.org, Jun 13 2018

Status: Verified (was: Assigned)
Thank you for confirming.

The regression you mention could be relevant to https://bugs.chromium.org/p/chromium/issues/detail?id=843650.

I would be more than happy to investigate the regression if you find it reproducible.

Closed.
giyuh, I have been able to reliably reproduce the bluetooth issue I mentioned in Comment 7 above.  Please see https://bugs.chromium.org/p/chromium/issues/detail?id=853037 for details.

Sign in to add a comment