binidngs: Make callInternalFunction return MaybeLocal

Source/bindings/core/v8/V8PerIsolateData.cpp

Index: Source/bindings/core/v8/V8PerIsolateData.cpp
diff --git a/Source/bindings/core/v8/V8PerIsolateData.cpp b/Source/bindings/core/v8/V8PerIsolateData.cpp
index 111292e5fa919c0a8b6a586e249dd3aceb3bcc17..9b6daacf9947b6333bc295e82db50066ae8fe72c 100644
--- a/Source/bindings/core/v8/V8PerIsolateData.cpp
+++ b/Source/bindings/core/v8/V8PerIsolateData.cpp
@@ -242,7 +242,10 @@ static void constructorOfToString(const v8::FunctionCallbackInfo<v8::Value>& inf
         v8SetReturnValue(info, v8::String::Empty(info.GetIsolate()));
         return;
     }
-    v8SetReturnValue(info, V8ScriptRunner::callInternalFunction(v8::Handle<v8::Function>::Cast(value), info.This(), 0, 0, info.GetIsolate()));
+    v8::Local<v8::Value> result;
+    if (!V8ScriptRunner::callInternalFunction(v8::Handle<v8::Function>::Cast(value), info.This(), 0, 0, info.GetIsolate()).ToLocal(&result))

[yurys, 2015/03/31 07:20:20]

If we fail to get a valid result there must be a pending exception which the
caller will have to handle. So I wonder what's the reason for bothering to set
any value at all? (I understand that it was written this way before your change
but since you are here it may be a good time to improve this).

[bashi, 2015/04/01 01:51:27]

Changed not to set return value on failure.

+        result = v8::String::Empty(info.GetIsolate());
+    v8SetReturnValue(info, result);
 }
 
 v8::Handle<v8::FunctionTemplate> V8PerIsolateData::toStringTemplate()