Remove a thread-local termination GC

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

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 215 matching lines @@
 
   releaseStaticPersistentNodes();
 
   // From here on ignore all conservatively discovered
   // pointers into the heap owned by this thread.
   m_isTerminating = true;
 
   // Set the terminate flag on all heap pages of this thread. This is used to
   // ensure we don't trace pages on other threads that are not part of the
   // thread local GC.
+  // TODO(haraken): Remove this. This is not needed once we remove a thread-

[sof, 2017/02/08 12:06:19]

oh, i suppose that is now possible with orphaned pages gone.

+  // local termination GC.
   prepareForThreadStateTermination();
 
   ProcessHeap::crossThreadPersistentRegion().prepareForThreadStateTermination(
       this);
 
   // Do thread local GC's as long as the count of thread local Persistents
   // changes and is above zero.
   int oldCount = -1;
   int currentCount = getPersistentRegion()->numberOfPersistents();
   ASSERT(currentCount >= 0);
   while (currentCount != oldCount) {
-    collectGarbageForTerminatingThread();
+    collectGarbage(BlinkGC::NoHeapPointersOnStack, BlinkGC::GCWithSweep,
+                   BlinkGC::ThreadTerminationGC);
     // Release the thread-local static persistents that were
     // instantiated while running the termination GC.
     releaseStaticPersistentNodes();
     oldCount = currentCount;
     currentCount = getPersistentRegion()->numberOfPersistents();
   }
   // We should not have any persistents left when getting to this point,
   // if we have it is probably a bug so adding a debug ASSERT to catch this.
   ASSERT(!currentCount);
   // All of pre-finalizers should be consumed.
@@ skipping 1523 matching lines @@
     collectGarbage(BlinkGC::NoHeapPointersOnStack, BlinkGC::GCWithSweep,
                    BlinkGC::ForcedGC);
     size_t liveObjects = heap().heapStats().markedObjectSize();
     if (liveObjects == previousLiveObjects)
       break;
     previousLiveObjects = liveObjects;
   }
 }
 
 }  // namespace blink