Removing TLS fetches on v8::String API

Removing TLS fetches on v8::String API
BUG=
TEST=

[Dmitry Lomov, 2011-03-21 19:10:46]

Vitaly,

could you have a look and see if it is the right way to do it?

This is just a quick hack to get isolate from v8::String instance and not from
TLS.

This quick change gets us 10% speed-up on getElementsByName in
dromaeo.com/?dom-query.
If this looks good, I'll go ahead and update other v8::String APIs.

[Vitaly Repeshko, 2011-03-21 19:26:47]

LGTM. Yes, this seems to be right approach.

I'll land


Thanks,
Vitaly

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

http://codereview.chromium.org/6716004/diff/1/src/api.cc#newcode4054
src/api.cc:4054: i::Isolate* isolate = (*obj)->GetIsolate();
obj->GetIsolate() should also work.

[Dmitry Lomov, 2011-03-21 20:20:25]

More places

[Vitaly Repeshko, 2011-03-22 14:48:52]

LGTM! I'll land.


Thanks,
Vitaly

http://codereview.chromium.org/6716004/diff/4001/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/6716004/diff/4001/src/api.cc#newcode3919
src/api.cc:3919: EnsureInitializedForIsolate(isolate, "v8::String::New()");
The string tags are wrong here.