Closed Bug 1149177 Opened 6 years ago Closed 6 years ago

[Secure Element] UiccConnector should use nsIIccService to listen for card state changes

Categories

(Firefox OS Graveyard :: NFC, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(firefox40 fixed)

RESOLVED FIXED
2.2 S9 (3apr)
Tracking Status
firefox40 --- fixed

People

(Reporter: tauzen, Assigned: tauzen)

References

Details

Attachments

(1 file, 1 obsolete file)

In Bug 1114935 nsIIccProvider parts responsible for card state changes were moved to a new nsIIccService interface. These changes broke UiccConnector initialisation. SE API is not able to access SIM card right now.
Assignee: nobody → kmioduszewski
See Also: → 1114935
Hi Yoshi, could you review this?
Attachment #8585541 - Flags: review?(allstars.chh)
Comment on attachment 8585541 [details] [diff] [review]
0001-Bug-1149177-Secure-Element-UiccConnector-using-nsIIc.patch

Review of attachment 8585541 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/secureelement/gonk/UiccConnector.js
@@ +83,5 @@
>                   Ci.nsIIccListener,
>                   Ci.nsIObserver]
>    }),
>  
> +  UICC_NOT_READY_STATES:[

put this to local
Attachment #8585541 - Flags: review?(allstars.chh) → review+
Attached patch Patch v 1.1Splinter Review
Addressed comment from Yoshi. Try build progressing: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4a1293097fcf
Attachment #8585541 - Attachment is obsolete: true
Attachment #8585943 - Flags: review+
Try build is green.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0a014b0fc944
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S9 (3apr)
You need to log in before you can comment on or make changes to this bug.