Closed Bug 940297 Opened 11 years ago Closed 11 years ago

[Gaia] There is no 'absent' cardState in New IccManager's API

Categories

(Firefox OS Graveyard :: Gaia, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
1.3 Sprint 6 - 12/6

People

(Reporter: edgar, Assigned: edgar)

References

Details

Attachments

(1 file, 1 obsolete file)

46 bytes, text/x-github-pull-request
arthurcc
: review+
fcampo
: review+
alive
: review+
Details | Review
In bug 814637, we introduce a new IccManager API for supporting multiple sim. In the new IccManager API, each MozIcc object represent a icc card, and only be created when the card is been detected. So there is no 'absent' in cardState any more, because when card is absent, MozIccManager even doesn't create a MozIcc object.
Assignee: nobody → echen
Attached file WIP (obsolete) —
Attached file pull request #13891
Attachment #8335158 - Attachment is obsolete: true
Comment on attachment 8335831 [details] [review]
pull request #13891

Hi Arthur, could you help to review setting and system part?
Hi Fernando, could you help to review the ftu part?

I have left some comments in github.

Thank you
Attachment #8335831 - Flags: review?(fer.campo.garcia+bugzilla)
Attachment #8335831 - Flags: review?(arthur.chen)
Comment on attachment 8335831 [details] [review]
pull request #13891

FTU part looks nice, thanks
Attachment #8335831 - Flags: review?(fer.campo.garcia+bugzilla) → review+
blocking-b2g: --- → 1.3?
Comment on attachment 8335831 [details] [review]
pull request #13891

r=me for the settings part with comment addressed. Thanks for the effort.

Flagging Alive for the system part.
Attachment #8335831 - Flags: review?(arthur.chen)
Attachment #8335831 - Flags: review?(alive)
Attachment #8335831 - Flags: review+
Attachment #8335831 - Flags: review?(alive) → review+
Keywords: checkin-needed
Master: https://github.com/mozilla-b2g/gaia/commit/03a830c0e590de450caa45ea0597c8726bbe4234
Status: NEW → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.3 Sprint 6 - 12/6
Already in 1.3, so clearing nom.
blocking-b2g: 1.3? → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: