Closed Bug 953036 Opened 6 years ago Closed 6 years ago

[Bluetooth][Certification][PTS][BlueZ][1.3] HSP TC_AG_IAC_BV_01_I Failed

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(blocking-b2g:1.3+, firefox27 wontfix, firefox28 fixed, firefox29 fixed, b2g-v1.3 fixed)

VERIFIED FIXED
1.3 C2/1.4 S2(17jan)
blocking-b2g 1.3+
Tracking Status
firefox27 --- wontfix
firefox28 --- fixed
firefox29 --- fixed
b2g-v1.3 --- fixed

People

(Reporter: echang, Assigned: jaliu)

References

Details

Attachments

(2 files, 3 obsolete files)

Attached file TC_AG_IAC_BV_01_I.zip
### ENV
Buri/BlueZ
[1] https://pvtbuilds.mozilla.org/pvt/mozilla.org/b2gotoro/nightly/mozilla-b2g26_v1_2-hamachi/2013/12/2013-12-17-00-40-01/

### STR
1. PTS 5.0
2. HSP TC_AG_IAC_BV_01_I

### Expected
Pass 

### Actual
est case : TC_AG_IAC_BV_01_I started
	- SDP Service record for PTS: 'Headset HS' successfully registered
	- AT: SPP connect succeeded
	- AT: RING
	- MTC: MTC: Call is connected
	- FATAL ERROR (MTC): Audio connection is expected, but not available.
	- AT: Service Level Connection disabled
	- MTC: Test case ended
Final Verdict : Fail
Flags: needinfo?(jaliu)
Summary: [Bluetooth][Certification][PTS][BlueZ] HSP TC_AG_IAC_BV_01_I Failed → [Bluetooth][Certification][PTS][BlueZ][1.3] HSP TC_AG_IAC_BV_01_I Failed
Blocks: 951055
Assignee: nobody → jaliu
Flags: needinfo?(jaliu)
Comment on attachment 8357078 [details] [diff] [review]
[HSP] Set HSP flag when BluetoothHfpManager chooses to connect HSP. (draft)

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

I think the patch can fix the issue, however mHspConnected has to be renamed to a more reasonable name such as mIsHsp.
Attachment #8357078 - Flags: feedback?(echou) → feedback+
Attachment #8357078 - Attachment is obsolete: true
Attachment #8358140 - Flags: review?(echou)
Comment on attachment 8358140 [details] [diff] [review]
[HSP] Set HSP flag when BluetoothHfpManager chooses to connect HSP.  (v1)

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

Hi Jamin, please see my comments.

::: dom/bluetooth/bluez/BluetoothHfpManager.cpp
@@ +1647,5 @@
>      mHeadsetSocket->Disconnect();
>      mHeadsetSocket = nullptr;
>    } else if (aSocket == mHeadsetSocket) {
>      MOZ_ASSERT(!mSocket);
> +    mIsHsp = true;

Since mIsHsp is assigned to true here, how about also assigning mIsHsp to false in the block above which a HFP incoming connection is established?
Attachment #8358140 - Attachment is obsolete: true
Attachment #8358140 - Flags: review?(echou)
Attachment #8358172 - Flags: review?(echou)
Comment on attachment 8358172 [details] [diff] [review]
[HSP] Set HSP flag when BluetoothHfpManager chooses to connect HSP. (v2)

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

Looks good. Thanks.
Attachment #8358172 - Flags: review?(echou) → review+
Certification blocker. Nominate as 1.3+.
blocking-b2g: --- → 1.3?
Status: NEW → ASSIGNED
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c42e826c8ed3
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
blocking-b2g: 1.3? → 1.3+
Duplicate of this bug: 953042
Duplicate of this bug: 953040
Duplicate of this bug: 953039
Okay with latest v1.3

   │ Gaia      47049555282a9a01fb60d1e1421b57e2810c96f5                           │  
   │ Gecko     http://hg.mozilla.org/releases/mozilla-aurora/rev/6f7dfe36ab6c     │  
   │ BuildID   20140121004137                                                     │  
   │ Version   28.0a2                                                             │  
   │ ro.build.version.incremental=eng.archermind.20131114.105818                  │  
   │ ro.build.date=Thu Nov 14 10:58:33 CST 2013
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.