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

RESOLVED FIXED in Firefox 40

Status

Firefox OS
NFC
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: tauzen, Assigned: tauzen)

Tracking

unspecified
2.2 S9 (3apr)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

3 years ago
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)

Updated

3 years ago
Assignee: nobody → kmioduszewski
Blocks: 1119152, 1044428
See Also: → bug 1114935
(Assignee)

Comment 1

3 years ago
Created attachment 8585541 [details] [diff] [review]
0001-Bug-1149177-Secure-Element-UiccConnector-using-nsIIc.patch

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

Comment 3

3 years ago
Created attachment 8585943 [details] [diff] [review]
Patch v 1.1

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

Comment 4

3 years ago
Try build is green.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0a014b0fc944
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S9 (3apr)
You need to log in before you can comment on or make changes to this bug.