Closed
Bug 90383
Opened 23 years ago
Closed 23 years ago
Don't convert backslashes in URIs to forward slashes
Categories
(Core :: DOM: Navigation, defect)
Tracking
()
VERIFIED
WORKSFORME
People
(Reporter: braden, Assigned: adamlock)
References
()
Details
(Keywords: platform-parity)
Attachments
(1 file)
(deleted),
text/html
|
Details |
Why is this done? I thought the conclusion of previous discussions about this
was that Mozilla would be liberal about what it accepted in the location bar,
but respect the specification as far as URIs in documents.
* I think that whatever the conclusion is, it should be the same for all
platforms. Having links "work" on one platform and be nonfunctional on another
is just silly. It is certainly not consistent with a goal of providing a
platform-independent user experience.
* Accommodating the Wrong Thing just encourages bad practice; and silently, too.
Keywords: mozilla1.0,
pp
Comment 1•23 years ago
|
||
-> Embedding: Docshell
Assignee: asa → adamlock
Component: Browser-General → Embedding: Docshell
QA Contact: doronr → adamlock
I don't believe default URI fixup occurs for link clicks but it definitely does
for the address bar. I should have mentioned that in my npm.general post if that
is what you were responding to.
I will verify with a test case containing various silly URLs.
Marking WORKSFORME.
The only "bad url" to work is the file://c:\ one. Probably the file URI parser
is doing the conversion for that. The other two are correctly rejected.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WORKSFORME
Adam: Yup, this is in response to your ng posting. I don't have access to
Windows, so I couldn't test exactly where this occurs. Sorry if I seemed to be
jumping the gun, but I didn't want this to fall through the cracks if it
actually was an issue.
Marking VERIFIED.
(The file: URL thing is probably a side effect of the fact that Mozilla will
accept *anything* in the authority part of a file: URL. This is wrong, but not
terribly important since file: URLs are inevitably system-specific.)
Status: RESOLVED → VERIFIED
Comment 6•23 years ago
|
||
There is still some conversion stuff in nsURLHelper. See bug 32895 for more
information.
You need to log in
before you can comment on or make changes to this bug.
Description
•