1207998 - [PBAP] Assertion failure: (mType) == (aType(unexpected type tag) at BluetoothTypes.h:248

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Bluetooth, defect)

Description

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

Assertion failure: (mType) == (aType(unexpected type tag) 

https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/bluedroid/BluetoothPbapManager.cpp?case=true&from=BluetoothPbapManager.cpp#540

 nsCString text((char *) buf);
 AppendNamedValue(aValues, "searchText", text);

https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/common/webapi/BluetoothAdapter.capp?from=BluetoothAdapter.cpp#1302
    } else if (name.EqualsLiteral("searchText")) {
      init.mSearchValue = value.get_nsString();

Comment 1

[Ben Tian (inactive)]

Attachment: [final] Patch 1: Convert nsCString to nsString for BluetoothVCardListingEvent.mSearchValue, r=shuang

Comment 2

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

Comment on attachment 8681810 [details] [diff] [review]
[final] Patch 1: Convert nsCString to nsString for BluetoothVCardListingEvent.mSearchValue, r=shuang

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

LGTM

Comment 3

[Pulsebot]

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

Comment 4

[Ben Tian (inactive)]

Attachment: [2.2r] Patch 1: Convert nsCString to nsString for BluetoothVCardListingEvent.mSearchValue, r=shuang

Comment 5

[Ben Tian (inactive)]

Set 2.2r blocker since the bug impacts on functionality of required bluetooth feature PBAP.

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/ea945385ee9d

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/ea945385ee9d

Comment 8

[Carsten Book [:Tomcat]]

landed on 2.2r as https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/2c3d9aa77346

Comment 9

[Carsten Book [:Tomcat]]

removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion