https://codereview.chromium.org/475423003/diff/100001/src/objects.cc
File src/objects.cc (right):
https://codereview.chromium.org/475423003/diff/100001/src/objects.cc#newcode9412
src/objects.cc:9412: clone->set_properties(
For F.p.toMethod we should not copy the properties. However if we use
private own property for the [[HomeObject]] we need to at least handle
that.
https://codereview.chromium.org/475423003/diff/100001/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/475423003/diff/100001/src/runtime.cc#newcode2058
src/runtime.cc:2058: Handle<JSFunction> clone = JSFunction::Copy(fun);
This is going to copy the properties which is not right.
https://codereview.chromium.org/475423003/diff/100001/test/mjsunit/harmony/toMethod.js
File test/mjsunit/harmony/toMethod.js (right):
https://codereview.chromium.org/475423003/diff/100001/test/mjsunit/harmony/toMethod.js#newcode1
test/mjsunit/harmony/toMethod.js:1: // Copyright 2013 the V8 project
authors. All rights reserved.
Use short copyright header with the year 2014
https://codereview.chromium.org/475423003/diff/100001/test/mjsunit/harmony/toMethod.js#newcode132
test/mjsunit/harmony/toMethod.js:132: (function TestPreserveProperties()
{
This test is not right.
https://codereview.chromium.org/475423003/
--
--
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/d/optout.
