1094090 - Should be declared that trusted apps doesn't have |homescreen-webapps-manage| permission

Status: RESOLVED FIXED

(Core Graveyard :: DOM: Apps, defect)

Description

[Junior [inactive]]

Truested apps doesn't have |homescreen-webapps-manage| permission. The fact should be declared explicitly in dom/apps/PermissionsTable.jsm

Comment 1

[chris]

Attachment: Added trusted: DENY_ACTION for homescreen-webapps-manage

Comment 2

[[:fabrice] Fabrice Desré]

Comment on attachment 8521116 [details] [diff] [review]
Added trusted: DENY_ACTION for homescreen-webapps-manage

Review of attachment 8521116 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

Comment 3

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/integration/b2g-inbound/rev/2498fca6fea4

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/2498fca6fea4

Comment 5

[chris]

The docs are somewhat ambiguous at this point. Do I commit the change or does somebody else have to?

Comment 6

[[:fabrice] Fabrice Desré]

(In reply to chris from comment #5)
> The docs are somewhat ambiguous at this point. Do I commit the change or
> does somebody else have to?

We did already, since you don't have the privileges to do so yourself. That's why the bug was marked as RESOLVED FIXED when it was merged into mozilla-central.

Thanks for your contribution!

Comment 7

[u522318]

I was working on this and waiting for try-server result.
Reassign to Chris since he solved the issue.