Expose hasOwnProperty() through API.

Expose hasOwnProperty() through API.

BUG=http://code.google.com/p/v8/issues/detail?id=1342
TEST=test-api/HasOwnProperty

[Vitaly Repeshko, 2011-04-27 09:03:59]

Drive by comment.

http://codereview.chromium.org/6903059/diff/5/test/cctest/test-api.cc
File test/cctest/test-api.cc (right):

http://codereview.chromium.org/6903059/diff/5/test/cctest/test-api.cc#newcode214
test/cctest/test-api.cc:214: Handle<Object> object = value->ToObject();
It'd be interesting to test this with __defineGetter__, callbacks, and
interceptors.

[Mads Ager (chromium), 2011-04-27 09:26:32]

I agree with Vitaly's comment. More testing is needed. The change itself looks
good.

[Karl Klose, 2011-04-28 08:32:43]

I added more testing, please have another look.

http://codereview.chromium.org/6903059/diff/5/test/cctest/test-api.cc
File test/cctest/test-api.cc (right):

http://codereview.chromium.org/6903059/diff/5/test/cctest/test-api.cc#newcode214
test/cctest/test-api.cc:214: Handle<Object> object = value->ToObject();
On 2011/04/27 09:03:59, Vitaly Repeshko wrote:
> It'd be interesting to test this with __defineGetter__, callbacks, and
> interceptors.

Done.

[Mads Ager (chromium), 2011-04-28 08:38:27]

LGTM

http://codereview.chromium.org/6903059/diff/1004/test/cctest/test-api.cc
File test/cctest/test-api.cc (right):

http://codereview.chromium.org/6903059/diff/1004/test/cctest/test-api.cc#newc...
test/cctest/test-api.cc:13990: templ->SetNamedPropertyHandler(0, 0,
HasOwnPropertyNamedPropertyQuery);
Maybe add a case with interceptor getters and interceptor query that disagree on
which properties are there and check that the query method wins in connection
with HasOwnProperty.