Changeset 216561 in webkit for trunk/Source/JavaScriptCore/runtime/JSBoundFunction.cpp

Timestamp:

May 9, 2017, 6:01:26 PM (8 years ago)

Author:

[email protected]

Message:

JSInjectedScriptHost should get a copy of the boundArgs
​https://bugs.webkit.org/show_bug.cgi?id=171897

Reviewed by Joseph Pecoraro.

The boundArgs array is very special - it cannot be mutated in any way. So, it makes sense
for the inspector to get a copy of it.

• inspector/JSInjectedScriptHost.cpp:

(Inspector::JSInjectedScriptHost::getInternalProperties):

• runtime/JSBoundFunction.cpp:

(JSC::JSBoundFunction::boundArgsCopy):

• runtime/JSBoundFunction.h:

(JSC::JSBoundFunction::boundArgs):

File:

• trunk/Source/JavaScriptCore/runtime/JSBoundFunction.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/JSBoundFunction.cpp

@@ -192 +192 @@
 }
 
+JSArray* JSBoundFunction::boundArgsCopy(ExecState* exec)
+{
+    VM& vm = exec->vm();
+    auto scope = DECLARE_THROW_SCOPE(vm);
+    JSArray* result = constructEmptyArray(exec, nullptr, globalObject());
+    RETURN_IF_EXCEPTION(scope, nullptr);
+    for (unsigned i = 0; i < m_boundArgs->length(); ++i) {
+        result->push(exec, m_boundArgs->getIndexQuickly(i));
+        RETURN_IF_EXCEPTION(scope, nullptr);
+    }
+    return result;
+}
+
 void JSBoundFunction::finishCreation(VM& vm, NativeExecutable* executable, int length)
 {