869332 - [Bluetooth][Certification]TC_AG_ICR_BV_01_I failed due to CIEV missing after call rejected

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Bluetooth, defect)

Description

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

RING
+CLIP: "xxxxxx", 129
AT+CHUP
OK
We shall expect there will be AT commands "+CIEV:4,0" here.
This test based on gecko m-c and gaia:master

I used to test before 3 months before, it works fine.
Not sure it's related to ril interface. Somehow, we need to follow up this problem.
Found similar pattern in Bug 868963.

Comment 1

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

I will test v.1.0.1 again see if problem also exist

Comment 2

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

steal!

Comment 3

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

Attachment: patch 1: v1: fix the problem

When we reject an incoming call, CallStateChanged events would be fired in a series:

Incoming => Busy => Disconnected

Since "Busy" is an intermediate state and it lasts only fof a very short pefiod of time, we should ignore this state and do nothing until the next CallStateChange event coming. However, in current BluetoothHfpManager.cpp, we reset call status (sCINDItems, maintaining Call/CallSetup/CallHeld indicators) while receiving unhandled events. It's not an appropriate way to deal with this kind of events, so I removed it.

Comment 4

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

Nominate as tef+ since it blocks Bluetooth certification.

Comment 5

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

(In reply to Shawn Huang from comment #1)
> I will test v.1.0.1 again see if problem also exist

Confirmed problem now also happened v1.0.1

Comment 6

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

Comment on attachment 746756 [details] [diff] [review]
patch 1: v1: fix the problem

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

Great! We shouldn't reset CIND values for unhandled call state change.

Comment 7

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

birch: https://hg.mozilla.org/projects/birch/rev/ea89e932e5a3

Comment 8

[Ryan VanderMeulen [:RyanVM]]

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

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/c102c5a5afbe
https://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/152c35b8af35

Comment 10

[Mila Davydova]

Unable to verify. Lack of resources. Need Bluetooth PTS tool to verify.
Marking as QARegressExclude.