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

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

Issue 1361853005: [heap] Prepare code for smaller large object allocation limit than max allocatable memory. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 3 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-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index ab44af7f934aae2a2dde02dcdc2f7545d5ca5d26..2047fdeee7c522f1c4a27261bc33dbef29d37817 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -6547,7 +6547,7 @@ static void IndependentWeakHandle(bool global_gc, bool interlinked) {
}
// We are relying on this creating a big flag array and reserving the space
// up front.
- v8::Handle<Value> big_array = CompileRun("new Array(50000)");
+ v8::Handle<Value> big_array = CompileRun("new Array(5000)");
Michael Lippautz 2015/09/30 14:14:24 I assume there's no way get constants here.
Hannes Payer (out of office) 2015/10/05 13:13:10 Acknowledged.
a->Set(v8_str("y"), big_array);
big_heap_size = CcTest::heap()->SizeOfObjects();
}
@@ -6569,7 +6569,7 @@ static void IndependentWeakHandle(bool global_gc, bool interlinked) {
}
// A single GC should be enough to reclaim the memory, since we are using
// phantom handles.
- CHECK_LT(CcTest::heap()->SizeOfObjects(), big_heap_size - 200000);
+ CHECK_LT(CcTest::heap()->SizeOfObjects(), big_heap_size - 20000);
CHECK(object_a.flag);
CHECK(object_b.flag);
}
« no previous file with comments | « test/cctest/cctest.h ('k') | test/cctest/test-heap.cc » ('j') | test/cctest/test-heap.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698