There doesn't seem to be any (direct) users of the exchangeAPDU commands in gaia/apps, which target the SIM. They are being replaced with newer Secure Element API proposals: https://bug879861.bugzilla.mozilla.org/attachment.cgi?id=8444259 We should remove them as a prereq.

Removal try results: https://tbpl.mozilla.org/?tree=Try&rev=0b2f004a967e

Comment on attachment 8448961 [details] [diff] [review] Bug 1030602: Remove extra MozICC dom APIs (Gecko side) Review of attachment 8448961 [details] [diff] [review]: ----------------------------------------------------------------- I think you only need to remove the public Web API, not for the internal API and Impl.

(In reply to Yoshi Huang[:allstars.chh] from comment #3) > Comment on attachment 8448961 [details] [diff] [review] > Bug 1030602: Remove extra MozICC dom APIs (Gecko side) > > Review of attachment 8448961 [details] [diff] [review]: > ----------------------------------------------------------------- > > I think you only need to remove the public Web API, not for the internal API > and Impl. I'm not sure we can keep the internals? However, keeping some internals might be useful for reference during implementation. The new proposed SE API has different usage, names and parameters. For example: IccProvider: iccOpenChannel --> MozSESession: openLogicalChannel(aid) iccExchangeAPDU --> MozSEChannel: transmit(MozSECommand) iccCloseChannel --> MozSESession/MozSEChannel: close()

feature/enhancement not a blocker

Bug 879861 will be a new version of Icc API, and will obsolete this bug.

Bug 879861 is for 'Secure Element', Component: NFC, not for ICC, This bug is filed for the ICC part, that's why seperated bugs are filed. Also you have requested r?, reviewer has already spent time on looking your patch. Next time please discuss with reviewer.

Noted. I spoke to Sid before duping it. I'll submit just the MozICC webidl removal, since it's completely obsolete (SE will also modify the interal Icc idl for removing window, so I won't touch that).

Comment on attachment 8448965 [details] Bug 1030602: Remove unused ICC dom APIs (Gaia) https://github.com/mozilla-b2g/gaia/pull/25190

Comment on attachment 8448965 [details] Bug 1030602: Remove unused ICC dom APIs (Gaia) https://github.com/mozilla-b2g/gaia/pull/25190

Recreate attachment.

Comment on attachment 8505809 [details] Bug 1030602: Remove unused ICC dom APIs (Gaia) r+ if it's really deprecated. No one tells why so I assume they are blocking the Secure Element feature.

(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #12) > Comment on attachment 8505809 [details] > Bug 1030602: Remove unused ICC dom APIs (Gaia) > > r+ if it's really deprecated. No one tells why so I assume they are blocking > the Secure Element feature. Thanks. There will be dev hardware support attached to the other end of the API in the new Secure Element API. It will have, among other things, access control, unlike the old one exposed through a WebIDL (originally for a modified Nexus S image).

Gecko side: remove the open/close/transceive DOM API. Try results: https://tbpl.mozilla.org/?tree=Try&rev=2c58d160e9a8

Gaia's Gij seems to be timing out frequently, though it passed once.

remote: WebIDL file dom/webidl/MozIcc.webidl altered in changeset fd26f977e331 without DOM peer review so it needs dom peer review, thanks!

Thanks. Updated patch subject with review tags.

Re-update.