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

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

Issue 1478303002: Revert of [runtime] Replace global object link with native context link in all contexts. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 1 month 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-heap.cc
diff --git a/test/cctest/test-heap.cc b/test/cctest/test-heap.cc
index db65cd8baaea683bbb6e5246509498dcb398c5f1..e52c0e4a1480244a2fc6cd2b6958f95e94798822 100644
--- a/test/cctest/test-heap.cc
+++ b/test/cctest/test-heap.cc
@@ -2364,7 +2364,7 @@
// Subtract two to compensate for the two global objects (not global
// JSObjects, of which there would only be one) that are part of the code stub
// context, which is always present.
- return count - 1;
+ return count - 2;
}
@@ -2384,7 +2384,7 @@
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(2, NumberOfGlobalObjects());
+ CHECK_EQ(4, NumberOfGlobalObjects());
{
v8::HandleScope inner_scope(isolate);
@@ -2410,7 +2410,7 @@
isolate->ContextDisposedNotification();
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(1, NumberOfGlobalObjects());
+ CHECK_EQ(2, NumberOfGlobalObjects());
ctx2p.Reset();
CcTest::heap()->CollectAllAvailableGarbage();
CHECK_EQ(0, NumberOfGlobalObjects());
@@ -2433,7 +2433,7 @@
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(2, NumberOfGlobalObjects());
+ CHECK_EQ(4, NumberOfGlobalObjects());
{
v8::HandleScope inner_scope(isolate);
@@ -2459,7 +2459,7 @@
isolate->ContextDisposedNotification();
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(1, NumberOfGlobalObjects());
+ CHECK_EQ(2, NumberOfGlobalObjects());
ctx2p.Reset();
CcTest::heap()->CollectAllAvailableGarbage();
CHECK_EQ(0, NumberOfGlobalObjects());
@@ -2480,7 +2480,7 @@
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(2, NumberOfGlobalObjects());
+ CHECK_EQ(4, NumberOfGlobalObjects());
{
v8::HandleScope inner_scope(isolate);
@@ -2506,7 +2506,7 @@
isolate->ContextDisposedNotification();
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(1, NumberOfGlobalObjects());
+ CHECK_EQ(2, NumberOfGlobalObjects());
ctx2p.Reset();
CcTest::heap()->CollectAllAvailableGarbage();
CHECK_EQ(0, NumberOfGlobalObjects());
@@ -2527,7 +2527,7 @@
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(2, NumberOfGlobalObjects());
+ CHECK_EQ(4, NumberOfGlobalObjects());
{
v8::HandleScope inner_scope(isolate);
@@ -2557,7 +2557,7 @@
isolate->ContextDisposedNotification();
}
CcTest::heap()->CollectAllAvailableGarbage();
- CHECK_EQ(1, NumberOfGlobalObjects());
+ CHECK_EQ(2, NumberOfGlobalObjects());
ctx2p.Reset();
CcTest::heap()->CollectAllAvailableGarbage();
CHECK_EQ(0, NumberOfGlobalObjects());
« no previous file with comments | « test/cctest/interpreter/test-bytecode-generator.cc ('k') | test/unittests/compiler/js-typed-lowering-unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698