Oilpan: Prepare moving DateTimeChooser to Oilpan

Source/core/html/shadow/PickerIndicatorElement.cpp

Index: Source/core/html/shadow/PickerIndicatorElement.cpp
diff --git a/Source/core/html/shadow/PickerIndicatorElement.cpp b/Source/core/html/shadow/PickerIndicatorElement.cpp
index 7c9806dc39802e6876d33b13fbd3904840fd5730..94a5123c7bdae4ca85bb600784d7244be2341543 100644
--- a/Source/core/html/shadow/PickerIndicatorElement.cpp
+++ b/Source/core/html/shadow/PickerIndicatorElement.cpp
@@ -62,8 +62,6 @@ PassRefPtrWillBeRawPtr<PickerIndicatorElement> PickerIndicatorElement::create(Do
 
 PickerIndicatorElement::~PickerIndicatorElement()
 {
-    closePopup();
-    ASSERT(!m_chooser);

[haraken, 2014/06/05 04:59:25]

Shall we leave this ASSERT in non-oilpan builds? It's up to you.

[keishi, 2014/06/05 08:06:39]

This ASSERT is working because PickerIndicatorElement is only used on Desktop
clients which don't use ipc to open the date time chooser. However
DateTimeChooserClient does not guarantee this behavior so I am removing it.

 }
 
 RenderObject* PickerIndicatorElement::createRenderer(RenderStyle*)
@@ -154,7 +152,9 @@ bool PickerIndicatorElement::isPickerIndicatorElement() const
 void PickerIndicatorElement::trace(Visitor* visitor)
 {
     visitor->trace(m_pickerIndicatorOwner);
+    visitor->trace(m_chooser);
     HTMLDivElement::trace(visitor);
+    DateTimeChooserClient::trace(visitor);
 }
 
 }