Closed
Bug 869332
Opened 8 years ago
Closed 8 years ago
[Bluetooth][Certification]TC_AG_ICR_BV_01_I failed due to CIEV missing after call rejected
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(blocking-b2g:tef+, firefox21 wontfix, firefox22 wontfix, firefox23 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 fixed)
RESOLVED
FIXED
blocking-b2g | tef+ |
People
(Reporter: shawnjohnjr, Assigned: echou)
References
Details
(Whiteboard: [fixed-in-birch] QARegressExclude)
Attachments
(1 file)
905 bytes,
patch
|
gyeh
:
review+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Comment 1•8 years ago
|
||
I will test v.1.0.1 again see if problem also exist
Blocks: bt-certi-blocking
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → shuang
Assignee | ||
Comment 3•8 years ago
|
||
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.
Attachment #746756 -
Flags: review?(gyeh)
Assignee | ||
Comment 4•8 years ago
|
||
Nominate as tef+ since it blocks Bluetooth certification.
blocking-b2g: --- → tef?
Reporter | ||
Comment 5•8 years ago
|
||
(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
Updated•8 years ago
|
blocking-b2g: tef? → tef+
Comment 6•8 years ago
|
||
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.
Attachment #746756 -
Flags: review?(gyeh) → review+
Assignee | ||
Comment 7•8 years ago
|
||
birch: https://hg.mozilla.org/projects/birch/rev/ea89e932e5a3
Assignee | ||
Updated•8 years ago
|
Whiteboard: [fixed-in-birch]
Comment 8•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ea89e932e5a3
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Comment 9•8 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g18/rev/c102c5a5afbe https://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/152c35b8af35
status-b2g18:
--- → fixed
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → fixed
status-firefox21:
--- → wontfix
status-firefox22:
--- → wontfix
status-firefox23:
--- → fixed
Comment 10•8 years ago
|
||
Unable to verify. Lack of resources. Need Bluetooth PTS tool to verify. Marking as QARegressExclude.
Whiteboard: [fixed-in-birch] → [fixed-in-birch] QARegressExclude
You need to log in
before you can comment on or make changes to this bug.
Description
•