Closed Bug 1036229 Opened 10 years ago Closed 10 years ago

Remove [Pref] from PushManager

Categories

(Core :: DOM: Notifications, defect)

Product:
Core
Component:
DOM: Notifications
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: reuben, Unassigned)

Details

PushManager is also behind a [CheckPermissions] check, it looks like the pref is mostly used to disable it on platforms that aren't B2G. Removing the pref would allow us to include push in the Navigator.getFeature list (bug 1009645).
So removing the pref would mean loading and running the push backend code on all platforms, which has security implications, etc. Let's keep the pref and use [HardwareSupport] or whatever we end up doing from bug 1009645 comment 56.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
Summary: Remove services.push.enabled pref → Remove [Pref] from PushManager
You need to log in before you can comment on or make changes to this bug.