360500 - "... has invalid __iterator__ value ..." message is broken

Status: RESOLVED INVALID

Description

[shutdown]

$ cat invalid-iterator.txt
var exp = 'foo has invalid __iterator__ value "bar"', act;
try {
  var foo={__iterator__:'bar'}; for(var baz in foo);
} catch(e) {
  act = e.message;
}
print("expect: " + exp + "\nactual: " + act);

CVS 2006-09-18 GMT:
expect: foo has invalid __iterator__ value "bar"
actual: foo has invalid __iterator__ value "bar"

CVS 2006-09-19 GMT:
expect: foo has invalid __iterator__ value "bar"
actual: "bar" has invalid __iterator__ value "bar"

regression from bug 346642.
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=%2Fmozilla%2Fjs&sortby=Date&date=explicit&mindate=1158537600&maxdate=1158624000&cvsroot=%2Fcvsroot

Comment 1

[Tom Schuster [:evilpie]]

We just don't bother with non-object __iterator__. No error at all.