1267907 - crash in mozilla::dom::workers::WorkerPrivate::NotifyFeatures

Status: RESOLVED DUPLICATE of bug 1268303

(Core :: DOM: Workers, defect)

Description

[Luke Wagner [:luke]]

Attachment: debug backtrace

This bug was filed from the Socorro interface and is 
report bp-ee431ba8-13f1-43f6-b0f8-4f67d2160427.
=============================================================

I'm able to reproduce this pretty reliably on trunk and nightly builds by loading up:
  http://lukewagner.github.io/AngryBotsPacked/
and clicking reload a few times after it had started.  

I was also able to repro in a trunk debug build which hits an assertion earlier, under WorkerGlobalScope::Close() (backtrace attached).

Here's the worker script, highlighting the self.close() call in question:
  https://github.com/lukewagner/AngryBotsPacked/blob/gh-pages/Release/asmjsunpack-worker.js#L83

Comment 1

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Attachment: Patch

This covers the assertion, at least.

Comment 3

[Andrea Marchesini [:baku]]

Comment on attachment 8747155 [details] [diff] [review]
Patch

My solution and your solution are actually the same.
I also discussed with smaug about WorkerFeature and we have a nice idea to propose you. The patch is already written, I'm fixing comments and then it will be in your review queue. I would say, today, max tomorrow.