Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(230)

Unified Diff: Source/platform/heap/HeapTest.cpp

Issue 452723002: Oilpan: blink_heap_unittests: Test case have memory leak checked by Lsan (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: change collectGarbage(...) to collectAllGarbage() in HeapTest main() Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/platform/heap/RunAllTests.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/heap/HeapTest.cpp
diff --git a/Source/platform/heap/HeapTest.cpp b/Source/platform/heap/HeapTest.cpp
index 1aafc8a11ed9fe2206c0b5e1b1d344df71588f18..db6eb0907dbab5c38943e551031c9fe7b4c0bb3e 100644
--- a/Source/platform/heap/HeapTest.cpp
+++ b/Source/platform/heap/HeapTest.cpp
@@ -4975,12 +4975,13 @@ TEST(HeapTest, RecursiveMutex)
}
template<typename T>
-class TraceIfNeededTester : public GarbageCollected<TraceIfNeededTester<T> > {
+class TraceIfNeededTester : public GarbageCollectedFinalized<TraceIfNeededTester<T> > {
public:
static TraceIfNeededTester<T>* create() { return new TraceIfNeededTester<T>(); }
static TraceIfNeededTester<T>* create(const T& obj) { return new TraceIfNeededTester<T>(obj); }
void trace(Visitor* visitor) { TraceIfNeeded<T>::trace(visitor, &m_obj); }
T& obj() { return m_obj; }
+ ~TraceIfNeededTester() { }
private:
TraceIfNeededTester() { }
explicit TraceIfNeededTester(const T& obj) : m_obj(obj) { }
« no previous file with comments | « no previous file | Source/platform/heap/RunAllTests.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698