Removing unneeded TLS fetch from v8::Null and friends

Removing unneeded TLS fetch from v8::Null and friends


BUG=
TEST=

[Dmitry Lomov, 2011-03-25 03:12:04]

Removed extra TLS feteches in v8::{Null,Undefined,True,False}. Some tummy
speedups in dom-query. 

I have inlined and removed ImplementationUtilities versions of those. Does this
look right?

[Vitaly Repeshko, 2011-03-25 13:48:03]

LGTM. I'll address my own comments and land.


Thanks,
Vitaly

http://codereview.chromium.org/6737003/diff/1/src/api.cc
File src/api.cc (left):

http://codereview.chromium.org/6737003/diff/1/src/api.cc#oldcode318
src/api.cc:318: v8::Handle<v8::Primitive> ImplementationUtilities::Undefined() {
Yes, inlining these is the right thing to do. IIRC ImplementationUtilities were
required to avoid exposing private stuff from some classes. Nothing private is
used here.

http://codereview.chromium.org/6737003/diff/1/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/6737003/diff/1/src/api.cc#newcode377
src/api.cc:377: LOG_API(isolate,  "Undefined");
This logging is useless.

http://codereview.chromium.org/6737003/diff/1/src/api.cc#newcode389
src/api.cc:389: if (!EnsureInitializedForIsolate(isolate, "v8::Null()"))
Let's make all if-s consistent by using {} when they don't fit in one line.

[Vitaly Repeshko, 2011-03-25 14:09:46]

Landed in r7374. Closing code review.


-- Vitaly