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

Unified Diff: Source/web/PopupContainer.h

Issue 603193005: Move the Widget hierarchy to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Switch LocalFrame::m_pluginElements rep to HashSet<HTMLPlugInElement*> Created 6 years, 2 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
Index: Source/web/PopupContainer.h
diff --git a/Source/web/PopupContainer.h b/Source/web/PopupContainer.h
index 1922a8a7d919d2cf278622935288a7b3a5ff97a8..6ffd4505e362b9d4bbbe55b14af2842f26e050fd 100644
--- a/Source/web/PopupContainer.h
+++ b/Source/web/PopupContainer.h
@@ -34,6 +34,7 @@
#include "platform/PopupMenuStyle.h"
#include "platform/geometry/FloatQuad.h"
+#include "platform/heap/Handle.h"
#include "web/PopupListBox.h"
namespace blink {
@@ -49,7 +50,7 @@ struct WebPopupMenuInfo;
// FIXME(skobes): This class can probably be combined with PopupListBox.
class PopupContainer final : public Widget {
public:
- static PassRefPtr<PopupContainer> create(PopupMenuClient*, bool deviceSupportsTouch);
+ static PassRefPtrWillBeRawPtr<PopupContainer> create(PopupMenuClient*, bool deviceSupportsTouch);
// Whether a key event should be sent to this popup.
bool isInterestedInEventForKey(int keyCode);
@@ -124,6 +125,8 @@ public:
void updateFromElement() { m_listBox->updateFromElement(); }
+ virtual void trace(Visitor*) OVERRIDE;
+
private:
friend class WTF::RefCounted<PopupContainer>;
@@ -144,8 +147,8 @@ private:
// Returns the ChromeClient of the page this popup is associated with.
ChromeClient& chromeClient();
- RefPtr<PopupListBox> m_listBox;
- RefPtr<FrameView> m_frameView;
+ RefPtrWillBeMember<PopupListBox> m_listBox;
+ RefPtrWillBeMember<FrameView> m_frameView;
// m_controlPosition contains the transformed position of the
// <select>/<input> associated with this popup. m_controlSize is the size

Powered by Google App Engine
This is Rietveld 408576698