818566 - Permissions Installer - states that a privileged app can access the camera API via PROMPT_ACTION, but permissions matrix says it should be DENY_ACTION

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect, P1)

Description

[Jason Smith [:jsmith]]

The permissions installer for camera states the following:

48                            camera: {
49                              app: DENY_ACTION,
50                              privileged: PROMPT_ACTION,
51                              certified: ALLOW_ACTION
52                            },

http://mxr.mozilla.org/mozilla-central/source/dom/apps/src/PermissionsInstaller.jsm

However, the permissions matrix states that privileged should be DENY_ACTION. Which is not right? The doc or the code?

Comment 1

[Jason Smith [:jsmith]]

Noming in case the code is wrong. If the doc is wrong, then minus and state to update the doc.

Comment 2

[Jason Smith [:jsmith]]

Permissions matrix: https://docs.google.com/spreadsheet/ccc?key=0Akyz_Bqjgf5pdENVekxYRjBTX0dCXzItMnRyUU1RQ0E#gid=0

Comment 3

[Bill Walker [:bwalker] [@wfwalker]]

The purpose of Privileged Apps is to allow developers everywhere to create HTML-based apps that can do things previously reserved for "real" (native) apps.

Considering the popularity of apps like Instagram, we must allow Privileged Apps to access the camera. Note that I'm not talking about switching away to the built-in Camera app using an Intent or Activity, but rather I'm talking about getting continuous access to the live camera preview so I can apply creative filters.

Comment 4

[Damon Sicore (:damons)]

Can we get a priority and target milestone for this>

Comment 5

[Lucas Adamski [:ladamski]]

(In reply to Bill Walker [:bwalker] [@wfwalker] from comment #3)
> The purpose of Privileged Apps is to allow developers everywhere to create
> HTML-based apps that can do things previously reserved for "real" (native)
> apps.
> 
> Considering the popularity of apps like Instagram, we must allow Privileged
> Apps to access the camera. Note that I'm not talking about switching away to
> the built-in Camera app using an Intent or Activity, but rather I'm talking
> about getting continuous access to the live camera preview so I can apply
> creative filters.

No disagreement about the goal here, however the implementation reality is that the camera API as implemented for basecamp is incompatible with process sandboxing, meaning any app that wants direct access to camera will need to break out of the sandbox.  Its something we want to fix, but realistically not for basecamp.

Comment 6

[Gregor Wagner [:gwagner]]

Attachment: patch

Comment 7

[Gregor Wagner [:gwagner]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0770315e3538

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0770315e3538

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/97b22dbac0d1
https://hg.mozilla.org/releases/mozilla-b2g18/rev/35b8db6c3dcc