191207 - meta http-equiv="refresh" with illegal URL incompatible with IE

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect)

Description

[Ralf-Peter Rohbeck]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2.1) Gecko/20021212
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2.1) Gecko/20021212

Not really a bug but an incompatibility with IE:
<meta http-equiv="refresh" content="3;url=mozredir 2.html"> will not open
'mozredir%202.html' but 'mozredir'.
Unfortunately, file names with embedded spaces are popular in Windoze/IE
dominated Intranets like ours :(


Reproducible: Always

Steps to Reproduce:
1. Open http://us.geocities.com/rohbeck/mozredir1.html

Actual Results:  
Mozilla redirects to http://us.geocities.com/rohbeck/mozredir


Expected Results:  
IE redirects to http://us.geocities.com/rohbeck/mozredir%202.html
(doesn't exist either because Geocities doesn't allow space in file name)

Comment 1

[Matthias Versen [:Matti]]

-> docshell

Comment 2

[Adam Lock]

Attachment: Patch

Problem was that the URL parser in the meta refresh stopped at the first
whitespace. I've changed the loop to carry on.

Comment 3

[Adam Lock]

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

Requesting r/sr on another meta refresh issue. This patch parses to the end of
the string or the closing quotation mark so that URLs containing spaces are
caught.

Comment 4

[Boris Zbarsky [:bzbarsky]]

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

>+        if (isQuotedURI && *iter == '"' || *iter == '\'')

&& has higher precedence than || does.	You need parens here.

sr=bzbarsky with that change.

Comment 5

[Alec Flett]

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

sr=alecf

Comment 6

[Adam Lock]

Fix is checked in with parens fix.