Changeset 221475 in webkit for trunk/Source/JavaScriptCore/runtime/ProxyRevoke.cpp

Timestamp:

Aug 31, 2017, 11:40:55 PM (8 years ago)

Author:

Yusuke Suzuki

Message:

[JSC] Fix "name" and "length" of Proxy revoke function
​https://bugs.webkit.org/show_bug.cgi?id=176155

Reviewed by Mark Lam.

JSTests:

• test262.yaml:

Source/JavaScriptCore:

ProxyRevoke's length should be configurable. And it does not have
its own name. We add NameVisibility enum to InternalFunction to
control visibility of the name.

• runtime/InternalFunction.cpp:

(JSC::InternalFunction::finishCreation):

• runtime/InternalFunction.h:
• runtime/ProxyRevoke.cpp:

(JSC::ProxyRevoke::finishCreation):

File:

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

trunk/Source/JavaScriptCore/runtime/ProxyRevoke.cpp

@@ -50 +50 @@
 void ProxyRevoke::finishCreation(VM& vm, const char* name, ProxyObject* proxy)
 {
-    Base::finishCreation(vm, String(name));
+    Base::finishCreation(vm, String(name), NameVisibility::Anonymous);
     m_proxy.set(vm, this, proxy);
 
-    putDirect(vm, vm.propertyNames->length, jsNumber(0), ReadOnly | DontDelete | DontEnum);
+    putDirect(vm, vm.propertyNames->length, jsNumber(0), ReadOnly | DontEnum);
 }