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

Unified Diff: test/cctest/test-mark-compact.cc

Issue 254273002: Remove BootUpMemoryUse test. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 8 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-mark-compact.cc
diff --git a/test/cctest/test-mark-compact.cc b/test/cctest/test-mark-compact.cc
index a68530189e383107375e747f9b6448ce2dc5a99a..3244cc9e63ce8ba3b2de8feefd9093463a7539ea 100644
--- a/test/cctest/test-mark-compact.cc
+++ b/test/cctest/test-mark-compact.cc
@@ -484,30 +484,6 @@ static intptr_t MemoryInUse() {
}
-TEST(BootUpMemoryUse) {
- intptr_t initial_memory = MemoryInUse();
- // Avoid flakiness.
- FLAG_crankshaft = false;
- FLAG_concurrent_osr = false;
- FLAG_concurrent_recompilation = false;
-
- // Only Linux has the proc filesystem and only if it is mapped. If it's not
- // there we just skip the test.
- if (initial_memory >= 0) {
- CcTest::InitializeVM();
- intptr_t delta = MemoryInUse() - initial_memory;
- printf("delta: %" V8_PTR_PREFIX "d kB\n", delta / 1024);
- if (v8::internal::Snapshot::IsEnabled()) {
- CHECK_LE(delta,
- 3200 * 1024 * FullCodeGenerator::kBootCodeSizeMultiplier / 100);
- } else {
- CHECK_LE(delta,
- 3350 * 1024 * FullCodeGenerator::kBootCodeSizeMultiplier / 100);
- }
- }
-}
-
-
intptr_t ShortLivingIsolate() {
v8::Isolate* isolate = v8::Isolate::New();
{ v8::Isolate::Scope isolate_scope(isolate);
« 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