Change AvailableIn to PrivilegedApps for NFC API

RESOLVED FIXED in 2.2 S2 (19dec)

Status

defect
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: allstars.chh, Assigned: dimi)

Tracking

({dev-doc-complete})

unspecified
2.2 S2 (19dec)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

In NFC WebIDL the [AvailableIn] attributes are all set to CertifiedApps. To make NFC API privileged, we need to change these to PrivilegedApps.
(Assignee)

Updated

5 years ago
Assignee: nobody → dlee
(Assignee)

Updated

5 years ago
Summary: Change AvailbleIn to PrivilegedApps → Change AvailableIn to PrivilegedApps
(Assignee)

Comment 1

5 years ago
Posted patch Patch v1 (obsolete) — Splinter Review
Attachment #8518627 - Flags: review?(allstars.chh)
(Assignee)

Comment 2

5 years ago
In the first stage, we will foucs on making p2p sharing API privileged.
Summary: Change AvailableIn to PrivilegedApps → Change AvailableIn to PrivilegedApps for p2p sharing API
(Assignee)

Updated

5 years ago
Attachment #8518627 - Flags: review?(allstars.chh)
(Assignee)

Comment 3

5 years ago
Posted patch [WIP] Patch v2 (obsolete) — Splinter Review
I will wait for bug 1048676 landed and rebase the patch
Attachment #8518627 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Depends on: 1048676
(Assignee)

Comment 4

4 years ago
Posted patch Patch v1Splinter Review
Attachment #8519709 - Attachment is obsolete: true
Attachment #8532290 - Flags: review?(allstars.chh)
Summary: Change AvailableIn to PrivilegedApps for p2p sharing API → Change AvailableIn to PrivilegedApps for NFC API
Attachment #8532290 - Flags: review?(allstars.chh) → review+
(Assignee)

Comment 5

4 years ago
Comment on attachment 8532290 [details] [diff] [review]
Patch v1

Add r? to smaug for the WebIDL changed.
Attachment #8532290 - Flags: review?(bugs)
Attachment #8532290 - Flags: review?(bugs) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/7b4da9717ee0
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S2 (19dec)
You need to log in before you can comment on or make changes to this bug.