Extend test for HTML comments (#684)

Extend test for HTML comments

The V8 engine incorrectly requires a leading newline character for
MultiLineComments which contain the optional trailing HTMLCloseComment
[1]. Extend the current tests to fail when such an invalid restriction
is in place.
[1] https://bugs.chromium.org/p/v8/issues/detail?id=5142

1 files changed, 31 insertions, 1 deletions

diff --git a/test/annexB/language/comments/multi-line-html-close.js b/test/annexB/language/comments/multi-line-html-close.js
index 8e307f594..22a8b173a 100644
--- a/test/annexB/language/comments/multi-line-html-close.js
+++ b/test/annexB/language/comments/multi-line-html-close.js
@@ -46,6 +46,36 @@ counter += 1;
 */ /**/ /* second optional SingleLineDelimitedCommentSequence */-->the comment extends to these characters
 counter += 1;
 
+// The V8 engine exhibited a bug where HTMLCloseComment was not recognized
+// within MultiLineComment in cases where MultiLineComment was not the first
+// token on the line of source text. The following tests demonstrate the same
+// productions listed above with the addition of such a leading token.
+
+0/*
+*/-->
+counter += 1;
+
+0/*
+*/-->the comment extends to these characters
+counter += 1;
+
+0/* optional FirstCommentLine
+*/-->the comment extends to these characters
+counter += 1;
+
+0/*
+optional
+MultiLineCommentChars */-->the comment extends to these characters
+counter += 1;
+
+0/*
+*/ /* optional SingleLineDelimitedCommentSequence */-->the comment extends to these characters
+counter += 1;
+
+0/*
+*/ /**/ /* second optional SingleLineDelimitedCommentSequence */-->the comment extends to these characters
+counter += 1;
+
 // Because this test concerns the interpretation of non-executable character
 // sequences within ECMAScript source code, special care must be taken to
 // ensure that executable code is evaluated as expected.
@@ -53,6 +83,6 @@ counter += 1;
 // Express the intended behavior by intentionally throwing an error; this
 // guarantees that test runners will only consider the test "passing" if
 // executable sequences are correctly interpreted as such.
-if (counter === 6) {
+if (counter === 12) {
   throw new Test262Error();
 }