Use v8::Isolate::GetCurrent() less in custom bindings

Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp

Index: Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp
diff --git a/Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp b/Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp
index 7559f4af5848163421ebe268621d142695a4937e..51b25c9c08150a9919d0278e80cdd6ccac219525 100644
--- a/Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp
+++ b/Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp
@@ -63,7 +63,7 @@ namespace WebCore {
 
 Node* InjectedScriptHost::scriptValueAsNode(ScriptValue value)
 {
-    v8::HandleScope scope(v8::Isolate::GetCurrent());
+    v8::HandleScope scope(value.isolate());

[marja, 2013/09/18 12:36:28]

What's this? I don't see ScriptValue::Isolate... does it do something that
requires a HandleScope?

[marja, 2013/09/18 12:37:45]

Ahh, you just landed ScriptValue::m_isolate. Hmm, is it ok to store the isolate
in every ScriptValue? At some point I was told not to do that.

[do-not-use, 2013/09/18 12:39:49]

Good question. At least no one complained yet :)
I know for example that Dictionary also stores both the v8::Value and the
Isolate.

     if (!value.isObject() || value.isNull())
         return 0;
     return V8Node::toNative(v8::Handle<v8::Object>::Cast(value.v8Value()));