407957 - Empty frames on http://www.sanitair-online.nl/

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect, P1)

Description

[Ria Klaassen (not reading all bugmail)]

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.

Comment 1

[Brendan Eich [:brendan]]

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

Comment 2

[Brendan Eich [:brendan]]

Attachment: fix

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

Comment 3

[Brendan Eich [:brendan]]

Fixed:

js/src/jsiter.c 3.79

/be

Comment 4

[Ria Klaassen (not reading all bugmail)]

Verified fixed 

Comment 6

[Bob Clary [:bc] (inactive)]

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