Closed
Bug 1148117
Opened 10 years ago
Closed 9 years ago
ServiceWorkerRegistration::GetPushManager will fail on workers.
Categories
(Core :: DOM: Notifications, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 1184574
People
(Reporter: dougt, Unassigned)
References
Details
> +
> +
> + JS::Rooted<JSObject*> jsImplObj(cx);
> + nsCOMPtr<nsIGlobalObject> unused = ConstructJSImplementation(cx, "@mozilla.org/push/PushManager;1",
> + global, &jsImplObj, aRv);
This is going to fail miserably on workers when we expose ServiceWorkerRegistration on workers (Bug 1131327). Could you file a follow-up and add a comment with the bug number here?
The easy solution is that for workers we create a C++ PushManagerProxy that routes calls to their equivalents on the main thread (see DataStore comparison in dom/workers/DataStore.{h,cpp} and dom/datastore)
Depends on: 1131327
Bug description is summary of https://bugzilla.mozilla.org/show_bug.cgi?id=1038811#c50
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•