Oilpan: blink_heap_unittests: Test case have memory leak checked by Lsan

Source/platform/heap/RunAllTests.cpp

Index: Source/platform/heap/RunAllTests.cpp
diff --git a/Source/platform/heap/RunAllTests.cpp b/Source/platform/heap/RunAllTests.cpp
index ad755437022245a72e73c2d57f0532df497f449e..8b500426493358ee8b57b3bfeb94c04d3ba56d06 100644
--- a/Source/platform/heap/RunAllTests.cpp
+++ b/Source/platform/heap/RunAllTests.cpp
@@ -58,6 +58,7 @@ int main(int argc, char** argv)
     blink::ThreadState::attachMainThread();
     blink::EventTracer::initialize();
     int result = base::RunUnitTestsUsingBaseTestSuite(argc, argv);
+    blink::Heap::collectGarbage(blink::ThreadState::NoHeapPointersOnStack);

[haraken, 2014/08/11 06:12:39]

I'd prefer blink::Heap::collectAllGarbage(), (which calls Heap::collectGarbage()
multiple times to ensure that everything is gone).

[Daniel Chow, 2014/08/11 06:32:25]

Acknowledged.

And there is another case crash that I did not make any change in this CL for it
(I think it is not exactly a issue but it's a noise) : 
the test case CheckAndMarkPointer will trigger a ASSERT in 
ThreadState::copyStackUntilSafePointScope
ASSERT(slotCount < 1024); // Catch potential performance issues.

     blink::ThreadState::detachMainThread();
     blink::Heap::shutdown();
     return result;