Changeset 19203 in webkit for trunk/JavaScriptCore

Timestamp:

Jan 28, 2007, 6:30:42 PM (18 years ago)

Author:

ddkilzer

Message:

JavaScriptCore:

Reviewed by Darin.

• fix ​http://bugs.webkit.org/show_bug.cgi?id=9815 JavaScript TypeError loading Dean Edwards' JS compressor/obfuscator

Creating a function using 'new Function()' was not setting its prototype with the
same flags as 'function() { }'.

Test: fast/js/function-prototype.html

• kjs/function_object.cpp: (FunctionObjectImp::construct): Change flags from DontEnum|DontDelete|ReadOnly to Internal|DontDelete to match FuncDeclNode::processFuncDecl() and FuncExprNode::evaluate() in kjs/nodes.cpp.

LayoutTests:

Reviewed by Darin.

• Test for ​http://bugs.webkit.org/show_bug.cgi?id=9815 JavaScript TypeError loading Dean Edwards' JS compressor/obfuscator

• fast/js/function-prototype-expected.txt: Added.
• fast/js/function-prototype.html: Added.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/function_object.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2007-01-28  David Kilzer  <[email protected]>
+
+        Reviewed by Darin.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=9815
+          JavaScript TypeError loading Dean Edwards' JS compressor/obfuscator
+
+        Creating a function using 'new Function()' was not setting its prototype with the
+        same flags as 'function() { }'.
+
+        Test: fast/js/function-prototype.html
+
+        * kjs/function_object.cpp:
+        (FunctionObjectImp::construct): Change flags from DontEnum|DontDelete|ReadOnly to
+        Internal|DontDelete to match FuncDeclNode::processFuncDecl() and
+        FuncExprNode::evaluate() in kjs/nodes.cpp.
+
 2007-01-27  Geoffrey Garen  <[email protected]>
 

trunk/JavaScriptCore/kjs/function_object.cpp

@@ -246 +246 @@
   JSObject *prototype = objCons->construct(exec,List::empty());
   prototype->put(exec, constructorPropertyName, fimp, DontEnum|DontDelete|ReadOnly);
-  fimp->put(exec, prototypePropertyName, prototype, DontEnum|DontDelete|ReadOnly);
+  fimp->put(exec, prototypePropertyName, prototype, Internal|DontDelete);
   return fimp;
 }