Closed
Bug 842239
Opened 12 years ago
Closed 11 years ago
B2G Multi-SIM: get the number of RadioInterfaces
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox28 | --- | fixed |
People
(Reporter: hsinyi, Assigned: hsinyi)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
vicamo
:
review+
|
Details | Diff | Splinter Review |
Figure out how DOM/gaia gets the number of sim cards.
|
Assignee | |
Comment 1•11 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #0)
> Figure out how DOM/gaia gets the number of sim cards.
In gaia, always using navigator.mozMobileConnections.length.
In DOM/RIL, using 'nsIRadioInterfaceLayer.numRadioInterfaces.' Patch is on the way.
|
|
Assignee | |
Updated•11 years ago
|
Summary: B2G Multi-SIM: get the number of sim cards → B2G Multi-SIM: get the number of RadioInterfaces
|
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → htsai
|
|
Assignee | |
Comment 2•11 years ago
|
||
|
|
Assignee | |
Updated•11 years ago
|
Attachment #812953 -
Flags: review?(vyang)
|
||
Comment 3•11 years ago
|
||
Comment on attachment 812953 [details] [diff] [review]
842239.patch
Review of attachment 812953 [details] [diff] [review]:
-----------------------------------------------------------------
Per offline discuss, we may still have racing condition in child process. Perfect solution should be deprecate RILContentHelper and move all IPC to IPDL (bug 815526).
Attachment #812953 -
Flags: review?(vyang) → review+
|
|
Assignee | |
Comment 4•11 years ago
|
||
(In reply to Vicamo Yang [:vicamo][:vyang] from comment #3)
> Comment on attachment 812953 [details] [diff] [review]
> 842239.patch
>
> Review of attachment 812953 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> Per offline discuss, we may still have racing condition in child process.
> Perfect solution should be deprecate RILContentHelper and move all IPC to
> IPDL (bug 815526).
Yes, I agree!!!
|
|
Assignee | |
Comment 5•11 years ago
|
||
|
||
Comment 6•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
|
||
Comment 7•11 years ago
|
||
This blocked more than one 1.3+ bugs. Marked this as 1.3+.
blocking-b2g: --- → 1.3+
|
||
Updated•11 years ago
|
status-firefox28:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•