Remove constructor from remaining functions that aren't constructors

third_party/WebKit/Source/bindings/core/v8/V8ScriptRunner.cpp

Index: third_party/WebKit/Source/bindings/core/v8/V8ScriptRunner.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/V8ScriptRunner.cpp b/third_party/WebKit/Source/bindings/core/v8/V8ScriptRunner.cpp
index ba9345ecfeecdf0b2ea6cdb8ca787b5080aa77ae..f85a0459eb6651996889c1593864dcc13f72e870 100644
--- a/third_party/WebKit/Source/bindings/core/v8/V8ScriptRunner.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/V8ScriptRunner.cpp
@@ -123,7 +123,7 @@ v8::Local<v8::Value> throwStackOverflowExceptionIfNeeded(v8::Isolate* isolate)
     }
     v8::MicrotasksScope microtasksScope(isolate, v8::MicrotasksScope::kDoNotRunMicrotasks);
     V8PerIsolateData::from(isolate)->setIsHandlingRecursionLevelError(true);
-    v8::Local<v8::Value> result = v8::Function::New(isolate, throwStackOverflowException)->Call(v8::Undefined(isolate), 0, 0);
+    v8::Local<v8::Value> result = v8::Function::New(isolate->GetCurrentContext(), throwStackOverflowException, v8::Local<v8::Value>(), 0, v8::ConstructorBehavior::kThrow).ToLocalChecked()->Call(v8::Undefined(isolate), 0, 0);

[haraken, 2016/07/07 11:14:09]

Yeah, in reality we want to just crash when V8 API returns an empty Maybe handle
where we don't expect it.

[jochen (gone - plz use gerrit), 2016/07/07 11:25:11]

well, I'm just changing one thing in this CL, and the ->Call() previously also
just crashed on an empty handle.

     V8PerIsolateData::from(isolate)->setIsHandlingRecursionLevelError(false);
     return result;
 }