B2G Multi-SIM: get the number of RadioInterfaces

RESOLVED FIXED in Firefox 28

Status

()

Core
DOM: Device Interfaces
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: hsinyi, Assigned: hsinyi)

Tracking

unspecified
mozilla27
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking-b2g:1.3+, firefox28 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Figure out how DOM/gaia gets the number of sim cards.
(Assignee)

Comment 1

5 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

5 years ago
Blocks: 818353
(Assignee)

Updated

5 years ago
Blocks: 814634
(Assignee)

Updated

5 years ago
Summary: B2G Multi-SIM: get the number of sim cards → B2G Multi-SIM: get the number of RadioInterfaces
(Assignee)

Updated

5 years ago
Assignee: nobody → htsai
(Assignee)

Comment 2

5 years ago
Created attachment 812953 [details] [diff] [review]
842239.patch
(Assignee)

Updated

5 years ago
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+
(Assignee)

Comment 4

5 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!!!
https://hg.mozilla.org/mozilla-central/rev/3959aecea555
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27

Updated

5 years ago
Blocks: 814637

Updated

5 years ago
Blocks: 926343

Comment 7

5 years ago
This blocked more than one 1.3+ bugs. Marked this as 1.3+.
blocking-b2g: --- → 1.3+
status-firefox28: --- → fixed
You need to log in before you can comment on or make changes to this bug.