Oilpan: Prepare moving DateTimeChooser to Oilpan

Source/core/html/shadow/PickerIndicatorElement.cpp

 /*
  * Copyright (C) 2012 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 44 matching lines @@
 PassRefPtrWillBeRawPtr<PickerIndicatorElement> PickerIndicatorElement::create(Document& document, PickerIndicatorOwner& pickerIndicatorOwner)
 {
     RefPtrWillBeRawPtr<PickerIndicatorElement> element = adoptRefWillBeRefCountedGarbageCollected(new PickerIndicatorElement(document, pickerIndicatorOwner));
     element->setShadowPseudoId(AtomicString("-webkit-calendar-picker-indicator", AtomicString::ConstructFromLiteral));
     element->setAttribute(idAttr, ShadowElementNames::pickerIndicator());
     return element.release();
 }
 
 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*)
 {
     return new RenderDetailsMarker(this);
 }
 
 void PickerIndicatorElement::defaultEventHandler(Event* event)
 {
     if (!renderer())
@@ skipping 70 matching lines @@
 }
 
 bool PickerIndicatorElement::isPickerIndicatorElement() const
 {
     return true;
 }
 
 void PickerIndicatorElement::trace(Visitor* visitor)
 {
     visitor->trace(m_pickerIndicatorOwner);
+    visitor->trace(m_chooser);
     HTMLDivElement::trace(visitor);
+    DateTimeChooserClient::trace(visitor);
 }
 
 }
 
 #endif