Remove a thread-local termination GC

third_party/WebKit/Source/platform/heap/ThreadState.cpp

Index: third_party/WebKit/Source/platform/heap/ThreadState.cpp
diff --git a/third_party/WebKit/Source/platform/heap/ThreadState.cpp b/third_party/WebKit/Source/platform/heap/ThreadState.cpp
index a656dfecef2e02a9c2983e0263dc42c4602ed4d9..30eca8745599191c39d7c463495a70e0e1bf7917 100644
--- a/third_party/WebKit/Source/platform/heap/ThreadState.cpp
+++ b/third_party/WebKit/Source/platform/heap/ThreadState.cpp
@@ -244,7 +244,8 @@ void ThreadState::runTerminationGC() {
   int currentCount = getPersistentRegion()->numberOfPersistents();
   ASSERT(currentCount >= 0);
   while (currentCount != oldCount) {
-    collectGarbageForTerminatingThread();
+    collectGarbage(BlinkGC::NoHeapPointersOnStack, BlinkGC::GCWithSweep,

[sof, 2017/02/08 11:49:47]

We're unnecessarily locking CTP access by doing this, and risk compacting --
what about the extra heap stat accounting done?

[haraken, 2017/02/08 12:00:10]

Added BlinkGC::ThreadTerminationGC. This prevents the GC from triggering a heap
compaction. See HeapCompact::shouldCompact().

I think we need to lock CTP though.

+                   BlinkGC::PreciseGC);
     // Release the thread-local static persistents that were
     // instantiated while running the termination GC.
     releaseStaticPersistentNodes();