240358 - [FIXr]XUL documents resulting from redirect get wrong base URI

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P1)

Description

[Boris Zbarsky [:bzbarsky]]

I don't know how long the testcase in the URL field will stick about, but this
is hard to reproduce without an http server.  For reference, see the thread in
n.p.m.netlib starting with the "04/10/2004 11:58 AM" message from Neil with the
subject "Re: Jar Redirect Problem" (message id <c5994c$[e-mail]>).

The basic problem is that nsXULDocument::StartDocumentLoad differs from
nsDocument::StartDocumentLoad in the way it gets mDocumentURI; as a result, the
pre-redirect URI is used as the base URI for the document if the response was
redirected (eg via HTTP 30x response).

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Patch

Comment 2

[Johnny Stenback (:jst)]

Comment on attachment 145972 [details] [diff] [review]
Patch

+    // XXXbz this code is repeated from nsDocument::ResetToURI; we
+    // really need to refactor this part better.

That's nsDocument::Reset(), not ResetToURI().

Wanna file a followup bug to track the refactoring of this code?

r+sr=jst

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 145972 [details] [diff] [review]
Patch

Could this please be approved for 1.7?	This is very safe; all it does is make
remote XUL that's been redirected to by an HTTP redirect end up with the right
base URI.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Checked in for 1.8a.  Bug 240419 filed.  Leaving open for possible 1.7 checkin.

Comment 5

[Boris Zbarsky [:bzbarsky]]

fixed on the 1.7 branch.

Comment 6

[Boris Zbarsky [:bzbarsky]]

*** Bug 240206 has been marked as a duplicate of this bug. ***