Crash @ nsDOMBlobBuilder::Flush

RESOLVED FIXED in mozilla15

Status

()

Core
DOM
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: emk, Assigned: emk)

Tracking

({crash, testcase})

Trunk
mozilla15
crash, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox12 affected, firefox13 affected, firefox14 affected)

Details

(crash signature)

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 619362 [details]
test case

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

5 years ago
Created attachment 619363 [details] [diff] [review]
Add a null check in nsXMLHttpRequest::CreatePartialBlob
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Attachment #619363 - Flags: review?(bugs)
(Assignee)

Updated

5 years ago
Blocks: 696586
(Assignee)

Updated

5 years ago
status-firefox12: --- → affected
status-firefox13: --- → affected
status-firefox14: --- → affected
status-firefox15: --- → affected
(Assignee)

Comment 2

5 years ago
Created attachment 619375 [details] [diff] [review]
Add a null check in nsXMLHttpRequest::CreatePartialBlob

.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

5 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

5 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

5 years ago
Whiteboard: [autoland-try:-b do -p all -u all -t none] → [autoland-in-queue]

Updated

5 years ago
Keywords: reproducible → testcase
(Assignee)

Comment 5

5 years ago
Pushed to try manually.
https://tbpl.mozilla.org/?tree=Try&rev=6b932928c210
Keywords: checkin-needed
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
https://hg.mozilla.org/mozilla-central/rev/28a8ddbe2070
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
status-firefox15: affected → ---

Comment 8

5 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.
You need to log in before you can comment on or make changes to this bug.