Closed
Bug 737793
Opened 13 years ago
Closed 13 years ago
Final call state on receiver's phone is 'disconnecting', not 'disconnected'
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: hsinyi, Assigned: hsinyi)
Details
Attachments
(2 files, 1 obsolete file)
|
(deleted),
text/plain
|
Details | |
|
(deleted),
patch
|
bent.mozilla
:
review+
|
Details | Diff | Splinter Review |
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).
|
Assignee | |
Comment 1•13 years ago
|
||
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)
|
|
Assignee | |
Comment 2•13 years ago
|
||
|
|
Assignee | |
Comment 3•13 years ago
|
||
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)
|
||
Updated•13 years ago
|
Attachment #607906 -
Flags: review?(bent.mozilla) → review+
|
|
Assignee | |
Updated•13 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 4•13 years ago
|
||
CCing Chris Jones for my "Commit Access (Level 1)". Thanks!
Please file a separate bug for getting level 1 commit access.
|
||
Comment 6•13 years ago
|
||
|
||
Comment 7•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•