Add Isolate* parameter to static API methods that don't take one.

Add Isolate* parameter to static API methods that don't take one.

We cannot yet deprecate the versions that don't take one, because
there's too much test code in v8 using them. To keep this CL small and
managable, I'll first introduce the new APIs.

BUG=none
R=dcarney@chromium.org, svenpanne@chromium.org
LOG=n

Committed: https://code.google.com/p/v8/source/detail?r=18068

[dcarney, 2013-11-25 15:35:52]

lgtm

https://codereview.chromium.org/84833006/diff/1/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/84833006/diff/1/include/v8.h#newcode1141
include/v8.h:1141: // Will be deprecated soon.
maybe move to isolate

https://codereview.chromium.org/84833006/diff/1/include/v8.h#newcode2951
include/v8.h:2951: // Will be deprecated soon.
maybe move to isolate

[Sven Panne, 2013-11-26 08:57:39]

LGTM with a nit. Furthermore (as discussed offline), we might consider changing
all those

   class Foo {
      static Local<Foo> New(Isolate*, ...);
   }

to

   class Isolate {
      Local<Foo> NewFoo(...);
   }

This has pros and cons, but at least keep the possibility in mind.

https://codereview.chromium.org/84833006/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/84833006/diff/40001/include/v8.h#newcode1997
include/v8.h:1997: static Local<Integer> New(int32_t value, Isolate*);
For consistency reasons, the Isolate parameter should come first. Same below.

[dcarney, 2013-11-26 09:35:54]

On 2013/11/26 08:57:39, Sven Panne wrote:
> LGTM with a nit. Furthermore (as discussed offline), we might consider
changing
> all those
> 
>    class Foo {
>       static Local<Foo> New(Isolate*, ...);
>    }
> 
> to
> 
>    class Isolate {
>       Local<Foo> NewFoo(...);
>    }
> 
> This has pros and cons, but at least keep the possibility in mind.

I'm not personally a huge fan of that api change, but pretty much everything
else that needs a isolate parameter should be on isolate for sure

[jochen (gone - plz use gerrit), 2013-11-26 09:45:24]

Committed patchset #3 manually as r18068.