Final call state on receiver's phone is 'disconnecting', not 'disconnected'

RESOLVED FIXED in mozilla14

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: hsinyi, Assigned: hsinyi)

Tracking

unspecified
mozilla14
ARM
Gonk (Firefox OS)
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

If you start two emulators and have one call the other, then have the receiver answer the call and then hang up, the call on the caller's phone correctly proceeds to the 'disconnected' state, but the receiver's call only proceeds to 'disconnecting' and never reaches 'disconnected'. This is reflected in the UI by the dialer app never returning to the main screen (the red 'hang up' button is forever displayed).
Created attachment 607901 [details] [diff] [review]
Made the final call state on receiver's phone to disconnected

After Receiver hangs up the call, receiver gets a response of REQUEST_HANGUP. Then, receiver should get an unsolicited response to update the call state from rild. But that didn't happen in this bug. 

This patch |getCurrentCalls()| to update the call state actively after REQUEST_HANGUP. Then, the final call state on Receiver's phone can reach "disconnected."
Attachment #607901 - Flags: review?(bent.mozilla)
Created attachment 607902 [details]
logcat of the patch
Created attachment 607906 [details] [diff] [review]
[Updated] Patch_Bug 737793

Unnecessary debug messages have been removed in this update version. Thanks!
Attachment #607901 - Attachment is obsolete: true
Attachment #607901 - Flags: review?(bent.mozilla)
Attachment #607906 - Flags: review?(bent.mozilla)
Attachment #607906 - Flags: review?(bent.mozilla) → review+
Keywords: checkin-needed
CCing Chris Jones for my "Commit Access (Level 1)". Thanks!
Please file a separate bug for getting level 1 commit access.
https://hg.mozilla.org/integration/mozilla-inbound/rev/c38c2569a026
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla14

Comment 7

5 years ago
https://hg.mozilla.org/mozilla-central/rev/c38c2569a026
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.