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

Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp

 /*
  * Copyright (C) 2007-2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 45 matching lines @@
 #include "core/inspector/InjectedScript.h"
 #include "core/inspector/InjectedScriptHost.h"
 #include "core/inspector/InspectorDOMAgent.h"
 #include "core/platform/JSONValues.h"
 #include "modules/webdatabase/Database.h"
 
 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()));
 }
 
 ScriptValue InjectedScriptHost::nodeAsScriptValue(ScriptState* state, Node* node)
 {
     v8::Isolate* isolate = state->isolate();
     v8::HandleScope scope(isolate);
     v8::Local<v8::Context> context = state->context();
@@ skipping 351 matching lines @@
     int lineNumber;
     int columnNumber;
     if (!getFunctionLocation(args, &scriptId, &lineNumber, &columnNumber))
         return;
 
     InjectedScriptHost* host = V8InjectedScriptHost::toNative(args.Holder());
     host->unmonitorFunction(scriptId, lineNumber, columnNumber);
 }
 
 } // namespace WebCore