Return v8::True or v8::False instead of using Boolean::New.

Return v8::True or v8::False instead of using Boolean::New.

BUG=None
TEST=None

R=mnaganov@chromium.org

Signed-off-by: Thiago Farina <tfarina@chromium.org>;

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

[mnaganov (inactive), 2011-08-30 10:36:59]

LGTM, with one comment.

What do you think about this? You are not going to remove v8::Boolean::New,
right?

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

http://codereview.chromium.org/7785001/diff/1/test/cctest/test-api.cc#newcode...
test/cctest/test-api.cc:9570: return args.IsConstructCall() ? v8::True() :
v8::False();
I believe, return v8::Boolean::New(args.IsConstructCall()) is more concise, and
this is the case where v8::Boolean::New is useful.

[tfarina, 2011-08-30 18:28:23]

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

http://codereview.chromium.org/7785001/diff/1/test/cctest/test-api.cc#newcode...
test/cctest/test-api.cc:9570: return args.IsConstructCall() ? v8::True() :
v8::False();
On 2011/08/30 10:37:00, Mikhail Naganov (Chromium) wrote:
> I believe, return v8::Boolean::New(args.IsConstructCall()) is more concise,
and
> this is the case where v8::Boolean::New is useful.

Done.