Changeset 21027 in webkit for trunk/JavaScriptCore/kjs/nodes.h

Timestamp:

Apr 23, 2007, 1:38:46 AM (18 years ago)

Author:

darin

Message:

JavaScriptCore:

Reviewed by Maciej.

• fix <rdar://problem/4840688> REGRESSION (r10588, r10621): JavaScript won't parse modifications of non-references (breaks 300themovie.warnerbros.com, fedex.com)

Despite the ECMAScript specification's claim that you can treat these as syntax
errors, doing so creates some website incompatibilities. So this patch turns them back
into evaluation errors instead.

Test: fast/js/modify-non-references.html

• kjs/grammar.y: Change makeAssignNode, makePrefixNode, and makePostfixNode so that they never fail to parse. Update rules that use them. Fix a little bit of indenting. Use new PostfixErrorNode, PrefixErrorNode, and AssignErrorNode classes.

• kjs/nodes.h: Added an overload of throwError that takes a char* argument. Replaced setExceptionDetailsIfNeeded and debugExceptionIfNeeded with handleException, which does both. Added PostfixErrorNode, PrefixErrorNode, and AssignErrorNode classes.

• kjs/nodes.cpp: Changed exception macros to use handleException; simpler and smaller code size than the two functions that we used before. (Node::throwError): Added the overload mentioned above. (Node::handleException): Added. Contains the code from both setExceptionDetailsIfNeeded and debugExceptionIfNeeded. (PostfixErrorNode::evaluate): Added. Throws an exception. (PrefixErrorNode::evaluate): Ditto. (AssignErrorNode::evaluate): Ditto. (ThrowNode::execute): Call handleException instead of debugExceptionIfNeeded; this effectively adds a call to setExceptionDetailsIfNeeded, which may help with getting the correct file and line number for these exceptions.

• kjs/nodes2string.cpp: (PostfixErrorNode::streamTo): Added. (PrefixErrorNode::streamTo): Added. (AssignErrorNode::streamTo): Added.

LayoutTests:

Reviewed by Maciej.

• test for <rdar://problem/4840688> REGRESSION (r10588, r10621): JavaScript won't parse modifications of non-references (breaks 300themovie.warnerbros.com, fedex.com)

• fast/js/modify-non-references-expected.txt: Added.
• fast/js/modify-non-references.html: Added.
• fast/js/resources/modify-non-references.js: Added.

• fast/js/assign-expected.txt: Updated for different exception text.
• fast/js/postfix-syntax-expected.txt: Ditto.
• fast/js/prefix-syntax-expected.txt: Ditto.

File:

• trunk/JavaScriptCore/kjs/nodes.h (modified) (6 diffs)

trunk/JavaScriptCore/kjs/nodes.h

@@ -4 +4 @@
  *  Copyright (C) 1999-2000 Harri Porten ([email protected])
  *  Copyright (C) 2001 Peter Kelly ([email protected])
- *  Copyright (C) 2003, 2004, 2005, 2006 Apple Computer, Inc.
+ *  Copyright (C) 2003, 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
  *
  *  This library is free software; you can redistribute it and/or
@@ -101 +101 @@
 
     JSValue *throwError(ExecState *, ErrorType, const char *msg);
+    JSValue* throwError(ExecState *, ErrorType, const char* msg, const char*);
     JSValue *throwError(ExecState *, ErrorType, const char *msg, JSValue *, Node *);
     JSValue *throwError(ExecState *, ErrorType, const char *msg, const Identifier &);
@@ -109 +110 @@
     JSValue *throwUndefinedVariableError(ExecState *, const Identifier &);
 
-    void setExceptionDetailsIfNeeded(ExecState*);
-    void debugExceptionIfNeeded(ExecState*, JSValue*);
+    void handleException(ExecState*);
+    void handleException(ExecState*, JSValue*);
 
     int m_line;
@@ -454 +455 @@
   };
 
+  class PostfixErrorNode : public Node {
+  public:
+    PostfixErrorNode(Node* e, Operator o) : m_expr(e), m_oper(o) {}
+    JSValue* evaluate(ExecState*);
+    virtual void streamTo(SourceStream&) const;
+  private:
+    RefPtr<Node> m_expr;
+    Operator m_oper;
+  };
+
   class DeleteResolveNode : public Node {
   public:
@@ -548 +559 @@
     RefPtr<Node> m_base;
     Identifier m_ident;
+    Operator m_oper;
+  };
+
+  class PrefixErrorNode : public Node {
+  public:
+    PrefixErrorNode(Node* e, Operator o) : m_expr(e), m_oper(o) {}
+    JSValue* evaluate(ExecState*);
+    virtual void streamTo(SourceStream&) const;
+  private:
+    RefPtr<Node> m_expr;
     Operator m_oper;
   };
@@ -721 +742 @@
     RefPtr<Node> m_base;
     Identifier m_ident;
+    Operator m_oper;
+    RefPtr<Node> m_right;
+  };
+
+  class AssignErrorNode : public Node {
+  public:
+    AssignErrorNode(Node* left, Operator oper, Node* right)
+      : m_left(left), m_oper(oper), m_right(right) {}
+    JSValue* evaluate(ExecState*);
+    virtual void streamTo(SourceStream&) const;
+  protected:
+    RefPtr<Node> m_left;
     Operator m_oper;
     RefPtr<Node> m_right;