Remove the SecureElement API

RESOLVED FIXED in Firefox 60

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: hsivonen, Assigned: adrian17)

Tracking

({good-first-bug})

unspecified
mozilla60
Points:
---

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

SecureElement and SecureElementManager are chrome-only and always preffed off. They were added in bug 879861 for Firefox OS.

We should remove them, since they aren't in use.
Keywords: good-first-bug
Priority: -- → P3
Assignee: nobody → kyle
Would love to give this a shot if someone can tell me how to get started!
Flags: needinfo?(hsivonen)
(In reply to nikshepsvn from comment #1)
> Would love to give this a shot if someone can tell me how to get started!

Search for SecureElement and SecureElementManager in the codebase and remove them. Then fix things until the code compiles again.
Flags: needinfo?(hsivonen)
Comment on attachment 8947232 [details]
Bug 1353329 - Remove remains of SecureElement API.

https://reviewboard.mozilla.org/r/216986/#review222850

r=me.  Thank you for picking this up!
Attachment #8947232 - Flags: review?(bzbarsky) → review+
Did you run this through try already?
Flags: needinfo?(adrian.wielgosik)
Assignee: kyle → adrian.wielgosik
Thank you!  I triggered autolanding.
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/331af427e379
Remove remains of SecureElement API. r=bz
https://hg.mozilla.org/mozilla-central/rev/331af427e379
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.