Closed
Bug 1174420
Opened 9 years ago
Closed 9 years ago
SimplePush API broken on B2G
Categories
(Core :: DOM: Push Subscriptions, defect)
Core
DOM: Push Subscriptions
Tracking
()
RESOLVED
FIXED
2.2 S14 (12june)
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: gerard-majax, Assigned: gerard-majax)
References
Details
(Keywords: regression, Whiteboard: [systemsfe])
Attachments
(1 file, 1 obsolete file)
963 bytes,
patch
|
gerard-majax
:
review+
|
Details | Diff | Splinter Review |
On my devices running B2G master, FindMyDevice is never able to find my devices, even though the registration seems to be done. Further debugging in bug 1159046 shows that |navigator.push.register()| never trigger a success or error callback or Promise keep waiting. So far I could find that PushServiceLauncher.js was never even loaded. Extracting omni.ja there is definition in the manifest for this service to be triggered upon "app-startup" but this seems not to be working. MOZ_SIMPLEPUSH=1 is present in objdir-gecko/config.status Extracted omni.ja defines app-startup on PushServiceLauncher: > gecko/omni $ grep -R 'PushServiceLauncher' > components/components.manifest:component {4b8caa3b-3c58-4f3c-a7f5-7bd9cb24c11d} PushServiceLauncher.js > components/components.manifest:category app-startup PushServiceLauncher service,@mozilla.org/push/ServiceLauncher;1 But I cannot find PushServiceLauncher in omni.ja: > gecko/omni$ find . | grep PushService > ./chrome/toolkit/content/global/PushServiceChildPreload.js > ./modules/PushService.jsm > gecko/omni$
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → lissyx+mozillians
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8621982 -
Flags: review?(fabrice)
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3a0baa4ea9a1
Assignee | ||
Updated•9 years ago
|
Whiteboard: [systemsfe]
Target Milestone: --- → 2.2 S14 (12june)
Comment 3•9 years ago
|
||
Comment on attachment 8621982 [details] [diff] [review] Package PushService launcher on B2G Review of attachment 8621982 [details] [diff] [review]: ----------------------------------------------------------------- r=me ::: b2g/installer/package-manifest.in @@ +648,5 @@ > @RESPATH@/components/AppsService.manifest > @RESPATH@/components/Push.js > @RESPATH@/components/Push.manifest > @RESPATH@/components/PushNotificationService.js > +@RESPATH@/components/PushServiceLauncher.js We'll want to update the component IDs and paths in `Push.manifest`, too, since the old Simple Push and new Web Push APIs currently use the same ones. But let's land this first, and follow up with bug 1159046.
Attachment #8621982 -
Flags: review?(fabrice) → review+
Assignee | ||
Comment 4•9 years ago
|
||
Just changing the r=, carrying r+
Attachment #8621982 -
Attachment is obsolete: true
Attachment #8621992 -
Flags: review+
You need to log in
before you can comment on or make changes to this bug.
Description
•