41718 - javascript: protocol is case sensitive.

Status: VERIFIED FIXED

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

Description

[Prashant Desale]

protocol javascript: is case sensitive and does not allow a single alphabet in 
upper case [Like Javascript:, JAVASCRIPT:]

And there are several sites which use "Javascript:" instead of "javascript:"

According to RFC 1738 [http://www.isi.edu/in-notes/rfc1738.txt]
In general, URLs are written as follows:

       <scheme>:<scheme-specific-part>

   A URL contains the name of the scheme being used (<scheme>) followed
   by a colon and then a string (the <scheme-specific-part>) whose
   interpretation depends on the scheme.

   Scheme names consist of a sequence of characters. The lower case
   letters "a"--"z", digits, and the characters plus ("+"), period
   ("."), and hyphen ("-") are allowed. For resiliency, programs
   interpreting URLs should treat upper case letters as equivalent to
   lower case in scheme names (e.g., allow "HTTP" as well as "http").

So javascript: protocol could also be allowed to use with uppercase letters.

BUILDS: 2000-060508.

STEPS TO REPRODUCE:
1] Please load the testcase I'm going to attach with above specified builds.
2] You'll see four links. Click these all links one by one and observe.

EXPECTED RESULTS:
All four links should generate alerts. [All links use different case combination 
for protocol javascript:]

ACTUAL RESULTS:
One first link generates alert.

Comment 1

[Prashant Desale]

Attachment: Testcase attached.

Comment 2

[Prashant Desale]

Nominating nsbeta2 because multile site use Javascript: istead of javascript:

Comment 3

[Johnny Stenback (:jst)]

Hmm, this is interesting, mixed case javascript urls work if you type them in
the url bar, I'll have a look why they don't work in links any more (they used
to, I'm sure)...

Comment 4

[John Morrison]

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

Comment 5

[Michael La Guardia]

Need info.  What does 4.x do in this case?  What about other protocols?  (file:,
http:)

Comment 6

[Johnny Stenback (:jst)]

4.x treats the protocol in a URL caseinsensitively, so this could affect a lot
of sites out there, I think this is a problem only for javascript: urls in
mozilla.

Comment 7

[Johnny Stenback (:jst)]

Aha! Found the problem, the fix is a oneliner in the docshell, I'll attach a
patch, this should definitely be nsbeta2+

Comment 8

[Johnny Stenback (:jst)]

Attachment: Proposed fix

Comment 9

[Prashant Desale]

I also think this should be nsbeta2+ Johnny.
It could affect tons of sites out there.

COOL You Have Fix.

Comment 10

[leger]

Putting on [nsbeta2+] [will be minus on 6/15] radar.

Comment 11

[Johnny Stenback (:jst)]

I just checked in the patch, JaVaScRiPt URL's work again!

Marking FIXED.

Comment 12

[Prashant Desale]

Marking VERIFIED. 
Working perfect now. Tested with 2000-060809 on Win-95.