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

Unified Diff: Source/core/html/shadow/PickerIndicatorElement.h

Issue 273653003: Oilpan: Prepare to move PickerIndicatorElement to Oilpan heap. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Removed WILL_BE_USING_GARBAGE_COLLECTED_MIXIN Created 6 years, 7 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
« no previous file with comments | « no previous file | Source/core/html/shadow/PickerIndicatorElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/shadow/PickerIndicatorElement.h
diff --git a/Source/core/html/shadow/PickerIndicatorElement.h b/Source/core/html/shadow/PickerIndicatorElement.h
index 8b7a5e3d7401f89cbda23dc77a3440b39413a781..961bc311fb1b608895309639a4f6b45651d85e21 100644
--- a/Source/core/html/shadow/PickerIndicatorElement.h
+++ b/Source/core/html/shadow/PickerIndicatorElement.h
@@ -45,7 +45,7 @@ class PickerIndicatorElement FINAL : public HTMLDivElement, public DateTimeChoos
public:
// PickerIndicatorOwner implementer must call removePickerIndicatorOwner when
// it doesn't handle event, e.g. at destruction.
- class PickerIndicatorOwner {
+ class PickerIndicatorOwner : public WillBeGarbageCollectedMixin {
public:
virtual ~PickerIndicatorOwner() { }
virtual bool isPickerIndicatorOwnerDisabledOrReadOnly() const = 0;
@@ -55,12 +55,14 @@ public:
virtual bool setupDateTimeChooserParameters(DateTimeChooserParameters&) = 0;
};
- static PassRefPtr<PickerIndicatorElement> create(Document&, PickerIndicatorOwner&);
+ static PassRefPtrWillBeRawPtr<PickerIndicatorElement> create(Document&, PickerIndicatorOwner&);
virtual ~PickerIndicatorElement();
+ virtual void trace(Visitor*) OVERRIDE;
+
void openPopup();
void closePopup();
virtual bool willRespondToMouseClickEvents() OVERRIDE;
- void removePickerIndicatorOwner() { m_pickerIndicatorOwner = 0; }
+ void removePickerIndicatorOwner() { m_pickerIndicatorOwner = nullptr; }
// DateTimeChooserClient implementation.
virtual void didChooseValue(const String&) OVERRIDE;
@@ -76,7 +78,7 @@ private:
HTMLInputElement* hostInput();
- PickerIndicatorOwner* m_pickerIndicatorOwner;
+ RawPtrWillBeMember<PickerIndicatorOwner> m_pickerIndicatorOwner;
RefPtr<DateTimeChooser> m_chooser;
};
« no previous file with comments | « no previous file | Source/core/html/shadow/PickerIndicatorElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698