Pass isolate to ScriptValue constructor

Source/bindings/v8/ScriptPromise.h

Index: Source/bindings/v8/ScriptPromise.h
diff --git a/Source/bindings/v8/ScriptPromise.h b/Source/bindings/v8/ScriptPromise.h
index 02f0a227b39aa701b9e539d8c07e953d79d9e451..ff7ca63ee96bbc200628db0619d0dca276ae71e9 100644
--- a/Source/bindings/v8/ScriptPromise.h
+++ b/Source/bindings/v8/ScriptPromise.h
@@ -57,7 +57,7 @@ public:
     }
 
     explicit ScriptPromise(v8::Handle<v8::Value> promise)
-        : m_promise(promise)
+        : m_promise(promise, v8::Isolate::GetCurrent())

[haraken, 2013/09/16 16:39:53]

Would it be possible to fix caller sites of ScriptPromise() to pass Isolates, so
that we can avoid calling Isolate::GetCurrent()?

[do-not-use, 2013/09/16 20:28:05]

I was planning to do this in a follow-up patch (ditto for ScriptString) to
reduce patch size. This is a temporary situation. To my knowledge, it does not
make the situation any worse. The GetCurrent() call is merely explicit instead
of implicit.

However, I can do those in the same patch if you wish.

[do-not-use, 2013/09/17 06:52:13]

Added a FIXME comment for now.

     {
         ASSERT(!m_promise.hasNoValue());
     }