Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1432)

Unified Diff: test/cctest/test-parsing.cc

Issue 2644333002: test-parsing/NoPessimisticContextAllocation: add a missing test (Closed)
Patch Set: code review (adamk@) Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/test-parsing.cc
diff --git a/test/cctest/test-parsing.cc b/test/cctest/test-parsing.cc
index 58b0acd14ce4bc51a0e8d47a143186e3b833ef11..8a50b6edbdae943f13d63fc5c3152794deca0e5a 100644
--- a/test/cctest/test-parsing.cc
+++ b/test/cctest/test-parsing.cc
@@ -8702,9 +8702,9 @@ TEST(NoPessimisticContextAllocation) {
// Context allocating because we need to:
{"", "my_var;", true},
{"", "if (true) { let my_var; } my_var;", true},
- {"", "eval(\"foo\");", true},
+ {"", "eval('foo');", true},
{"", "function inner2() { my_var; }", true},
- {"", "function inner2() { eval(\"foo\"); }", true},
+ {"", "function inner2() { eval('foo'); }", true},
{"", "var {my_var : a} = {my_var};", true},
{"", "let {my_var : a} = {my_var};", true},
{"", "const {my_var : a} = {my_var};", true},
@@ -8717,6 +8717,8 @@ TEST(NoPessimisticContextAllocation) {
{"[a] = [my_var]", "", true},
{"", "function inner2([a] = [my_var]) { }", true},
{"", "([a] = [my_var]) => { }", true},
+ {"", "function inner2(a = eval('')) { }", true},
+ {"", "(a = eval('')) => { }", true},
{"", "try { } catch (my_var) { } my_var;", true},
{"", "for (my_var in {}) { my_var; }", true},
{"", "for (my_var in {}) { }", true},
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698