826166 - Remove an ability to set higher JS versions via language attribute

Status: RESOLVED FIXED

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

Description

[Masatoshi Kimura [:emk]]

The language attribute is obsolete in HTML5 and "JavaScript" is the only speced value. It is ignored when the type attribute is present.
http://www.whatwg.org/specs/web-apps/current-work/multipage/obsolete.html#obsolete-but-conforming-features
IE and Opera doesn't support language="JavaScript1.6" to  language="JavaScript1.8". Chrome recognize the language="JavaScript1.6" and language="JavaScript1.7", but no language extensions are enabled.
XUL doesn't support the language attribute, so it should have no add-on impact.
No in-tree consumers use the language attribute to enable language extensions.

Comment 1

[Masatoshi Kimura [:emk]]

Attachment: patch

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Get rid of the outparam, please.

Comment 3

[Masatoshi Kimura [:emk]]

Attachment: patch v2

Removed the useless outparam.
According to the comment, the language attribute already lost the ability in HTML.

> XUL doesn't support the language attribute
And it looks like it was wrong.

Comment 4

[Masatoshi Kimura [:emk]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fcb1d5ec2d60

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fcb1d5ec2d60

Comment 6

[Kohei Yoshino [:kohei]]

I've added this bug to the compatibility doc. Please correct the info if I'm wrong.
https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_21

Comment 7

[Masatoshi Kimura [:emk]]

(In reply to Kohei Yoshino from comment #6)
> I've added this bug to the compatibility doc. Please correct the info if I'm
> wrong.
> https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_21

The language attribute in HTML used to ignore the JS version before this bug. This bug is just a cleanup for Web pages.
On the other hand, XUL language attribute has changed the behavior. Therefore, this is a add-on compatibility bug (not site compatibility).

Comment 8

[Kohei Yoshino [:kohei]]

Oh, Okay. Removed this bug from the document.