|
2.01 KB,
patch
|
sicking
:
review+
christian
:
approval-mozilla-aurora+
christian
:
approval-mozilla-beta-
|
Details | Diff | Splinter Review |
Page:
<!DOCTYPE HTML>
<html>
<body>
<script type="text/javascript">
var worker = new Worker("boom.js");
worker.postMessage("abcd");
</script>
</body>
</html>
Worker:
var xhr = new XMLHttpRequest();
onmessage = function(event) {
xhr.open("POST", event.data);
xhr.send(null);
}
Result (when run locally):
Assertion failure: foundIndex != size_t(-1), at c:/dev/mozilla-central/dom/worke
rs/Exceptions.cpp:259
On an opt build we print an exception to the error console that has random garbage for text.
Note that I can only reproduce this with a local testcase.
|
(Assignee) | |
Comment 1•6 years ago
|
||
Created attachment 572054 [details] [diff] [review] Patch, v1 We need to safeguard against unknown error codes.
|
|
(Assignee) | |
Comment 2•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6b3cde4cec7f
|
|
(Assignee) | |
Updated•6 years ago
|
||
|
|
(Assignee) | |
Comment 3•6 years ago
|
||
Comment on attachment 572054 [details] [diff] [review] Patch, v1 I'd love to get this into the branches before we merge, it's a tiny fix that avoids a JS_ASSERT and a (harmless) undefined memory access.
|
||
Comment 4•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6b3cde4cec7f
Comment on attachment 572054 [details] [diff] [review] Patch, v1 [triage comment] We're wiling to take this on aurora as the risk looks low (even if the reward is low). Please land today. It's too late for beta.
|
|
(Assignee) | |
Comment 6•6 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/66f33a0dc393
|
||
Comment 7•6 years ago
|
||
Is there something QA can do to verify this fix?
|
(Reporter) | |
Comment 8•6 years ago
|
||
You can verify that (In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #0) > On an opt build we print an exception to the error console that has random > garbage for text. does not happen.
|
|
||
Comment 9•6 years ago
|
||
qa- for someone who is already set up to test this bug to verify the fix.
Description
•