Move the Widget hierarchy to the Oilpan heap.

Source/web/PopupListBox.cpp

 /*
  * Copyright (c) 2011, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 69 matching lines @@
     , m_lastCharTime(0)
     , m_maxWindowWidth(std::numeric_limits<int>::max())
     , m_container(container)
 {
 }
 
 PopupListBox::~PopupListBox()
 {
     clear();
 
+    // Oilpan: the scrollbars of the ScrollView are self-sufficient,
+    // capable of detaching themselves from their animator on
+    // finalization.
+#if !ENABLE(OILPAN)
     setHasVerticalScrollbar(false);
+#endif
+}
+
+void PopupListBox::trace(Visitor* visitor)
+{
+    visitor->trace(m_capturingScrollbar);
+    visitor->trace(m_lastScrollbarUnderMouse);
+    visitor->trace(m_focusedElement);
+    visitor->trace(m_container);
+    visitor->trace(m_verticalScrollbar);
+    Widget::trace(visitor);
 }
 
 bool PopupListBox::handleMouseDownEvent(const PlatformMouseEvent& event)
 {
     Scrollbar* scrollbar = scrollbarAtWindowPoint(event.position());
     if (scrollbar) {
         m_capturingScrollbar = scrollbar;
         m_capturingScrollbar->mouseDown(event);
         return true;
     }
@@ skipping 394 matching lines @@
         Font font(d);
         font.update(nullptr);
         return font;
     }
 
     return itemFont;
 }
 
 void PopupListBox::abandon()
 {
-    RefPtr<PopupListBox> keepAlive(this);
+    RefPtrWillBeRawPtr<PopupListBox> protect(this);
 
     m_selectedIndex = m_originalIndex;
 
     hidePopup();
 
     if (m_acceptedIndexOnAbandon >= 0) {
         if (m_popupClient)
             m_popupClient->valueChanged(m_acceptedIndexOnAbandon);
         m_acceptedIndexOnAbandon = -1;
     }
@@ skipping 27 matching lines @@
 
     if (index < 0) {
         if (m_popupClient) {
             // Enter pressed with no selection, just close the popup.
             hidePopup();
         }
         return false;
     }
 
     if (isSelectableItem(index)) {
-        RefPtr<PopupListBox> keepAlive(this);
+        RefPtrWillBeRawPtr<PopupListBox> protect(this);
 
         // Hide ourselves first since valueChanged may have numerous side-effects.
         hidePopup();
 
         // Tell the <select> PopupMenuClient what index was selected.
         m_popupClient->valueChanged(index);
 
         return true;
     }
 
@@ skipping 264 matching lines @@
     setContentsSize(IntSize(contentWidth, getRowBounds(numItems() - 1).maxY()));
 
     if (hostWindow())
         scrollToRevealSelection();
 
     invalidate();
 }
 
 void PopupListBox::clear()
 {
-    for (Vector<PopupItem*>::iterator it = m_items.begin(); it != m_items.end(); ++it)
-        delete *it;
+    deleteAllValues(m_items);

[haraken, 2014/09/29 14:16:36]

It would be better to use Vector<OwnPtr<PopupItem>> though.

     m_items.clear();
 }
 
 bool PopupListBox::isPointInBounds(const IntPoint& point)
 {
     return numItems() && IntRect(0, 0, width(), height()).contains(point);
 }
 
 int PopupListBox::popupContentHeight() const
 {
@@ skipping 191 matching lines @@
     maximumOffset.clampNegativeToZero();
     return maximumOffset;
 }
 
 IntPoint PopupListBox::minimumScrollPosition() const
 {
     return IntPoint(-scrollOrigin().x(), -scrollOrigin().y());
 }
 
 } // namespace blink