New issue
Advanced search Search tips

Issue 684073 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocked on:
issue 583544

Blocking:
issue 510287


Show other hotlists

Hotlists containing this issue:
web-bluetooth


Sign in to add a comment

bluetooth: disconnect should cancel any pending connect calls

Project Member Reported by ortuno@chromium.org, Jan 23 2017

Issue description

Per Step 1 of disconnect[1] we should clear the activealgorithms map to cancel any pending connections.

https://webbluetoothcg.github.io/web-bluetooth/#dom-bluetoothremotegattserver-disconnect
 
Dear ortuno,
Hi, I have a question about this issue.
According from latest code, "activealgorithms" was already cleared by below sequence.
1) BluetoothRemoteGATTServer::disconnect
2) BluetoothDevice::cleanupDisconnectedDeviceAndFireEvent
3) m_gatt->ClearActiveAlgorithms();

Do we need more additional work for clearing "activealgorithms"?
Blocking: -476735 510287
Owner: mattreynolds@chromium.org
Status: Started (was: Available)
djmix.kim@:

Note that BluetoothRemoteGATTServer::disconnect will exit early if connected_ is false. connected_ is only set once ConnectCallback is invoked with a success result, so it will have no effect on pending connections.

Comment 6 by ortuno@chromium.org, Jan 23 2018

fwiw: this shouldn't be fixed without first refactoring device/bluetooth to allow for canceling pending connects.

Comment 7 by cco3@chromium.org, Jan 29 2018

Owner: cco3@chromium.org

Comment 8 by cco3@chromium.org, Jan 29 2018

Blockedon: 583544
 Issue 647662  has been merged into this issue.
Owner: ----
Status: Untriaged (was: Started)
Marking issues assigned to cco3@ as untriaged as they are no longer working on Chromium.
Status: Available (was: Untriaged)

Sign in to add a comment