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

Unified Diff: src/heap/heap.cc

Issue 1145103002: Take freed handles into account when scheduling idle GCs. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 7 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 | « src/heap/gc-idle-time-handler.cc ('k') | test/unittests/heap/gc-idle-time-handler-unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/heap.cc
diff --git a/src/heap/heap.cc b/src/heap/heap.cc
index eb7c7a8cd12525a5e97342b841eceb8f0d1e2551..72769fe385a2f98e1ff633fef759667d3cb3d1f6 100644
--- a/src/heap/heap.cc
+++ b/src/heap/heap.cc
@@ -931,7 +931,7 @@ bool Heap::CollectGarbage(GarbageCollector collector, const char* gc_reason,
}
if (collector == MARK_COMPACTOR) {
- gc_idle_time_handler_.NotifyMarkCompact();
+ gc_idle_time_handler_.NotifyMarkCompact(next_gc_likely_to_collect_more);
Hannes Payer (out of office) 2015/05/20 11:39:13 Here it would also make sense to take fragmentatio
ulan 2015/05/20 11:45:10 I wanted to add fragmentation check after landing
} else {
gc_idle_time_handler_.NotifyScavenge();
}
« no previous file with comments | « src/heap/gc-idle-time-handler.cc ('k') | test/unittests/heap/gc-idle-time-handler-unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698