941416 - Assertion failure: NS_IsMainThread(),dom/bluetooth/BluetoothService.cpp:520

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Bluetooth, defect)

Description

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

Assertion in SetEnabled() and GetAllBluetoothActors in debugs whenever bluetooth truns on.
BluetoothService::SetEnabled(bool aEnabled)
{
  MOZ_ASSERT(NS_IsMainThread());


GetAllBluetoothActors(InfallibleTArray<BluetoothParent*>& aActors)
{
  MOZ_ASSERT(NS_IsMainThread());

Comment 1

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

Could be related to patch of bug 936995

Comment 2

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

This is a regression of bug 941416. I'll fix it today.

Comment 3

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

(In reply to Eric Chou [:ericchou] [:echou] from comment #2)
> This is a regression of bug 941416. I'll fix it today.

typo: bug 936995

Comment 4

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

Attachment: patch 1: v1: fix

Comment 5

[Gina Yeh [:gyeh] [:ginayeh]]

Comment on attachment 8335823 [details] [diff] [review]
patch 1: v1: fix

Review of attachment 8335823 [details] [diff] [review]:
-----------------------------------------------------------------

Great! Thanks!

Comment 6

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

https://hg.mozilla.org/integration/b2g-inbound/rev/9c70bf3d8b6d

Thanks. :)

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/9c70bf3d8b6d