Changeset 21027 in webkit for trunk/JavaScriptCore/kjs/nodes2string.cpp

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/nodes2string.cpp (modified) (5 diffs)

trunk/JavaScriptCore/kjs/nodes2string.cpp

@@ -2 +2 @@
  *  This file is part of the KDE libraries
  *  Copyright (C) 2002 Harri Porten ([email protected])
- *  Copyright (C) 2003 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
@@ -283 +283 @@
 }
 
-void DeleteResolveNode::streamTo(SourceStream &s) const
-{
-  s << "delete " << m_ident;
-}
-
-void DeleteBracketNode::streamTo(SourceStream &s) const
-{
-  s << "delete " << m_base << "[" << m_subscript << "]";
-}
-
-void DeleteDotNode::streamTo(SourceStream &s) const
-{
-  s << "delete " << m_base << "." << m_ident;
-}
-
-void DeleteValueNode::streamTo(SourceStream &s) const
-{
-  s << "delete " << m_expr;
-}
-
-void VoidNode::streamTo(SourceStream &s) const
-{
-  s << "void " << expr;
-}
-
-void TypeOfValueNode::streamTo(SourceStream &s) const
-{
-  s << "typeof " << m_expr;
-}
-
-void TypeOfResolveNode::streamTo(SourceStream &s) const
-{
-  s << "typeof " << m_ident;
-}
-
-void PrefixResolveNode::streamTo(SourceStream &s) const
-{
+void PostfixErrorNode::streamTo(SourceStream& s) const
+{
+  s << m_expr;
   if (m_oper == OpPlusPlus)
     s << "++";
   else
     s << "--";
-  s << m_ident;
-}
-
-void PrefixBracketNode::streamTo(SourceStream &s) const
+}
+
+void DeleteResolveNode::streamTo(SourceStream &s) const
+{
+  s << "delete " << m_ident;
+}
+
+void DeleteBracketNode::streamTo(SourceStream &s) const
+{
+  s << "delete " << m_base << "[" << m_subscript << "]";
+}
+
+void DeleteDotNode::streamTo(SourceStream &s) const
+{
+  s << "delete " << m_base << "." << m_ident;
+}
+
+void DeleteValueNode::streamTo(SourceStream &s) const
+{
+  s << "delete " << m_expr;
+}
+
+void VoidNode::streamTo(SourceStream &s) const
+{
+  s << "void " << expr;
+}
+
+void TypeOfValueNode::streamTo(SourceStream &s) const
+{
+  s << "typeof " << m_expr;
+}
+
+void TypeOfResolveNode::streamTo(SourceStream &s) const
+{
+  s << "typeof " << m_ident;
+}
+
+void PrefixResolveNode::streamTo(SourceStream &s) const
 {
   if (m_oper == OpPlusPlus)
@@ -333 +333 @@
   else
     s << "--";
-  s << m_base << "[" << m_subscript << "]";
-}
-
-void PrefixDotNode::streamTo(SourceStream &s) const
+  s << m_ident;
+}
+
+void PrefixBracketNode::streamTo(SourceStream &s) const
 {
   if (m_oper == OpPlusPlus)
@@ -342 +342 @@
   else
     s << "--";
+  s << m_base << "[" << m_subscript << "]";
+}
+
+void PrefixDotNode::streamTo(SourceStream &s) const
+{
+  if (m_oper == OpPlusPlus)
+    s << "++";
+  else
+    s << "--";
   s << m_base << "." << m_ident;
+}
+
+void PrefixErrorNode::streamTo(SourceStream& s) const
+{
+  if (m_oper == OpPlusPlus)
+    s << "++";
+  else
+    s << "--";
+  s << m_expr;
 }
 
@@ -526 +544 @@
 }
 
+void AssignErrorNode::streamTo(SourceStream& s) const
+{
+  s << m_left;
+  streamAssignmentOperatorTo(s, m_oper);
+  s << m_right;
+}
+
 void CommaNode::streamTo(SourceStream &s) const
 {