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

Unified Diff: src/api.cc

Issue 1314863003: [heap] More flag cleanup. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Added more comments Created 5 years, 4 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 | src/debug/debug.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/api.cc
diff --git a/src/api.cc b/src/api.cc
index 42a887582723524f4773847d51cdb2da5f58827a..2c8c93722060a61a7fa061afd73fedf29d09c804 100644
--- a/src/api.cc
+++ b/src/api.cc
@@ -6880,7 +6880,7 @@ void Isolate::CollectAllGarbage(const char* gc_reason) {
kGCCallbackFlagSynchronousPhantomCallbackProcessing, gc_reason);
} else {
heap->CollectAllGarbage(
- i::Heap::kNoGCFlags, gc_reason,
+ gc_reason, i::Heap::kNoGCFlags,
kGCCallbackFlagSynchronousPhantomCallbackProcessing);
}
} else {
@@ -7071,13 +7071,13 @@ void Isolate::RequestGarbageCollectionForTesting(GarbageCollectionType type) {
CHECK(i::FLAG_expose_gc);
if (type == kMinorGarbageCollection) {
reinterpret_cast<i::Isolate*>(this)->heap()->CollectGarbage(
- i::NEW_SPACE, "Isolate::RequestGarbageCollection",
+ i::NEW_SPACE, "Isolate::RequestGarbageCollection", i::Heap::kNoGCFlags,
kGCCallbackFlagForced);
} else {
DCHECK_EQ(kFullGarbageCollection, type);
reinterpret_cast<i::Isolate*>(this)->heap()->CollectAllGarbage(
- i::Heap::kAbortIncrementalMarkingMask,
- "Isolate::RequestGarbageCollection", kGCCallbackFlagForced);
+ "Isolate::RequestGarbageCollection",
+ i::Heap::kAbortIncrementalMarkingMask, kGCCallbackFlagForced);
}
}
« no previous file with comments | « no previous file | src/debug/debug.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698