Function::Call and Object::CallAsFunction APIs should allow v8::Value as a receiver

Function::Call and Object::CallAsFunction APIs should allow v8::Value as a receiver

Since the primitive values can be a receiver of strict mode functions in
ECMA262 5.1th, v8::Function::Call and Object::CallAsFunction should take
v8::Value as a receiver instead of v8::Object.

BUG=v8:2915
TEST=cctest/test-api
R=mstarzinger@chromium.org

Committed: http://code.google.com/p/v8/source/detail?r=17003

[Michael Starzinger, 2013-09-27 12:38:42]

The same applies to Object::CallAsFunction, I think we should fix it in one CL
to preserve consistency.

[yusukesuzuki, 2013-09-27 13:44:17]

On 2013/09/27 12:38:42, Michael Starzinger wrote:
> The same applies to Object::CallAsFunction, I think we should fix it in one CL
> to preserve consistency.

Thanks for your review.
So I've added the Object::CallAsFunction changes to this CL.

[Michael Starzinger, 2013-09-30 11:17:41]

LGTM. I'll land this once the nits in the test-case are addressed.

https://codereview.chromium.org/24920003/diff/5001/test/cctest/test-api.cc
File test/cctest/test-api.cc (right):

https://codereview.chromium.org/24920003/diff/5001/test/cctest/test-api.cc#ne...
test/cctest/test-api.cc:10342: Local<v8::Value> a1 =
instance->CallAsFunction(v8::Undefined(isolate), 0, NULL);
nit: 80 characters limit throughout this file.

[yusukesuzuki, 2013-09-30 11:32:19]

https://chromiumcodereview.appspot.com/24920003/diff/5001/test/cctest/test-ap...
File test/cctest/test-api.cc (right):

https://chromiumcodereview.appspot.com/24920003/diff/5001/test/cctest/test-ap...
test/cctest/test-api.cc:10342: Local<v8::Value> a1 =
instance->CallAsFunction(v8::Undefined(isolate), 0, NULL);
On 2013/09/30 11:17:41, Michael Starzinger wrote:
> nit: 80 characters limit throughout this file.

Done.

[Michael Starzinger, 2013-09-30 11:47:46]

Committed patchset #3 manually as r17003 (presubmit successful).