Commit c356b220 authored by Sonny Sasaka's avatar Sonny Sasaka Committed by Commit Bot

device/bluetooth: Remove DCHECK(IsDiscovering()) until BlueZ is fixed.

BlueZ bug makes it wrong to assume that the Discovering property is
updated when StartDiscovery/StopDiscovery returns (crbug.com/822104).
BlueZ bug fixing is deprioritized so here we remove the DCHECK first
until BlueZ bug is fixed.

BUG=814595
TEST=No longer crashes

Change-Id: I12c6d0df808553d3098525b70ab998d8c957b918
Reviewed-on: https://chromium-review.googlesource.com/963845Reviewed-by: default avatarMiao-chen Chou <mcchou@chromium.org>
Commit-Queue: Sonny Sasaka <sonnysasaka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543495}
parent 94b3728a
......@@ -1383,7 +1383,9 @@ void BluetoothAdapterBlueZ::AddDiscoverySession(
// The adapter is already discovering.
if (num_discovery_sessions_ > 0) {
DCHECK(IsDiscovering());
// DCHECK(IsDiscovering()) is removed due to BlueZ bug
// (https://crbug.com/822104).
// TODO(sonnysasaka): Put it back here when BlueZ bug is fixed.
DCHECK(!discovery_request_pending_);
num_discovery_sessions_++;
SetDiscoveryFilter(BluetoothDiscoveryFilter::Merge(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment