819884 - Register new `open-remote-window`app permission.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Alexandre Poirot [:ochameau]]

For now, we hardcode the permission for the homescreen to use window.open() in order to allow opening a new window in another process.
We would like to allow any app to do so through this new permission.
Note that the app needs the permission and to pass `remote=true` in third window.open argument.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Add this permission

Carry over jonas r+ from bug 807438.

Landed on inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/64dabdd4df4a

Unfortunately it will have the wrong bug number in the commit.

Comment 2

[Jason Smith [:jsmith]]

Make sure Paul agrees with this from the security side so that he can update the perms matrix based on what the perms model should be for this permissio. Looking at the patch, I think this makes sense, but make sure Paul knows.

Comment 3

[Alexandre Poirot [:ochameau]]

https://hg.mozilla.org/mozilla-central/rev/64dabdd4df4a

Comment 4

[Paul Theriault [:pauljt] (no longer reading bugmail)]

(In reply to Jason Smith [:jsmith] from comment #2)
> Make sure Paul agrees with this from the security side so that he can update
> the perms matrix based on what the perms model should be for this permissio.
> Looking at the patch, I think this makes sense, but make sure Paul knows.

Looks like Jonas already added this to the spreadsheet, so all good from my perspective.

Comment 5

[Gregor Wagner [:gwagner]]

This is fixed now (on inbound and mc)?

Comment 6

[Alexandre Poirot [:ochameau]]

Yes (inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/64dabdd4df4a)

Comment 7

[Gregor Wagner [:gwagner]]

If you land on mc directly don't forget to close the bug and set the appropriate flags.

Comment 8

[Gene Lian (I already quit Mozilla)]

I'm wondering this one should also be checked in for aurora/beta because all the permission changes need to be bb+ as far as I know (please correct me if I'm wrong).

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/62178c1d1969
https://hg.mozilla.org/releases/mozilla-b2g18/rev/35c796ac53bc

(Landed with the correct bug #)