Commit 02218adb authored by caitpotter88's avatar caitpotter88 Committed by Commit bot

Report SyntaxError on Token::ILLEGAL in ParseTemplateLiteral

BUG=v8:3820, v8:3821
LOG=N
R=arv@chromium.org, dslomov@chromium.org, marja@chromium.org

Review URL: https://codereview.chromium.org/857433004

Cr-Commit-Position: refs/heads/master@{#26132}
parent 6daecbd5
...@@ -2942,11 +2942,17 @@ ParserBase<Traits>::ParseTemplateLiteral(ExpressionT tag, int start, bool* ok) { ...@@ -2942,11 +2942,17 @@ ParserBase<Traits>::ParseTemplateLiteral(ExpressionT tag, int start, bool* ok) {
do { do {
CheckTemplateOctalLiteral(pos, peek_position(), CHECK_OK); CheckTemplateOctalLiteral(pos, peek_position(), CHECK_OK);
next = peek(); next = peek();
if (!next) { if (next == Token::EOS) {
ReportMessageAt(Scanner::Location(start, peek_position()), ReportMessageAt(Scanner::Location(start, peek_position()),
"unterminated_template"); "unterminated_template");
*ok = false; *ok = false;
return Traits::EmptyExpression(); return Traits::EmptyExpression();
} else if (next == Token::ILLEGAL) {
Traits::ReportMessageAt(
Scanner::Location(position() + 1, peek_position()),
"unexpected_token", "ILLEGAL", false);
*ok = false;
return Traits::EmptyExpression();
} }
int expr_pos = peek_position(); int expr_pos = peek_position();
...@@ -2966,10 +2972,16 @@ ParserBase<Traits>::ParseTemplateLiteral(ExpressionT tag, int start, bool* ok) { ...@@ -2966,10 +2972,16 @@ ParserBase<Traits>::ParseTemplateLiteral(ExpressionT tag, int start, bool* ok) {
Next(); Next();
pos = position(); pos = position();
if (!next) { if (next == Token::EOS) {
ReportMessageAt(Scanner::Location(start, pos), "unterminated_template"); ReportMessageAt(Scanner::Location(start, pos), "unterminated_template");
*ok = false; *ok = false;
return Traits::EmptyExpression(); return Traits::EmptyExpression();
} else if (next == Token::ILLEGAL) {
Traits::ReportMessageAt(
Scanner::Location(position() + 1, peek_position()),
"unexpected_token", "ILLEGAL", false);
*ok = false;
return Traits::EmptyExpression();
} }
Traits::AddTemplateSpan(&ts, next == Token::TEMPLATE_TAIL); Traits::AddTemplateSpan(&ts, next == Token::TEMPLATE_TAIL);
......
...@@ -4552,6 +4552,28 @@ TEST(ScanUnterminatedTemplateLiterals) { ...@@ -4552,6 +4552,28 @@ TEST(ScanUnterminatedTemplateLiterals) {
} }
TEST(TemplateLiteralsIllegalTokens) {
const char* context_data[][2] = {{"'use strict';", ""},
{"function foo(){ 'use strict';"
" var a, b, c; return ", "}"},
{NULL, NULL}};
const char* data[] = {
"`hello\\x`",
"`hello\\x${1}`",
"`hello${1}\\x`",
"`hello${1}\\x${2}`",
"`hello\\x\n`",
"`hello\\x\n${1}`",
"`hello${1}\\x\n`",
"`hello${1}\\x\n${2}`",
NULL};
static const ParserFlag always_flags[] = {kAllowHarmonyTemplates};
RunParserSyncTest(context_data, data, kError, NULL, 0, always_flags,
arraysize(always_flags));
}
TEST(LexicalScopingSloppyMode) { TEST(LexicalScopingSloppyMode) {
const char* context_data[][2] = { const char* context_data[][2] = {
{"", ""}, {"", ""},
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment