Changeset 215235 in webkit for trunk/Source/JavaScriptCore/parser/Lexer.cpp

Timestamp:

Apr 11, 2017, 9:54:49 AM (8 years ago)

Author:

[email protected]

Message:

test262: test262/test/annexB/language/comments/multi-line-html-close.js
​https://bugs.webkit.org/show_bug.cgi?id=170648

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-04-11
Reviewed by Keith Miller.

JSTests:

• test262.yaml:

Source/JavaScriptCore:

• parser/Lexer.cpp:

(JSC::Lexer<T>::lex):
A multi-line comment that contains a line terminator is itself treated
like a line terminator. An HTML Close Comment that comes after it can
therefore treat it like it is at the start of a line, because it was
immediately preceeded by the equivalent of a line terminator.

LayoutTests:

• js/parser-xml-close-comment-expected.txt:
• js/script-tests/parser-xml-close-comment.js:

Update test coverage for JavaScript HTML comments.

File:

• trunk/Source/JavaScriptCore/parser/Lexer.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/parser/Lexer.cpp

@@ -1992 +1992 @@
         if (m_current == '-') {
             shift();
-            if (m_atLineStart && m_current == '>') {
+            if ((m_atLineStart || m_terminator) && m_current == '>') {
                 if (m_scriptMode == JSParserScriptMode::Classic) {
                     shift();