[webkit-changes] cvs commit: JavaScriptCore/kjs nodes.cpp

[Eric]

eseidel     05/09/27 16:38:22

  Modified:    .        ChangeLog
               kjs      nodes.cpp
  Log:
  Bug #: 3537
  Submitted by: eseidel
  Reviewed by: mjs
          * kjs/nodes.cpp:
          (FuncExprNode::evaluate): Now sets .constructor properly.
          Test cases added to WebCore/layout-tests.
          http://bugzilla.opendarwin.org/show_bug.cgi?id=3537
  
  Revision  Changes    Path
  1.839     +9 -0      JavaScriptCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/JavaScriptCore/ChangeLog,v
  retrieving revision 1.838
  retrieving revision 1.839
  diff -u -r1.838 -r1.839
  --- ChangeLog	27 Sep 2005 22:36:45 -0000	1.838
  +++ ChangeLog	27 Sep 2005 23:38:21 -0000	1.839
  @@ -1,3 +1,12 @@
  +2005-09-27  Eric Seidel  <eseidel at apple.com>
  +
  +        Reviewed by mjs.
  +
  +        * kjs/nodes.cpp:
  +        (FuncExprNode::evaluate): Now sets .constructor properly.
  +        Test cases added to WebCore/layout-tests.
  +        http://bugzilla.opendarwin.org/show_bug.cgi?id=3537
  +
   2005-09-26  Maciej Stachowiak  <mjs at apple.com>
   
           Reviewed by John.
  
  
  
  1.83      +2 -1      JavaScriptCore/kjs/nodes.cpp
  
  Index: nodes.cpp
  ===================================================================
  RCS file: /cvs/root/JavaScriptCore/kjs/nodes.cpp,v
  retrieving revision 1.82
  retrieving revision 1.83
  diff -u -r1.82 -r1.83
  --- nodes.cpp	24 Sep 2005 22:34:13 -0000	1.82
  +++ nodes.cpp	27 Sep 2005 23:38:22 -0000	1.83
  @@ -2345,7 +2345,8 @@
   
     FunctionImp *fimp = new DeclaredFunctionImp(exec, ident, body.get(), context->scopeChain());
     ValueImp *ret(fimp);
  -  ValueImp *proto = exec->lexicalInterpreter()->builtinObject()->construct(exec, List::empty());
  +  ObjectImp *proto = exec->lexicalInterpreter()->builtinObject()->construct(exec, List::empty());
  +  proto->put(exec, constructorPropertyName, ret, ReadOnly|DontDelete|DontEnum);
     fimp->put(exec, prototypePropertyName, proto, Internal|DontDelete);
   
     int plen = 0;