binidngs: Make callInternalFunction return MaybeLocal

Source/bindings/core/v8/ScriptRegexp.cpp

 /*
  * Copyright (C) 2004, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Collabora Ltd.
  * Copyright (C) 2011 Peter Varga (pvarga@webkit.org), University of Szeged
  * Copyright (C) 2013 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:
  * 1. Redistributions of source code must retain the above copyright
@@ skipping 62 matching lines @@
     v8::HandleScope handleScope(isolate);
     v8::Local<v8::Context> context = V8PerIsolateData::from(isolate)->ensureScriptRegexpContext();
     v8::Context::Scope contextScope(context);
     v8::TryCatch tryCatch;
 
     v8::Local<v8::RegExp> regex = m_regex.newLocal(isolate);
     v8::Local<v8::Value> exec;
     if (!regex->Get(context, v8AtomicString(isolate, "exec")).ToLocal(&exec))
         return -1;
     v8::Local<v8::Value> argv[] = { v8String(isolate, string.substring(startFrom)) };
-    v8::Local<v8::Value> returnValue = V8ScriptRunner::callInternalFunction(exec.As<v8::Function>(), regex, WTF_ARRAY_LENGTH(argv), argv, isolate);
-
-    if (tryCatch.HasCaught())
+    v8::Local<v8::Value> returnValue;
+    if (!V8ScriptRunner::callInternalFunction(exec.As<v8::Function>(), regex, WTF_ARRAY_LENGTH(argv), argv, isolate).ToLocal(&returnValue))
         return -1;
 
     // RegExp#exec returns null if there's no match, otherwise it returns an
     // Array of strings with the first being the whole match string and others
     // being subgroups. The Array also has some random properties tacked on like
     // "index" which is the offset of the match.
     //
     // https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/RegExp/exec
 
     ASSERT(!returnValue.IsEmpty());
     if (!returnValue->IsArray())
         return -1;
 
     v8::Local<v8::Array> result = returnValue.As<v8::Array>();
     v8::Local<v8::Value> matchOffset;
     if (!result->Get(context, v8AtomicString(isolate, "index")).ToLocal(&matchOffset))
         return -1;
     if (matchLength) {
         v8::Local<v8::Value> match;
         if (!result->Get(context, 0).ToLocal(&match))
             return -1;
         *matchLength = match.As<v8::String>()->Length();
     }
 
     return matchOffset.As<v8::Int32>()->Value() + startFrom;
 }
 
 } // namespace blink