450275 - TM: js1_5/extensions/regress-434837-01.js FAIL

Status: VERIFIED WONTFIX

(Core :: JavaScript Engine, defect, P3)

Description

[Bob Clary [:bc] (inactive)]

[1].__count__ reason: Expected value '1', Actual value '3'

[1].__count__ reason: Expected value '9', Actual value '3'

[].__count__ reason: Expected value '0', Actual value '3'

x.test1 reason: Expected value 'true', Actual value 'false'

x.test2 reason: Expected value 'false', Actual value 'true'

x.test3 (1) reason: Expected value 'true', Actual value 'false'

y.__count__ reason: Expected value '4', Actual value '3'

tracemonkey: 20f1f3c72ae7 tip

Comment 1

[Bob Clary [:bc] (inactive)]

occurs on mozilla-central now.

Comment 2

[Robert Sayre]

Still happening?

Comment 3

[Brian Crowder]

This is going to suck for the same reason that we suck at __proto__ lookup, as I discuss in bug 450274, comment 4.  WONTFIX, too?

sayrer:  We can drop the priority on this one as well, likely.

Comment 4

[Brendan Eich [:brendan]]

WONTFIX and remove __count__ in mozilla2.

/be

Comment 5

[Bob Clary [:bc] (inactive)]

exclude it.

/cvsroot/mozilla/js/tests/spidermonkey-n-1.9.1.tests,v  <--  spidermonkey-n-1.9.1.tests
new revision: 1.2; previous revision: 1.1

http://hg.mozilla.org/mozilla-central/rev/6ee837a12440

Comment 6

[Bob Clary [:bc] (inactive)]

js1_5/extensions/regress-434837-01.js should have been excluded for 1.9.1, 1.9.2. I mistakenly also excluded it for 1.9.0 as well.

brendan, i excluded the entire test, but it appears that was overly aggressive. Are the failures of test1, test2, test3 above wontfix on 1.9.1 and later ?

Comment 7

[Brendan Eich [:brendan]]

All __count__ stuff is wontfix.

Are the test[123] failures covered by another bug? If not, please file. We want these tests split out.

/be

Comment 8

[Bob Clary [:bc] (inactive)]

test and exclusion lists modified in mc http://hg.mozilla.org/mozilla-central/rev/63ddf84277ae and cvs.

filed Bug 474402

Comment 9

[Eric Shepherd [:sheppy]]

What needs documenting here?

Comment 10

[Eric Shepherd [:sheppy]]

There is no obvious need for documentation here, and no response to my request for details a month ago, so I'm removing the doc needed keyword.