Remove calls to HandleScope default ctor.

Source/bindings/v8/V8Binding.cpp

Index: Source/bindings/v8/V8Binding.cpp
diff --git a/Source/bindings/v8/V8Binding.cpp b/Source/bindings/v8/V8Binding.cpp
index 5abfe4412209609e9b3989f080d26744e75160dd..812fb8289ae97fc5afa276487fb24026ac575441 100644
--- a/Source/bindings/v8/V8Binding.cpp
+++ b/Source/bindings/v8/V8Binding.cpp
@@ -590,4 +590,15 @@ v8::Local<v8::Value> getHiddenValueFromMainWorldWrapper(v8::Isolate* isolate, Sc
     return wrapper.IsEmpty() ? v8::Local<v8::Value>() : wrapper->GetHiddenValue(key);
 }
 
+v8::Isolate* getIsolateFromScriptExecutionContext(ScriptExecutionContext* context)

[haraken, 2013/09/02 08:05:56]

The implementation of this method looks hacky. A better way to retrieve an
Isolate would be:

a) Get a V8 context from ScriptExecutionContext. Then, get an Isolate from the
V8 context. (This is the common way we currently retrieve the isolate from
ScriptExecutionContext.)

b) Cache an isolate on ScriptExecutionContext so that we can retrieve the
isolate using context->isolate().

FIXME is fine for now, but please consider implementing a) or b) in a follow-up
CL.

[marja, 2013/09/02 09:05:42]

Added a FIXME to do b).

+{
+    if (context->isDocument()) {

[haraken, 2013/09/02 08:05:56]

Nit: You might want to add ASSERT(context).

[marja, 2013/09/02 09:05:42]

Done.

+        static v8::Isolate* mainWorldIsolate = 0;
+        if (!mainWorldIsolate)
+            mainWorldIsolate = v8::Isolate::GetCurrent();
+        return mainWorldIsolate;
+    }
+    return v8::Isolate::GetCurrent();
+}
+
 } // namespace WebCore