Closed
Bug 750023
Opened 13 years ago
Closed 13 years ago
Crash @ nsDOMBlobBuilder::Flush
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: emk, Assigned: emk)
References
Details
(Keywords: crash, testcase)
Crash Data
Attachments
(2 files, 1 obsolete file)
506 bytes,
text/html
|
Details | |
2.11 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
https://crash-stats.mozilla.com/report/index/bp-7cf58b26-e994-4125-9c57-90a1e2120429
Steps to reproduce:
Load the attached test case. (Do not download the test. it doesn't reproduce locally)
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
status-firefox12:
--- → affected
status-firefox13:
--- → affected
status-firefox14:
--- → affected
status-firefox15:
--- → affected
Assignee | ||
Comment 2•13 years ago
|
||
.response shouldn't throw even if aborted.
Attachment #619363 -
Attachment is obsolete: true
Attachment #619363 -
Flags: review?(bugs)
Attachment #619375 -
Flags: review?(bugs)
Comment 3•13 years ago
|
||
Comment on attachment 619375 [details] [diff] [review]
Add a null check in nsXMLHttpRequest::CreatePartialBlob
> load 709384.html
> load 713417.html
> load 713417-2.html
> load 715056.html
> load 741163-1.html
>+HTTP(..) load xhr_abortinprogress.html
I have no idea what HTTP(..) means here.
Is it that the page is loaded from http: and not from file: ?
Attachment #619375 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Yes. crashtest is just a kind of reftest.
https://mxr.mozilla.org/mozilla-central/source/layout/tools/reftest/README.txt?rev=866777368827#176
Whiteboard: [autoland-try:-b do -p all -u all -t none]
Updated•13 years ago
|
Whiteboard: [autoland-try:-b do -p all -u all -t none] → [autoland-in-queue]
Updated•13 years ago
|
Keywords: reproducible → testcase
Assignee | ||
Comment 5•13 years ago
|
||
Pushed to try manually.
https://tbpl.mozilla.org/?tree=Try&rev=6b932928c210
Keywords: checkin-needed
Comment 6•13 years ago
|
||
Try run looks good, so I pushed this to m-i:
https://hg.mozilla.org/integration/mozilla-inbound/rev/28a8ddbe2070
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: [autoland-in-queue]
Target Milestone: --- → mozilla15
Comment 7•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
status-firefox15:
affected → ---
Comment 8•13 years ago
|
||
Autoland Patchset:
Patches: 619375
Branch: mozilla-central => try
Patch 619375 could not be applied to mozilla-central.
file content/base/crashtests/xhr_abortinprogress.html already exists
1 out of 1 hunks FAILED -- saving rejects to file content/base/crashtests/xhr_abortinprogress.html.rej
patching file content/base/src/nsXMLHttpRequest.cpp
Hunk #1 FAILED at 1027
1 out of 1 hunks FAILED -- saving rejects to file content/base/src/nsXMLHttpRequest.cpp.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working dir
Patchset could not be applied and pushed.
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•