941584 - [bluedroid] Assertion failure: !mController, dom/bluetooth/bluedroid/BluetoothHfpManager.cpp:1243

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Bluetooth, defect)

Description

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

STR:
1. Connect HFP/A2DP headset
2. Force to turn off bluetooth

BluetoothHfpManager::Disconnect(BluetoothProfileController* aController)
{
  MOZ_ASSERT(NS_IsMainThread());

  NS_ENSURE_TRUE_VOID(sBluetoothHfpInterface);

  bt_bdaddr_t deviceBdAddress;
  StringToBdAddressType(mDeviceAddress, &deviceBdAddress);
  NS_ENSURE_TRUE_VOID(BT_STATUS_SUCCESS ==
    sBluetoothHfpInterface->disconnect(&deviceBdAddress));

  MOZ_ASSERT(!mController);
  mController = aController;
}

Comment 1

[Eric Chou [:ericchou] [:echou]]

I've tested HFP for several days but I have never run into this issue. Since I couldn't reproduce, I'm going to remove this blocker in order to not affect the time line of HFP porting on bluedroid.

ni? Shawn for more investigation.

Comment 2

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

(In reply to Eric Chou [:ericchou] [:echou] from comment #1)
> I've tested HFP for several days but I have never run into this issue. Since
> I couldn't reproduce, I'm going to remove this blocker in order to not
> affect the time line of HFP porting on bluedroid.
> 
> ni? Shawn for more investigation.

Here is the more accurate STR:
1. Connect HFP/A2DP headset
2. Play music via A2DP
3. Force to turn off bluetooth

It looks like this will be easier to hit the problem.

Comment 3

[Joe Cheng [:jcheng] (please needinfo)]

triage: not blocking release. please fix in on trunk and ask for approval to uplift. thanks

Comment 4

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

I cannot reproduce this bug on the latest build, strange :(
3 weeks ago, I can hit the bug.