Chromium Code Reviews
DescriptionPostTask MaybeReleaseAdapter in BluetoothEventRouter::AdapterDiscoveringChanged
Calling MaybeReleaseAdapter directly from
BluetoothEventRouter::AdapterDiscoveringChanged could be a problem since
this can happen:
BluetoothAdapterBlueZ::AdapterPropertyChanged ->
BluetoothAdapterBlueZ::DiscoveringChanged ->
BluetoothEventRouter::AdapterDiscoveringChanged -> MaybeReleaseAdapter
If BluetoothEventRouter is the only pointer holder left then the caller
of BluetoothEventRouter::AdapterDiscoveringChanged could be gone even
before that function returns.
BUG=710216
Review-Url: https://codereview.chromium.org/2852773002
Cr-Commit-Position: refs/heads/master@{#468180}
(cherry picked from commit 518c599b12849fb5ca778427c877c2250d116455)
Review-Url: https://codereview.chromium.org/2865383002 .
Cr-Commit-Position: refs/branch-heads/3071@{#487}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed: https://chromium.googlesource.com/chromium/src/+/277d00927d269a72d50eed92c78bf1a9cc0e9b26
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
|||||||||||||||||||