Closed Bug 890928 Opened 7 years ago Closed 7 years ago
Crash when creating object url in subworker
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:22.0) Gecko/20100101 Firefox/22.0 (Beta/Release) Build ID: 20130618035212 Steps to reproduce: Create a web worker (worker1) which spawns a new web worker (worker2) and sends it a message. Let worker2 create a objectURL. I made you a fiddle with a "Crash firefox"-button: http://jsfiddle.net/HmNLP/ Actual results: Firefox crashes Expected results: Firefox shouldn't crash... and make my objectURL
This WFM on Nightly. Can you please post one/more crash IDs? You can find them by going to about:crashes in your browser. Thank you! :-)
I can, however, reproduce on beta: https://crash-stats.mozilla.com/report/index/2923ba5c-81e4-403b-bae1-ec0402130708 ...
Might be related to bug 883784.
Definitely looks like bug 883784.
Severity: normal → critical
Crash Signature: [@ CreateURLRunnable::MainThreadRun() ]
Component: Untriaged → DOM: Workers
Product: Firefox → Core
Hardware: x86_64 → x86
(In reply to David Rajchenbach Teller [:Yoric] from comment #5) > Definitely looks like bug 883784. Yes, getting a nightly that doesn't have that at the minute. Maybe we want to uplift that if the crash is web-triggerable? It's already on Aurora now, I can't judge whether that's a patch that we could safely land on beta.
Confirmed. Yoric, wanna see if we can land that on beta? (I don't actually know what to do with this bug now... technically it's FIXED, I guess? Confirming for now... :-\ )
We should see that with baku.
Assignee: nobody → amarchesini
bug 883784 is needed but it's not enough. Here a patch to fix this issue.
Comment on attachment 772111 [details] [diff] [review] patch Review of attachment 772111 [details] [diff] [review]: ----------------------------------------------------------------- Thanks!
Attachment #772111 - Flags: review?(bent.mozilla) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/f48599a7efa2 Friendly reminder - commit messages should be summarizing what the patch is doing, not just blindly restating the bug summary.
Backed out for B2G mochitest orange. https://hg.mozilla.org/integration/mozilla-inbound/rev/becae65c3e08 https://tbpl.mozilla.org/php/getParsedLog.php?id=25232368&tree=Mozilla-Inbound
https://tbpl.mozilla.org/?tree=Try&rev=8970065a78fe I don't see any problem in try... Could it be something else?
(In reply to Andrea Marchesini (:baku) from comment #14) > https://tbpl.mozilla.org/?tree=Try&rev=8970065a78fe > > I don't see any problem in try... Could it be something else? It's there...B2G mochitest-9. And shockingly, it broke on birch too. Backed out. https://hg.mozilla.org/projects/birch/rev/befbe3aa46fc
Attachment #772111 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.