Closed
Bug 766891
Opened 10 years ago
Closed 10 years ago
B2G RIL: nsIRadioInterfaceLayer.DATACALL_STATE_* values is not synced with ril_consts.js
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: kanru, Assigned: kanru)
References
Details
Attachments
(1 file, 2 obsolete files)
2.52 KB,
patch
|
Details | Diff | Splinter Review |
And the GECKO_NETWORK_STATE_* values in ril_consts.js is synced with nsINetworkManager.NETWORK_STATE_* Should we conciliate this to one place?
Assignee | ||
Comment 1•10 years ago
|
||
Assignee: nobody → kchen
Attachment #635222 -
Flags: review?(philipp)
Comment 2•10 years ago
|
||
Comment on attachment 635222 [details] [diff] [review] Sync nsIRadioInterfaceLayer.DATACALL_STATE_* with ril_consts.js Review of attachment 635222 [details] [diff] [review]: ----------------------------------------------------------------- Nice catch!
Attachment #635222 -
Flags: review?(philipp) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/edbb25e71396
Target Milestone: --- → mozilla16
Comment 4•10 years ago
|
||
(In reply to Philipp von Weitershausen [:philikon] from comment #2) > Comment on attachment 635222 [details] [diff] [review] > Sync nsIRadioInterfaceLayer.DATACALL_STATE_* with ril_consts.js > Philipp correct me if I'm wrong but we should have (also) marked the interface for iid rev given the _UNKNOWN integer value change.
Assignee | ||
Comment 5•10 years ago
|
||
Backout https://hg.mozilla.org/integration/mozilla-inbound/rev/dced59dc1100
Assignee | ||
Comment 6•10 years ago
|
||
Sorry philikon, last patch was backed out because of b2g bustage. I think it's better to remove DATACALL_STATE_* completely so I made this new patch. try: https://tbpl.mozilla.org/?tree=Try&rev=f288f07c2a25
Attachment #635222 -
Attachment is obsolete: true
Attachment #635657 -
Flags: review?(philipp)
Comment 7•10 years ago
|
||
Ugh, I somehow read the patch as modifying ril_consts.js instead of nsIRadioInterfaceLayer. Must've been distracted, sorry.
Comment 8•10 years ago
|
||
Comment on attachment 635657 [details] [diff] [review] Remove nsIRadioInterfaceLayer.DATACALL_STATE_* Review of attachment 635657 [details] [diff] [review]: ----------------------------------------------------------------- Yes, yes, this makes much more sense. Thanks!
Attachment #635657 -
Flags: review?(philipp) → review+
Assignee | ||
Comment 10•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/14707b4a5677
Comment 11•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/14707b4a5677
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•