Closed Bug 520892 Opened 11 years ago Closed 11 years ago

Switch application/x-javascript to application/javascript, SeaMonkey part

Categories

(SeaMonkey :: General, defect, trivial)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.0

People

(Reporter: sgautherie, Assigned: sgautherie)

References

(Blocks 1 open bug, )

Details

(Keywords: fixed-seamonkey2.0)

Attachments

(2 files)

"Found 188 matching lines in 66 files"
Flags: in-testsuite-
Used a tool to replace
|<script type="application/x-javascript"|
by
|<script type="application/javascript"|

*.xul: "Found 181 matching lines in 61 files"
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #404943 - Flags: review?(neil)
Attachment #404943 - Flags: review?(neil) → review+
Comment on attachment 404943 [details] [diff] [review]
(Av1) All .xul files
[Checkin: Comment 3]

"approval-seamonkey2.0=?":
no risk, nice little cleanup/improvement.
Attachment #404943 - Flags: superreview?(neil)
Attachment #404943 - Flags: approval-seamonkey2.0?
Attachment #404943 - Flags: superreview?(neil) → superreview+
Attachment #404943 - Flags: approval-seamonkey2.0? → approval-seamonkey2.0+
Comment on attachment 404943 [details] [diff] [review]
(Av1) All .xul files
[Checkin: Comment 3]


http://hg.mozilla.org/comm-central/rev/c235554b121d
Attachment #404943 - Attachment description: (Av1) All .xul files → (Av1) All .xul files [Checkin: Comment 3]
This leaves only
{
/suite/browser/nsBrowserStatusHandler.js
    * line 148 -- contentType == "application/x-javascript" ||
}
which I assume we'll have to keep "forever".
Attachment #405393 - Flags: superreview?(neil)
Attachment #405393 - Flags: review?(neil)
Attachment #405393 - Flags: approval-seamonkey2.0?
Comment on attachment 405393 [details] [diff] [review]
(Bv1) All .[x]html files
[Checkin: Comment 6]

Yeah, that last one is used to disable things like view - character set for images.
Attachment #405393 - Flags: superreview?(neil)
Attachment #405393 - Flags: superreview+
Attachment #405393 - Flags: review?(neil)
Attachment #405393 - Flags: review+
Attachment #405393 - Flags: approval-seamonkey2.0? → approval-seamonkey2.0+
Comment on attachment 405393 [details] [diff] [review]
(Bv1) All .[x]html files
[Checkin: Comment 6]


http://hg.mozilla.org/comm-central/rev/5331895338a1
Attachment #405393 - Attachment description: (Bv1) All .[x]html files → (Bv1) All .[x]html files [Checkin: Comment 6]
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.0
You need to log in before you can comment on or make changes to this bug.