bindings: Use Maybe version of Call()

Source/bindings/core/v8/NPV8Object.cpp

 /*
  * Copyright (C) 2004, 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) 2007, 2008, 2009 Google, Inc.  All rights reserved.
  * Copyright (C) 2014 Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 255 matching lines @@
         VOID_TO_NPVARIANT(*result);
         return false;
     }
 
     LocalFrame* frame = v8NpObject->rootObject->frame();
     ASSERT(frame);
 
     // Call the function object.
     v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(functionObject);
     OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(isolate, arguments, argumentCount, npObject);
-    v8::Local<v8::Value> resultObject = frame->script().callFunction(function, v8Object, argumentCount, argv.get());
-
     // If we had an error, return false.  The spec is a little unclear here, but says "Returns true if the method was
     // successfully invoked".  If we get an error return value, was that successfully invoked?
-    if (resultObject.IsEmpty())
+    v8::Local<v8::Value> resultObject;
+    if (!frame->script().callFunction(function, v8Object, argumentCount, argv.get()).ToLocal(&resultObject))
         return false;
 
     convertV8ObjectToNPVariant(isolate, resultObject, npObject, result);
     return true;
 }
 
 // FIXME: Fix it same as _NPN_Invoke (HandleScope and such).
 bool _NPN_InvokeDefault(NPP npp, NPObject* npObject, const NPVariant* arguments, uint32_t argumentCount, NPVariant* result)
 {
     if (!npObject)
@@ skipping 17 matching lines @@
         return false;
 
     ScriptState::Scope scope(scriptState);
     ExceptionCatcher exceptionCatcher;
 
     // Lookup the function object and call it.
     v8::Local<v8::Object> functionObject = v8::Local<v8::Object>::New(isolate, v8NpObject->v8Object);
     if (!functionObject->IsFunction())
         return false;
 
-    v8::Local<v8::Value> resultObject;
     v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(functionObject);
-    if (!function->IsNull()) {
-        LocalFrame* frame = v8NpObject->rootObject->frame();
-        ASSERT(frame);
+    if (function->IsNull())
+        return false;
 
-        OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(isolate, arguments, argumentCount, npObject);
-        resultObject = frame->script().callFunction(function, functionObject, argumentCount, argv.get());
-    }
+    LocalFrame* frame = v8NpObject->rootObject->frame();
+    ASSERT(frame);
+
+    OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(isolate, arguments, argumentCount, npObject);
     // If we had an error, return false.  The spec is a little unclear here, but says "Returns true if the method was
     // successfully invoked".  If we get an error return value, was that successfully invoked?
-    if (resultObject.IsEmpty())
+    v8::Local<v8::Value> resultObject;
+    if (!frame->script().callFunction(function, functionObject, argumentCount, argv.get()).ToLocal(&resultObject))
         return false;
 
     convertV8ObjectToNPVariant(isolate, resultObject, npObject, result);
     return true;
 }
 
 bool _NPN_Evaluate(NPP npp, NPObject* npObject, NPString* npScript, NPVariant* result)
 {
     // FIXME: Give the embedder a way to control this.
     bool popupsAllowed = false;
@@ skipping 275 matching lines @@
 
         convertV8ObjectToNPVariant(scriptState->isolate(), resultObject, npObject, result);
         return true;
     }
 
     if (NP_CLASS_STRUCT_VERSION_HAS_CTOR(npObject->_class) && npObject->_class->construct)
         return npObject->_class->construct(npObject, arguments, argumentCount, result);
 
     return false;
 }