add enumerateAll to navigator.mozApps API

RESOLVED FIXED in mozilla11

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: fabrice, Assigned: fabrice)

Tracking

Other Branch
mozilla11
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
The current API only has an |enumerate()| call that sends different results based on calling context. This can be confusing, so we split it into two parts :

- non privileged callers can use enumerate() to get themselves (apps) or apps installed from their origin (stores)

- privileged callers (eg. dashboards) can use enumerateAll() to get the full list of installed apps.
(Assignee)

Comment 1

6 years ago
Created attachment 580990 [details] [diff] [review]
patch

Exposes enumerateAll(), which was already used internally for the privileged case.
Also add documentation in the IDL and fixes a bug in enumerate()
Attachment #580990 - Flags: review?(gal)

Comment 2

6 years ago
Fabrice, I think we should request privilege if the app does enumerateAll(). That way even if the app is not installed we can pick up the privilege (the dashboard thing the webapps guys are doing will want this I think).
(Assignee)

Comment 3

6 years ago
My plan for the "official dashboard" issue was to use a whitelist pref initially, until we get UX sorted on what they want. This is what the add-on manager is doing to whitelist installs from addons.mozilla.org for instance.
(Assignee)

Comment 4

6 years ago
Created attachment 581704 [details] [diff] [review]
whitelist patch

Adding whitelist support.

Comment 5

6 years ago
Comment on attachment 580990 [details] [diff] [review]
patch

track errors -> report errors
Attachment #580990 - Flags: review?(gal) → review+
(Assignee)

Comment 6

6 years ago
pushed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/abdc706d75d1

Updated

6 years ago
Blocks: 711138

Comment 7

6 years ago
https://hg.mozilla.org/mozilla-central/rev/abdc706d75d1
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.