Closed
Bug 1207850
Opened 9 years ago
Closed 9 years ago
FetchBody::ContinueConsumeBody called twice
Categories
(Core :: DOM: Service Workers, defect)
Core
DOM: Service Workers
Tracking
()
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: catalinb, Assigned: catalinb)
Details
Attachments
(1 file)
2.41 KB,
patch
|
nsm
:
review+
|
Details | Diff | Splinter Review |
If the service worker is shutting down we run into a wierd case where we try to cancel the pump twice. https://pastebin.mozilla.org/8847298
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8665187 -
Flags: review?(nsm.nikhil)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → catalin.badea392
Status: NEW → ASSIGNED
Comment on attachment 8665187 [details] [diff] [review] Temporary fix for canceling the pump used by FetchBody. r?nsm Review of attachment 8665187 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/fetch/Fetch.cpp @@ +1142,5 @@ > > bool Notify(JSContext* aCx, workers::Status aStatus) override > { > MOZ_ASSERT(aStatus > workers::Running); > + if (!mWasNotified) { Could you also change this to aStatus > Canceling? Ideally this should never be called twice since the call to ContinueConsumeBody should remove the feature, but I can see how having that blocking runnable in ContinueConsumeBody may lead to this being called twice.
Attachment #8665187 -
Flags: review?(nsm.nikhil) → review+
Backed out for android m12 failures: https://hg.mozilla.org/integration/mozilla-inbound/rev/85b9ecca55e5 https://treeherder.mozilla.org/logviewer.html#?job_id=14986157&repo=mozilla-inbound
Flags: needinfo?(catalin.badea392)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(catalin.badea392)
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/00d99c936b39
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in
before you can comment on or make changes to this bug.
Description
•