test/cctest/test-parsing.cc - Issue 908173003: Parsing: Make Parser not know about Isolate during background parsing.

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

Issue 908173003: Parsing: Make Parser not know about Isolate during background parsing. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master

Patch Set: Created 5 years, 10 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

Index: test/cctest/test-parsing.cc

diff --git a/test/cctest/test-parsing.cc b/test/cctest/test-parsing.cc

index ba2fa975fd7ad7728098686d2941e03fe279959f..10f23e6b75ae1a65714281fe40691bc9cc43e346 100644

--- a/test/cctest/test-parsing.cc

+++ b/test/cctest/test-parsing.cc

@@ -160,8 +160,8 @@ TEST(ScanHTMLEndComments) {

i::Zone zone;

i::AstValueFactory ast_value_factory(

&zone, CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log, stack_limit);

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

+ stack_limit);

preparser.set_allow_lazy(true);

i::PreParser::PreParseResult result = preparser.PreParseProgram();

CHECK_EQ(i::PreParser::kPreParseSuccess, result);

@@ -178,8 +178,8 @@ TEST(ScanHTMLEndComments) {

i::Zone zone;

i::AstValueFactory ast_value_factory(

&zone, CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log, stack_limit);

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

+ stack_limit);

preparser.set_allow_lazy(true);

i::PreParser::PreParseResult result = preparser.PreParseProgram();

// Even in the case of a syntax error, kPreParseSuccess is returned.

@@ -328,8 +328,8 @@ TEST(StandAlonePreParser) {

i::Zone zone;

i::AstValueFactory ast_value_factory(

&zone, CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log, stack_limit);

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

+ stack_limit);

preparser.set_allow_lazy(true);

preparser.set_allow_natives(true);

preparser.set_allow_harmony_arrow_functions(true);

@@ -366,8 +366,8 @@ TEST(StandAlonePreParserNoNatives) {

i::Zone zone;

i::AstValueFactory ast_value_factory(

&zone, CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log, stack_limit);

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

+ stack_limit);

preparser.set_allow_lazy(true);

i::PreParser::PreParseResult result = preparser.PreParseProgram();

CHECK_EQ(i::PreParser::kPreParseSuccess, result);

@@ -435,8 +435,7 @@ TEST(RegressChromium62639) {

i::Zone zone;

i::AstValueFactory ast_value_factory(&zone,

CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log,

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

CcTest::i_isolate()->stack_guard()->real_climit());

preparser.set_allow_lazy(true);

i::PreParser::PreParseResult result = preparser.PreParseProgram();

@@ -471,8 +470,7 @@ TEST(Regress928) {

i::Zone zone;

i::AstValueFactory ast_value_factory(&zone,

CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log,

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

CcTest::i_isolate()->stack_guard()->real_climit());

preparser.set_allow_lazy(true);

i::PreParser::PreParseResult result = preparser.PreParseProgram();

@@ -524,8 +522,8 @@ TEST(PreParseOverflow) {

i::Zone zone;

i::AstValueFactory ast_value_factory(&zone,

CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log, stack_limit);

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

+ stack_limit);

preparser.set_allow_lazy(true);

preparser.set_allow_harmony_arrow_functions(true);

i::PreParser::PreParseResult result = preparser.PreParseProgram();

@@ -1054,7 +1052,7 @@ TEST(ScopeUsesArgumentsSuperThis) {

parser.set_allow_harmony_classes(true);

parser.set_allow_harmony_scoping(true);

info.MarkAsGlobal();

- parser.Parse();

+ parser.Parse(&info);

CHECK(i::Rewriter::Rewrite(&info));

CHECK(i::Scope::Analyze(&info));

CHECK(info.function() != NULL);

@@ -1306,7 +1304,7 @@ TEST(ScopePositions) {

parser.set_allow_harmony_arrow_functions(true);

info.MarkAsGlobal();

info.SetLanguageMode(source_data[i].language_mode);

- parser.Parse();

+ parser.Parse(&info);

CHECK(info.function() != NULL);

// Check scope types and positions.

@@ -1442,8 +1440,8 @@ void TestParserSyncWithFlags(i::Handle<i::String> source,

i::Zone zone;

i::AstValueFactory ast_value_factory(

&zone, CcTest::i_isolate()->heap()->HashSeed());

- i::PreParser preparser(CcTest::i_isolate(), &zone, &scanner,

- &ast_value_factory, &log, stack_limit);

+ i::PreParser preparser(&zone, &scanner, &ast_value_factory, &log,

+ stack_limit);

SetParserFlags(&preparser, flags);

scanner.Initialize(&stream);

i::PreParser::PreParseResult result = preparser.PreParseProgram(

@@ -1464,7 +1462,7 @@ void TestParserSyncWithFlags(i::Handle<i::String> source,

i::Parser parser(&info, &parse_info);

SetParserFlags(&parser, flags);

info.MarkAsGlobal();

- parser.Parse();

+ parser.Parse(&info);

function = info.function();

if (function) {

parser_materialized_literals = function->materialized_literal_count();

@@ -3429,7 +3427,7 @@ TEST(InnerAssignment) {

isolate->heap()->HashSeed(), isolate->unicode_cache()};

i::Parser parser(&info, &parse_info);

parser.set_allow_harmony_scoping(true);

- CHECK(parser.Parse());

+ CHECK(parser.Parse(&info));

CHECK(i::Compiler::Analyze(&info));

CHECK(info.function() != NULL);

@@ -4866,7 +4864,7 @@ TEST(BasicImportExportParsing) {

parser.set_allow_harmony_modules(true);

parser.set_allow_harmony_scoping(true);

info.MarkAsModule();

- if (!parser.Parse()) {

+ if (!parser.Parse(&info)) {

i::Handle<i::JSObject> exception_handle(

i::JSObject::cast(isolate->pending_exception()));

i::Handle<i::String> message_string =

@@ -4896,7 +4894,7 @@ TEST(BasicImportExportParsing) {

parser.set_allow_harmony_modules(true);

parser.set_allow_harmony_scoping(true);

info.MarkAsGlobal();

- CHECK(!parser.Parse());

+ CHECK(!parser.Parse(&info));

}

@@ -4988,7 +4986,7 @@ TEST(ImportExportParsingErrors) {

parser.set_allow_harmony_modules(true);

parser.set_allow_harmony_scoping(true);

info.MarkAsModule();

- CHECK(!parser.Parse());

+ CHECK(!parser.Parse(&info));

}

@@ -5082,7 +5080,7 @@ void TestLanguageMode(const char* source,

i::Parser parser(&info, &parse_info);

parser.set_allow_strong_mode(true);

info.MarkAsGlobal();

- parser.Parse();

+ parser.Parse(&info);

CHECK(info.function() != NULL);

CHECK_EQ(expected_language_mode, info.function()->language_mode());

}

« src/parser.cc ('K') | « src/preparser.h ('k') | no next file » | no next file with comments »