Implement V8ScriptRunner::callFunction()

Source/bindings/v8/V8ScriptRunner.cpp

 /*
  * Copyright (C) 2009 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
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 92 matching lines @@
 }
 
 v8::Local<v8::Value> V8ScriptRunner::compileAndRunInternalScript(v8::Handle<v8::String> source, v8::Isolate* isolate, v8::Local<v8::Context> context, const String& fileName, const TextPosition& scriptStartPosition, v8::ScriptData* scriptData)
 {
     TRACE_EVENT0("v8", "v8.run");
     v8::Local<v8::Value> result;
     if (context.IsEmpty())
         context = v8::Context::New(isolate);
     if (context.IsEmpty())
         return result;
-    {
-        v8::Context::Scope scope(context);
-        v8::Handle<v8::Script> script = V8ScriptRunner::compileScript(source, fileName, scriptStartPosition, scriptData, isolate);
-        if (script.IsEmpty())
-            return result;
 
-        V8RecursionScope::MicrotaskSuppression recursionScope;
-        result = script->Run();
-    }
+    v8::Context::Scope scope(context);
+    v8::Handle<v8::Script> script = V8ScriptRunner::compileScript(source, fileName, scriptStartPosition, scriptData, isolate);
+    if (script.IsEmpty())
+        return result;
+
+    V8RecursionScope::MicrotaskSuppression recursionScope;
+    result = script->Run();
     return result;
 }
 
+static String functionInfo(const v8::Handle<v8::Function> function)
+{
+    String resourceName = "undefined";
+    int lineNumber = 1;
+    v8::ScriptOrigin origin = function->GetScriptOrigin();
+    if (!origin.ResourceName().IsEmpty()) {
+        resourceName = toWebCoreString(origin.ResourceName());
+        lineNumber = function->GetScriptLineNumber() + 1;
+    }
+
+    StringBuilder builder;
+    builder.append(resourceName);
+    builder.append(':');
+    builder.appendNumber(lineNumber);
+    return builder.toString();
+}
+
+v8::Local<v8::Value> V8ScriptRunner::callFunction(v8::Handle<v8::Function> function, ScriptExecutionContext* context, v8::Handle<v8::Object> receiver, int argc, v8::Handle<v8::Value> args[])
+{
+    TRACE_EVENT0("v8", "v8.callFunction");

[haraken, 2013/05/24 04:22:32]

Previously this was:

  TRACE_EVENT1("v8", "v8.callFunction", "callsite",
resourceString(function).utf8());

I think it's OK to remove the argument for now, since it is not used in
about:tracing. (c.f. we're not adding the argument in TRACE_EVENT macros for
Script::Run().)

+    V8GCController::checkMemoryUsage();
+
+    if (V8RecursionScope::recursionLevel() >= kMaxRecursionDepth)
+        return handleMaxRecursionDepthExceeded();
+
+    V8RecursionScope recursionScope(context);
+    v8::Local<v8::Value> result = function->Call(receiver, argc, args);
+
+    crashIfV8IsDead();
+    return result;
+}
+
 } // namespace WebCore