Refactor RegionTree

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

Index: third_party/WebKit/Source/platform/heap/HeapPage.cpp
diff --git a/third_party/WebKit/Source/platform/heap/HeapPage.cpp b/third_party/WebKit/Source/platform/heap/HeapPage.cpp
index b7d24a5cb246c92056ec108352c14fc458882d81..bb93c5f8e0432d6aad3d8b4112c4264b88889d1a 100644
--- a/third_party/WebKit/Source/platform/heap/HeapPage.cpp
+++ b/third_party/WebKit/Source/platform/heap/HeapPage.cpp
@@ -426,6 +426,7 @@ void NormalPageArena::allocatePage()
         //    [ guard os page | ... payload ... | guard os page ]
         //    ^---{ aligned to blink page size }
         PageMemoryRegion* region = PageMemoryRegion::allocateNormalPages();
+        Heap::addPageMemoryRegion(region);

[haraken, 2016/03/15 05:49:50]

Does addPageMemoryRegion need to be a method of Heap? It looks better if we
could call region->add().

[keishi, 2016/03/15 07:48:43]

Done. I had the addPageMemoryRegion because regionTree was local to Heap.cpp.
Changed to use Heap::getRegionTree. The RegionTree is passed from *PageArena so
pageMemory doesn't need to use Heap.

 
         // Setup the PageMemory object for each of the pages in the region.
         for (size_t i = 0; i < blinkPagesPerRegion; ++i) {