Oilpan: Ensure that classes with virtual trace methods always have vtables for their left-most base…

Source/core/events/DOMWindowEventQueue.cpp

Index: Source/core/events/DOMWindowEventQueue.cpp
diff --git a/Source/core/events/DOMWindowEventQueue.cpp b/Source/core/events/DOMWindowEventQueue.cpp
index b4705379ac5fbe5fe410add3f0ef89cb9d80bc50..9475f2dbed70bc8935402a4fbee74e5dfce21c2a 100644
--- a/Source/core/events/DOMWindowEventQueue.cpp
+++ b/Source/core/events/DOMWindowEventQueue.cpp
@@ -40,7 +40,11 @@ public:
     DOMWindowEventQueueTimer(DOMWindowEventQueue* eventQueue, ExecutionContext* context)
         : SuspendableTimer(context)
         , m_eventQueue(eventQueue) { }
-    void trace(Visitor* visitor) { visitor->trace(m_eventQueue); }
+    virtual void trace(Visitor* visitor)

[Mads Ager (chromium), 2014/08/12 06:07:04]

Separate the constructor and the trace method with a blank line?

+    {
+        visitor->trace(m_eventQueue);
+        SuspendableTimer::trace(visitor);
+    }
 
 private:
     virtual void fired() { m_eventQueue->pendingEventTimerFired(); }