Closed Bug 842239 Opened 9 years ago Closed 8 years ago

B2G Multi-SIM: get the number of RadioInterfaces

Categories

(Core :: DOM: Device Interfaces, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27
blocking-b2g 1.3+
Tracking Status
firefox28 --- fixed

People

(Reporter: hsinyi, Assigned: hsinyi)

References

Details

Attachments

(1 file)

Figure out how DOM/gaia gets the number of sim cards.
(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.
Summary: B2G Multi-SIM: get the number of sim cards → B2G Multi-SIM: get the number of RadioInterfaces
Assignee: nobody → htsai
Attachment #812953 - Flags: review?(vyang)
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+
(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!!!
https://hg.mozilla.org/mozilla-central/rev/3959aecea555
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
This blocked more than one 1.3+ bugs. Marked this as 1.3+.
blocking-b2g: --- → 1.3+
You need to log in before you can comment on or make changes to this bug.