Remove ServiceWorker registration when installation failure occurs.

RESOLVED FIXED in mozilla38

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: nsm, Assigned: nsm)

Tracking

unspecified
mozilla38
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 8539241 [details] [diff] [review]
Remove registration when installation failure occurs.
Attachment #8539241 - Flags: review?(amarchesini)
Created attachment 8539409 [details] [diff] [review]
broken h2 dependency id
Attachment #8539409 - Flags: review?(hurley)
Assignee: nsm.nikhil → mcmanus
Comment on attachment 8539409 [details] [diff] [review]
broken h2 dependency id

landed this in wrong bug. sorry.
Attachment #8539409 - Attachment is obsolete: true
Attachment #8539409 - Flags: review?(hurley)
Assignee: mcmanus → nsm.nikhil
Comment on attachment 8539241 [details] [diff] [review]
Remove registration when installation failure occurs.

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

::: dom/workers/ServiceWorkerManager.cpp
@@ +676,5 @@
> +  }
> +
> +  void
> +  MaybeRemoveRegistration()
> +  {

MOZ_ASSERT(mRegistration)
Attachment #8539241 - Flags: review?(amarchesini) → review+
Flags: needinfo?(nsm.nikhil)
https://hg.mozilla.org/mozilla-central/rev/56339532dd96
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in before you can comment on or make changes to this bug.