Closed
Bug 776480
Opened 12 years ago
Closed 12 years ago
B2G RIL: missing handler for ipc message 'RIL:GetRadioState'
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: hsinyi, Assigned: allstars.chh)
References
Details
Attachments
(1 file)
1.85 KB,
patch
|
philikon
:
review+
|
Details | Diff | Splinter Review |
In 'RILContentHelper.js' we send a sync message 'RIL:GetRadioState' to request radio info. However, there is no corresponding handler for this ipc message.
Reporter | ||
Comment 1•12 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #0) > In 'RILContentHelper.js' we send a sync message 'RIL:GetRadioState' to > request radio info. However, there is no corresponding handler for this ipc > message. Seems this is the regression of 762760? Maybe we should replace "RIL:GetRadioState" with "RIL:GetRilContext" ?
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #1) > (In reply to Hsin-Yi Tsai [:hsinyi] from comment #0) > > In 'RILContentHelper.js' we send a sync message 'RIL:GetRadioState' to > > request radio info. However, there is no corresponding handler for this ipc > > message. > > Seems this is the regression of bug 762760 ? Maybe we should replace > "RIL:GetRadioState" with "RIL:GetRilContext" ?
Comment 3•12 years ago
|
||
Yes, good find. Can you fix? Thanks!
Assignee | ||
Comment 4•12 years ago
|
||
I'll take it since it's my mistake. Thanks for spotting this.
Assignee: nobody → allstars.chh
OS: Linux → Gonk
Hardware: x86_64 → ARM
Assignee | ||
Comment 5•12 years ago
|
||
rename IPC message "GetRadioState" to "GetRilContext" in RILContentHelper.js
Attachment #645218 -
Flags: review?(philipp)
Updated•12 years ago
|
Attachment #645218 -
Flags: review?(philipp) → review+
Assignee | ||
Comment 6•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/288c6ff3dfd8
Target Milestone: --- → mozilla17
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/288c6ff3dfd8
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•