Oilpan: fix build after r181083

Source/core/frame/FrameConsole.h

Index: Source/core/frame/FrameConsole.h
diff --git a/Source/core/frame/FrameConsole.h b/Source/core/frame/FrameConsole.h
index 74f8e1787b6e1358ba5b8a0ba7db20bd12d9e5bd..a44784098ad5ec91751101c4cd6e79f817c3aa5c 100644
--- a/Source/core/frame/FrameConsole.h
+++ b/Source/core/frame/FrameConsole.h
@@ -45,10 +45,12 @@ class WorkerGlobalScopeProxy;
 
 // FrameConsole takes per-frame console messages and routes them up through the FrameHost to the ChromeClient and Inspector.
 // It's meant as an abstraction around ChromeClient calls and the way that Blink core/ can add messages to the console.
-class FrameConsole FINAL {
+class FrameConsole FINAL : public NoBaseWillBeGarbageCollected<FrameConsole> {
 public:
-    static PassOwnPtr<FrameConsole> create(LocalFrame& frame) { return adoptPtr(new FrameConsole(frame)); }
-    ~FrameConsole();

[Mads Ager (chromium), 2014/08/29 09:54:41]

Looking at the trybots it looks like the removal of the destructor leads to the
compiler-generated one being generated in the .h file where the
ConsoleMessageStorage type is not fully defined. Can we add the include here or
do we need the DECLARE_EMPTY_DESTRUCTOR thingie?

+    static PassOwnPtrWillBeRawPtr<FrameConsole> create(LocalFrame& frame)
+    {
+        return adoptPtrWillBeNoop(new FrameConsole(frame));
+    }
 
     void addMessage(PassRefPtrWillBeRawPtr<ConsoleMessage>);
     void adoptWorkerConsoleMessages(WorkerGlobalScopeProxy*);
@@ -60,12 +62,14 @@ public:
 
     ConsoleMessageStorage* messageStorage();
 
+    void trace(Visitor*);
+
 private:
     explicit FrameConsole(LocalFrame&);
 
     LocalFrame& m_frame;
 
-    OwnPtr<ConsoleMessageStorage> m_consoleMessageStorage;
+    OwnPtrWillBeMember<ConsoleMessageStorage> m_consoleMessageStorage;
 };
 
 } // namespace blink