Cătălin Badea (:catalinb) Assignee
	Description • 7 years ago

I think there's a race between running BeginConsumeBodyMainThread on MT and ReleaseObject on the worker thread when the worker is shutting down. This causes this crash (with the patches from 1344751):
https://treeherder.mozilla.org/logviewer.html#?job_id=113828957&repo=try&lineNumber=1891

One possible cause is http://searchfox.org/mozilla-central/source/dom/fetch/FetchConsumer.cpp#689 failing to dispatch when called from Notify, since the worker is already in Terminating state.

	Cătălin Badea (:catalinb) Assignee
	Comment 2 • 7 years ago

This might be the winner: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7df0135c9fa768c8ea63f4c849560f2de2b34255

	Pulsebot
	Comment 3 • 7 years ago

Pushed by catalin.badea392@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/530b1eb0a548
Use Killing when dispatching the sync runnable in FetchBodyConsumer::ShutDownMainThreadConsuming. r=baku

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 4 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/530b1eb0a548