1205233 - [Presentation WebAPI] Make Presentation API available for browser chrome code

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P1)

Description

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

We need to use Presentation API in browser chrome code for implementing bug 1129785.

Comment 1

[Sean Lin [:seanlin] (inactive from Mar 2016)]

Hi Josh,

Could you help mark 2.5+ to this one?

Comment 2

[Sean Lin [:seanlin] (inactive from Mar 2016)]

Attachment: Patch, v1

Remove "AvailableIn=" from presentation interfaces, so chrome scripts can use them. Yet the interfaces are still protected by "presentation" permission.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8662275 [details] [diff] [review]
Patch, v1

I'd like someone more familiar with permissions and different types of apps to check this too, so, perhaps fabrice can review this.

r+ assuming fabrice agrees.

Comment 4

[[:fabrice] Fabrice Desré]

Comment on attachment 8662275 [details] [diff] [review]
Patch, v1

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

Works for me. In general it's a bad thing to have both a permission check and an "availableIn" annotation in webidl since we can end up with a mismatch between the webidl annotation and PermissionTable.jsm
I started working on fixing that in Bug 1195946

Comment 5

[Sean Lin [:seanlin] (inactive from Mar 2016)]

Attachment: Patch, v1

Updating based on the latest code base.

Comment 6

[Sean Lin [:seanlin] (inactive from Mar 2016)]

The latest try-run. FYI.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f0b230a6d6b3

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/06698874c104

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/06698874c104