621376 - TM: Different output with Function, prototype, encodeURI

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

for (b in [0, 0, 0, 0, 0, 0, 0, 0, 0, 0]) {
    new Function.prototype
    print(encodeURI.prototype)
}


without -j:
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined

with -j:
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
[object Object]

Comment 1

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Changeset 54759 - 2c5f8185dd2f: (without -j)
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]

Changeset 54759 - 2c5f8185dd2f: (with -j)
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]

=====

Changeset 54760 - caa8bfd6f9b1: (without -j)
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined

Changeset 54760 - caa8bfd6f9b1: (with -j)
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
undefined
[object Object]

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   54760:caa8bfd6f9b1
user:        Brendan Eich
date:        Sun Oct 03 22:46:39 2010 -0700
summary:     Per ECMA-262, no .prototype for built-in functions and Function.prototype (445319, r=Waldo).

Comment 2

[Jeff Walden [:Waldo]]

Attachment: Patch and tests

I reviewed the patch that didn't do this, so consider this penance.  :-)

Comment 4

[Brendan Eich [:brendan]]

Comment on attachment 499731 [details] [diff] [review]
Patch and tests

Thanks!

/be

Comment 5

[Jeff Walden [:Waldo]]

http://hg.mozilla.org/tracemonkey/rev/6b68235ee417

Comment 6

[Chris Leary [:cdleary] (not checking bugmail)]

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