Closed Bug 473053 Opened 11 years ago Closed 11 years ago

Missing QI causes exclamation mark and DAV_REPORT_ERROR

Categories

(Calendar :: Provider: CalDAV, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

Attached patch Fix - v1Splinter Review
It seems that aLoader.request is now correctly QI'd to nsIRequest, but is missing a QI to nsIHttpChannel. This causes all our code that does aLoader.request.responseStatus to fail, since responseStatus is part of nsIHttpChannel.

This patch fixes, and also fixes a missing backslash in a regex pattern.
Attachment #356405 - Flags: review?(Berend.Cornelius)
If anyone else is around to review this patch, please do :-)
Comment on attachment 356405 [details] [diff] [review]
Fix - v1

r=mschroeder
Attachment #356405 - Flags: review?(Berend.Cornelius) → review+
Thanks for taking over!

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/2d3e259d2c12>

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.