Closed Bug 407957 Opened 12 years ago Closed 12 years ago

Categories

(Core :: JavaScript Engine, defect, P1)

defect

Tracking

()

VERIFIED FIXED
mozilla1.9beta3

People

(Reporter: ria.klaassen, Assigned: brendan)

References

()

Details

(Keywords: regression)

Attachments

(1 file)

On http://www.sanitair-online.nl/ I see only the contents of one frame, the rest is missing.

In the error console I see:

Error: redeclaration of const Iterator
Source File: http://www.sanitair-online.nl/contents/core.html?lmd=39427.558333
Line: 1218

Regression window is http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=1196737140&maxdate=1196738159 which points to bug 376957.
Assignee: general → brendan
OS: Windows XP → All
Priority: -- → P1
Hardware: PC → All
Target Milestone: --- → mozilla1.9 M11
Iterator is not a type name in ES4/JS2, so is going away (see the proposal at

http://wiki.ecmascript.org/doku.php?id=proposals:iterators_and_generators

for details). It should not have been made immutable.

/be
Status: NEW → ASSIGNED
Flags: blocking1.9+
Attached patch fixSplinter Review
Sorry, should have seen this earlier. Will ask for jwalden to r+ if he is able just to get faster joy.

Any chance of M10? Just say no, I won't mind. Thought I'd ask. Zero-risk fix, unknown how many other sites use Iterator as a global variable.

/be
Attachment #292656 - Flags: review?(igor)
Attachment #292656 - Flags: approvalM10?
Attachment #292656 - Flags: review?(jwalden+bmo)
Attachment #292656 - Flags: approval1.9+
Attachment #292656 - Flags: review?(jwalden+bmo) → review+
Fixed:

js/src/jsiter.c 3.79

/be
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Verified fixed 
Status: RESOLVED → VERIFIED
Duplicate of this bug: 407987
Flags: in-testsuite?
Checking in js1_5/Regress/regress-407957.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-407957.js,v  <--  regress-407957.js
initial revision: 1.1
done
RCS file: /cvsroot/mozilla/js/tests/js1_7/regress/regress-407957.js,v
done
Checking in js1_7/regress/regress-407957.js;
/cvsroot/mozilla/js/tests/js1_7/regress/regress-407957.js,v  <--  regress-407957.js
initial revision: 1.1
done
Flags: in-testsuite?
Flags: in-testsuite+
Flags: in-litmus-
Attachment #292656 - Flags: review?(igor)
You need to log in before you can comment on or make changes to this bug.