Added constructor call on object in InstantiateObject method
authordtalley <dtalley@gmail.com>
Wed, 17 Jun 2015 10:24:05 +0000 (03:24 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 17 Jun 2015 10:24:10 +0000 (10:24 +0000)
commite61a957b2a9726294cdd2802a6a2b6e3a9ef657d
tree66d44cfcaa7fe5f867b4d92b017f9bbbbd48fa83
parentbb1b54a776856280598bc5a0aa17ce2256996515
Added constructor call on object in InstantiateObject method

I found after upgrading from 4.2.2 where apinatives.js still
existed to 4.4.56 where everything had been converted to C++ in
api-natives.cc, my constructors for ObjectTemplate instantiated objects
were no longer being called.  After investigation, I noticed in
apinatives.js that a new call would handle that, but there was no
corresponding constructor call in api-natives.cc (or anywhere else
along the chain of InstantiateObject), so I added a call to
Execution::Call to actually construct the object.  Forgive me if that
isn't the right place to add it (InitializeBody in objects-inl.h also
looked like a good place), or if there's a reason constructors are
not being called.

I also added myself to the AUTHORS file in this CL.

Review URL: https://codereview.chromium.org/1137693003

Cr-Commit-Position: refs/heads/master@{#29076}
AUTHORS
src/api-natives.cc