Closed Bug 1266821 Opened 8 years ago Closed 8 years ago

Remove PushPermissionDescriptor from the Permissions API

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox48 + fixed
firefox49 + fixed

People

(Reporter: lina, Assigned: lina)

References

Details

(Keywords: dev-doc-complete, site-compat, Whiteboard: btpp-active)

Attachments

(1 file)

We currently throw "NS_ERROR_NOT_IMPLEMENTED" for PushPermissionDescriptors with `userVisibleOnly: true`. This is incompatible with Chrome, which requires that option.

Since we use a quota instead of userVisibleOnly, I think we should just remove PushPermissionDescriptor entirely, and treat it like any other permission request.
Attachment #8744430 - Flags: review?(amarchesini)
Status: NEW → ASSIGNED
Whiteboard: btpp-active
Attachment #8744430 - Flags: review?(amarchesini) → review+
https://hg.mozilla.org/mozilla-central/rev/d600bbea44ca
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Target Milestone: mozilla48 → mozilla49
Comment on attachment 8744430 [details] [diff] [review]
noPushDescriptor.patch

Approval Request Comment
[Feature/regressing bug #]: Bug 1261405.
[User impact if declined]: Potential web compat issue. Chrome's implementation of the Permissions API requires an extra field when requesting the push permission, but Firefox throws if that field is specified.
[Describe test coverage new/current, TreeHerder]: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d681274533f1, https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=d600bbea44ca
[Risks and why]: Low risk. The bulk of this patch deletes existing code, and the Permissions API is still experimental.
[String/UUID change made/needed]: None.
Attachment #8744430 - Flags: approval-mozilla-aurora?
Keywords: site-compat
Comment on attachment 8744430 [details] [diff] [review]
noPushDescriptor.patch

Fix web compatibility for push notification API.
Attachment #8744430 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Tracking to make sure this lands for 48.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.