Revert of "[heap] Divide available memory upon compaction tasks"

src/heap/spaces.h

Index: src/heap/spaces.h
diff --git a/src/heap/spaces.h b/src/heap/spaces.h
index 8f49185fae59a4d1f14e44cbcc9baa4d8248914c..de49b5630716c1d035c1e4e901c0230d9c44dab7 100644
--- a/src/heap/spaces.h
+++ b/src/heap/spaces.h
@@ -19,7 +19,6 @@
 namespace v8 {
 namespace internal {
 
-class CompactionSpaceCollection;
 class Isolate;
 
 // -----------------------------------------------------------------------------
@@ -1732,14 +1731,6 @@
            large_list_.available() + huge_list_.available();
   }
 
-  // The method tries to find a {FreeSpace} node of at least {size_in_bytes}
-  // size in the free list category exactly matching the size. If no suitable
-  // node could be found, the method falls back to retrieving a {FreeSpace}
-  // from the large or huge free list category.
-  //
-  // Can be used concurrently.
-  MUST_USE_RESULT FreeSpace* TryRemoveMemory(intptr_t hint_size_in_bytes);
-
   bool IsEmpty() {
     return small_list_.IsEmpty() && medium_list_.IsEmpty() &&
            large_list_.IsEmpty() && huge_list_.IsEmpty();
@@ -1851,8 +1842,6 @@
 
 class PagedSpace : public Space {
  public:
-  static const intptr_t kCompactionMemoryWanted = 500 * KB;
-
   // Creates a space with an id.
   PagedSpace(Heap* heap, AllocationSpace id, Executability executable);
 
@@ -2054,26 +2043,15 @@
   // Return size of allocatable area on a page in this space.
   inline int AreaSize() { return area_size_; }
 
-  virtual bool is_local() { return false; }
-
   // Merges {other} into the current space. Note that this modifies {other},
   // e.g., removes its bump pointer area and resets statistics.
   void MergeCompactionSpace(CompactionSpace* other);
 
-  void DivideUponCompactionSpaces(CompactionSpaceCollection** other, int num,
-                                  intptr_t limit = kCompactionMemoryWanted);
-
-  // Refills the free list from the corresponding free list filled by the
-  // sweeper.
-  virtual void RefillFreeList();
+  void MoveOverFreeMemory(PagedSpace* other);
+
+  virtual bool is_local() { return false; }
 
  protected:
-  void AddMemory(Address start, intptr_t size);
-
-  FreeSpace* TryRemoveMemory(intptr_t size_in_bytes);
-
-  void MoveOverFreeMemory(PagedSpace* other);
-
   // PagedSpaces that should be included in snapshots have different, i.e.,
   // smaller, initial pages.
   virtual bool snapshotable() { return true; }
@@ -2134,9 +2112,6 @@
 
   friend class MarkCompactCollector;
   friend class PageIterator;
-
-  // Used in cctest.
-  friend class HeapTester;
 };
 
 
@@ -2814,13 +2789,11 @@
     Free(start, size_in_bytes);
   }
 
-  virtual bool is_local() override { return true; }
-
-  virtual void RefillFreeList() override;
+  virtual bool is_local() { return true; }
 
  protected:
   // The space is temporary and not included in any snapshots.
-  virtual bool snapshotable() override { return false; }
+  virtual bool snapshotable() { return false; }
 };