Changeset 62727 in webkit for trunk/JavaScriptCore/parser/ASTBuilder.h

Timestamp:

Jul 7, 2010, 4:27:57 PM (15 years ago)

Author:

[email protected]

Message:

2010-07-07 Oliver Hunt <​[email protected]>

Reviewed by Geoffrey Garen.

Lazy mode of parser allows invalid syntax in object literals.
​https://bugs.webkit.org/show_bug.cgi?id=41809

Make the parser itself validate getter and setter syntax rather
than offloading it to the AST builder.

• parser/ASTBuilder.h: (JSC::ASTBuilder::createGetterOrSetterProperty):
• parser/JSParser.cpp: (JSC::JSParser::parseProperty):

File:

• trunk/JavaScriptCore/parser/ASTBuilder.h (modified) (1 diff)

trunk/JavaScriptCore/parser/ASTBuilder.h

@@ -254 +254 @@
         ASSERT(name);
         PropertyNode::Type type;
-        if (*getOrSet == "get")
+        if (*getOrSet == m_globalData->propertyNames->get)
             type = PropertyNode::Getter;
-        else if (*getOrSet == "set")
+        else if (*getOrSet == m_globalData->propertyNames->set)
             type = PropertyNode::Setter;
         else