Closed
Bug 340036
Opened 19 years ago
Closed 18 years ago
script tag should support version 1.7
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
VERIFIED
FIXED
People
(Reporter: bc, Assigned: bc)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mrbkap
:
review+
peterv
:
review+
brendan
:
superreview+
|
Details | Diff | Splinter Review |
It should be possible to request that a script only be run if JavaScript1.7 is supported either through the language or type attributes:
<script language="javascript1.7"></script>
<script type="text/javascript;version=1.7"></script>
Assignee | ||
Comment 1•18 years ago
|
||
I think this will do it. Note that there appears to be a bug in nsJSEnvironment.cpp in that it is missing JSVERSION_1_6.
Attachment #225683 -
Flags: review?(mrbkap)
Updated•18 years ago
|
Attachment #225683 -
Flags: review?(mrbkap) → review+
Assignee | ||
Comment 2•18 years ago
|
||
Mark, this patch contains a tweak to dom/src/base/nsJSEnvironment.cpp that we might have caught if we had tested the dom agnostic branch using the js test library as well as the dom.
Assignee | ||
Comment 3•18 years ago
|
||
Comment on attachment 225683 [details] [diff] [review]
patch
Peter, Johnny: this touches a little dom stuff. Can you r/sr please so I can get this in for testing?
Attachment #225683 -
Flags: superreview?(jst)
Attachment #225683 -
Flags: review?(peterv)
Comment 4•18 years ago
|
||
Comment on attachment 225683 [details] [diff] [review]
patch
I can sr in lieu of jst, who is honeymooning.
/be
Attachment #225683 -
Flags: superreview?(jst) → superreview+
Updated•18 years ago
|
Attachment #225683 -
Flags: review?(peterv) → review+
Assignee | ||
Updated•18 years ago
|
Assignee: general → bclary
Assignee | ||
Comment 5•18 years ago
|
||
Checking in dom/src/base/nsJSEnvironment.cpp;
/cvsroot/mozilla/dom/src/base/nsJSEnvironment.cpp,v <-- nsJSEnvironment.cpp
new revision: 1.284; previous revision: 1.283
Checking in js/src/jsapi.c;
/cvsroot/mozilla/js/src/jsapi.c,v <-- jsapi.c
new revision: 3.271; previous revision: 3.270
done
Checking in js/src/jspubtd.h;
/cvsroot/mozilla/js/src/jspubtd.h,v <-- jspubtd.h
new revision: 3.60; previous revision: 3.59
done
Status: NEW → RESOLVED
Closed: 18 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Assignee | ||
Comment 6•18 years ago
|
||
I know I filed this bug, and checked it in, but I use it regularly and it works so I am verifying it to get it off my to be verified list.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•