Oilpan: Move MockActiveDOMObject to Oilpan's heap

Source/core/dom/ActiveDOMObjectTest.cpp

Index: Source/core/dom/ActiveDOMObjectTest.cpp
diff --git a/Source/core/dom/ActiveDOMObjectTest.cpp b/Source/core/dom/ActiveDOMObjectTest.cpp
index a41a5add7f5364bb7c740720d25f36370bd4d1da..0514f2f588fdf0441794cbe5eda5b9d8f6bd6dbf 100644
--- a/Source/core/dom/ActiveDOMObjectTest.cpp
+++ b/Source/core/dom/ActiveDOMObjectTest.cpp
@@ -39,9 +39,11 @@ using namespace blink;
 
 namespace {
 
-class MockActiveDOMObject : public ActiveDOMObject {
+class MockActiveDOMObject final : public GarbageCollectedFinalized<MockActiveDOMObject>, public ActiveDOMObject {

[haraken, 2014/12/03 02:19:44]

At first, I wrote this as 'public GarbageCollected<MockActiveDOMObject>'. This
is wrong because ContextLifecycleObserver (which ActiveDOMObject derives) needs
a destructor, but the GC plugin didn't catch the error. Is it probably a bug of
the plugin?

[tkent, 2014/12/03 03:22:05]

I guess the reason is that MockActiveDOMObject is not in blink namespace.

 public:
-    MockActiveDOMObject(ExecutionContext* context) : ActiveDOMObject(context) { }
+    explicit MockActiveDOMObject(ExecutionContext* context) : ActiveDOMObject(context) { }
+
+    void trace(Visitor*) { }
 
     MOCK_METHOD0(suspend, void());
     MOCK_METHOD0(resume, void());
@@ -54,20 +56,20 @@ protected:
 
     Document& srcDocument() const { return m_srcPageHolder->document(); }
     Document& destDocument() const { return m_destPageHolder->document(); }
-    MockActiveDOMObject& activeDOMObject() { return m_activeDOMObject; }
+    MockActiveDOMObject& activeDOMObject() { return *m_activeDOMObject; }
 
 private:
     OwnPtr<DummyPageHolder> m_srcPageHolder;
     OwnPtr<DummyPageHolder> m_destPageHolder;
-    MockActiveDOMObject m_activeDOMObject;
+    Persistent<MockActiveDOMObject> m_activeDOMObject;
 };
 
 ActiveDOMObjectTest::ActiveDOMObjectTest()
     : m_srcPageHolder(DummyPageHolder::create(IntSize(800, 600)))
     , m_destPageHolder(DummyPageHolder::create(IntSize(800, 600)))
-    , m_activeDOMObject(&m_srcPageHolder->document())
+    , m_activeDOMObject(new MockActiveDOMObject(&m_srcPageHolder->document()))
 {
-    m_activeDOMObject.suspendIfNeeded();
+    m_activeDOMObject->suspendIfNeeded();
 }
 
 TEST_F(ActiveDOMObjectTest, NewContextObserved)