455762 - Remove the extraneous |class| attribute in |<script class="testbody" type="text/javascript">| of test.template.txt and xhtml.template.txt

Status: RESOLVED WONTFIX

(Testing :: Mochitest, defect)

Description

[Serge Gautherie (:sgautherie)]

And from all the 600- tests which have copied it ;-<

IIRC, Robert said he agreed.

***

While at it, replace these "text/javascript" by "application/javascript".
Some script already have that.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <*.template.txt> [Checkin: Comment 2]

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 339155 [details] [diff] [review]
(Av1) <*.template.txt>
[Checkin: Comment 2]

http://hg.mozilla.org/mozilla-central/rev/8b1da13a315e

Comment 3

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) Apply (x)html fixes

This patch size is 2 MB...
I used a "find and replace" utility, searching for whole lines.

Does this need sr ? (Who from ?)

Comment 4

[Robert Sayre]

Comment on attachment 339166 [details]
(Bv1) Apply (x)html fixes

This doesn't require SR, but it better land when the tree is perfectly green, and be tested ahead of time.

Comment 5

[Aiko]

At least some of the Mochitests can also be run under IE, which is nice to compare results. This change of the script type attribute would make that harder.

Comment 6

[Robert Sayre]

Comment on attachment 339166 [details]
(Bv1) Apply (x)html fixes

OK, let's not change the type. Sorry about that.

Comment 7

[Serge Gautherie (:sgautherie)]

(In reply to comment #5)
> At least some of the Mochitests can also be run under IE, which is nice to
> compare results. This change of the script type attribute would make that
> harder.

Could you tell me a little more about this:
How can our (automated) tests be run in MsIE ?
What brakes them in my patch ?
Should we rather "fix" some other tests in the opposite way ?
...

Comment 8

[Martijn Wargers (dead)]

Apparently, MsIE doesn't support the type=application/javascript.
You can run the tests in IE, while the server is up and running, so while running (or waiting to run) tests in Firefox.

Comment 9

[Andrew Halberstadt [:ahal]]

Mass closing mochitest bugs that haven't had activity in the past 5 years. Please re-open or file a new bug with modern context if this is still relevant.