https://codereview.chromium.org/118553003/diff/1/test/mjsunit/harmony/symbols.js
File test/mjsunit/harmony/symbols.js (right):

https://codereview.chromium.org/118553003/diff/1/test/mjsunit/harmony/symbols.js#newcode75
test/mjsunit/harmony/symbols.js:75: assertSame(Function.prototype,
Symbol.prototype)
On 2014/01/09 10:35:53, rossberg wrote:
On 2014/01/08 18:13:43, sof wrote:
> On 2014/01/08 17:52:25, rossberg wrote:
> > That seems wrong. Symbol.prototype is an ordinary object.
>
>

https://people.mozilla.org/%257Ejorendorff/es6-draft.html#sec-properties-of-the-symbol-constructor
> says it's Function.prototype.

Don't confuse .prototype and .[[Prototype]] (a.k.a. __proto__). As Arv
says,
that spec section says that Symbol.__proto__ === Function.prototype.

Symbol.prototype is specified elsewhere:


http://people.mozilla.org/%7Ejorendorff/es6-draft.html#sec-properties-of-the-symbol-prototype-object

I made that only available on the "SymbolWrapper" object that Object()
instantiates.

I will need to sit down and change the encoding of Symbol to expose the
expected .prototype, updating the tests here to reflect this. Sorry
about the mess.

https://codereview.chromium.org/118553003/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to