Oilpan: Prepare moving DateTimeChooser to Oilpan

Source/core/html/forms/BaseChooserOnlyDateAndTimeInputType.cpp

Index: Source/core/html/forms/BaseChooserOnlyDateAndTimeInputType.cpp
diff --git a/Source/core/html/forms/BaseChooserOnlyDateAndTimeInputType.cpp b/Source/core/html/forms/BaseChooserOnlyDateAndTimeInputType.cpp
index 045527016bac69b2e95d042b35e4035ad601f062..76f32aba3deb5b186a6516e5796adae5da778144 100644
--- a/Source/core/html/forms/BaseChooserOnlyDateAndTimeInputType.cpp
+++ b/Source/core/html/forms/BaseChooserOnlyDateAndTimeInputType.cpp
@@ -39,7 +39,6 @@ namespace WebCore {
 
 BaseChooserOnlyDateAndTimeInputType::~BaseChooserOnlyDateAndTimeInputType()
 {
-    closeDateTimeChooser();

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

Shall we add:

#if !ENABLE(OILPAN)
  ASSERT(!m_dataTimeChooser);
#endif

? I'm not an expert of the forms code, so it's up to you.

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

m_dataTimeChooser is only cleared in didEndChooser() which is called when the
"didEnd" ipc is delivered from the browser process. So this ASSERT won't work.

 }
 
 void BaseChooserOnlyDateAndTimeInputType::handleDOMActivateEvent(Event*)
@@ -142,5 +141,12 @@ void BaseChooserOnlyDateAndTimeInputType::accessKeyAction(bool sendMouseEvents)
     BaseClickableWithKeyInputType::accessKeyAction(element(), sendMouseEvents);
 }
 
+void BaseChooserOnlyDateAndTimeInputType::trace(Visitor* visitor)
+{
+    visitor->trace(m_dateTimeChooser);
+    BaseDateAndTimeInputType::trace(visitor);
+    DateTimeChooserClient::trace(visitor);
+}
+
 }
 #endif