Biru [:poiru] Assignee
	Description • 9 years ago

This was never enabled for non-Nightly. We should do so after fixing bug 1105827.

Intent to ship thread: https://lists.mozilla.org/pipermail/dev-platform/2015-August/011466.html

	Marco Castelluccio [:marco]
	Comment 1 • 9 years ago

What's the status here?

	Biru [:poiru] Assignee
	Comment 2 • 9 years ago

(In reply to Marco Castelluccio [:marco] from comment #1)
> What's the status here?

I'll have patches up for this and bug 1221104 soon.

	Andrea Marchesini [:baku]
	Comment 4 • 9 years ago

What about if we do 1 release cycle just enabling the pref. Then we remove it completely.

	Biru [:poiru] Assignee
	Comment 5 • 9 years ago

(In reply to Andrea Marchesini (:baku) from comment #4)
> What about if we do 1 release cycle just enabling the pref. Then we remove
> it completely.

Sure.

	Andrea Marchesini [:baku]
	Comment 6 • 9 years ago

Comment on attachment 8699859 [details] [diff] [review]
Let Permissions API ride the trains

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

Good! and file a follow up to remove the pref completely, but in the next release cycle. Thanks!

	Biru [:poiru] Assignee
	Comment 7 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2caa17e29515

	Biru [:poiru] Assignee
	Comment 8 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/541be3cffed34161fd6555d960320f7b7f933895
Bug 1221106 - Let Permissions API ride the trains. r=baku

	Carsten Book [:Tomcat]
	Comment 9 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/541be3cffed3

	Phil Ringnalda (:philor)
	Comment 10 • 8 years ago

You flipped the pref, but didn't update http://mxr.mozilla.org/mozilla-central/source/dom/tests/mochitest/general/test_interfaces.html?force=1#925 so that test is going to fail when this merges to aurora.

	Biru [:poiru] Assignee
	Comment 11 • 8 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/77a5afd82c63be762af14902439eb62b382ea59c
Bug 1221106 - Follow-up: Fix future test_interfaces.html failure for Permission{s,Status} interfaces. r=baku

	Biru [:poiru] Assignee
	Comment 12 • 8 years ago

(In reply to Phil Ringnalda (:philor) from comment #10)
> You flipped the pref, but didn't update
> http://mxr.mozilla.org/mozilla-central/source/dom/tests/mochitest/general/
> test_interfaces.html?force=1#925 so that test is going to fail when this
> merges to aurora.

Excellent catch, thanks! I committed a fix (and lied about "r=baku").

	Ryan VanderMeulen [:RyanVM]
	Comment 13 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/77a5afd82c63

	Matthew N. [:MattN]
	Comment 14 • 8 years ago

I fixed the version on the support table at https://developer.mozilla.org/en-US/docs/Web/API/Permissions_API#Browser_Support which initially said Fx45.

	Chris Mills [:cmills]
	Comment 15 • 8 years ago

(In reply to Matthew N. [:MattN] (behind on bugmail) from comment #14)
> I fixed the version on the support table at
> https://developer.mozilla.org/en-US/docs/Web/API/
> Permissions_API#Browser_Support which initially said Fx45.

Thanks for leading the charge on getting this updated, Matt. I've followed up on this by updating the permissions on these pages (and their member subpages where relevant):

https://developer.mozilla.org/en-US/docs/Web/API/WorkerNavigator
https://developer.mozilla.org/en-US/docs/Web/API/WorkerNavigator/permissions
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/permissions
https://developer.mozilla.org/en-US/docs/Web/API/Permissions
https://developer.mozilla.org/en-US/docs/Web/API/PermissionStatus

I've also added a note to the relevant release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/46#Miscellaneous

Let me know if anything looks untoward. Thanks!