Closed
Bug 857414
Opened 12 years ago
Closed 11 years ago
[meta] B2G RIL: Move all icc/sim-related stuff from mozMobileConnection to mozIccManager
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: edgar, Unassigned)
References
Details
(Keywords: dev-doc-needed)
Attachments
(6 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
To have a more clear design for mozMobileConnection and mozIccManager, we plan to move all icc/sim-related stuff from mozMobileConnection to mozIccManager as RIL developers have discussed in [1]. And also separate mozIccManager from mozMobileConnection.
[1] https://bugzilla.mozilla.org/show_bug.cgi?id=782603#c42
Reporter | ||
Comment 1•12 years ago
|
||
To avoid breaking icc related function, I plan to add icc-related stuff into IccManager but keep mobileConnection API unchanged. I will file another bug for removing those deprecated stuff from MobileConnection after Gaia has applied this new design. Any other good suggestions? :)
Reporter | ||
Comment 2•12 years ago
|
||
Reporter | ||
Comment 3•12 years ago
|
||
Reporter | ||
Comment 4•12 years ago
|
||
Reporter | ||
Comment 5•12 years ago
|
||
Reporter | ||
Comment 6•12 years ago
|
||
Reporter | ||
Comment 7•12 years ago
|
||
Comment on attachment 732752 [details] [diff] [review]
Part 1: Put icc related attribute and function into IccManager (IDL), v1
Review of attachment 732752 [details] [diff] [review]:
-----------------------------------------------------------------
1.Please discuss with smaug first,
for he's concerns to put icc into navigator in the first place. See Bug 744714 Comment 75.
2. Please break your patches into smaller parts, or make it seperate bugs.
I hope you understand you're changing 'iccinfo', 'card lock', 'card state', 'call forwaring', and these 4 functionalities are heavily used dairy in dogfooding.
3. Remove 'Mobile' in MobileCFInfo, also I would suggest using CallForwardingInfo for a more detail name.
Reporter | ||
Comment 9•12 years ago
|
||
(In reply to Yoshi Huang[:allstars.chh][:yoshi] from comment #8)
> Comment on attachment 732752 [details] [diff] [review]
> Part 1: Put icc related attribute and function into IccManager (IDL), v1
>
> Review of attachment 732752 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> 1.Please discuss with smaug first,
> for he's concerns to put icc into navigator in the first place. See Bug
> 744714 Comment 75.
Ok, I will discuss with :smaug about the naming in navigator.
>
> 2. Please break your patches into smaller parts, or make it seperate bugs.
> I hope you understand you're changing 'iccinfo', 'card lock', 'card state',
> 'call forwaring', and these 4 functionalities are heavily used dairy in
> dogfooding.
I will try to file separate bug.
>
> 3. Remove 'Mobile' in MobileCFInfo, also I would suggest using
> CallForwardingInfo for a more detail name.
Good suggestion, thanks
Reporter | ||
Updated•12 years ago
|
Summary: B2G RIL: Move all icc/sim-related stuff from mozMobileConnection to mozIccManager → [meta] B2G RIL: Move all icc/sim-related stuff from mozMobileConnection to mozIccManager
Updated•12 years ago
|
Keywords: dev-doc-needed
Reporter | ||
Updated•11 years ago
|
Assignee: echen → nobody
Reporter | ||
Comment 10•11 years ago
|
||
All bugs are RESOLVED, we could close this meta bug. :)
Thanks, everyone!
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•