Oilpan: Prepare moving DateTimeChooser to Oilpan

Source/web/DateTimeChooserImpl.h

Index: Source/web/DateTimeChooserImpl.h
diff --git a/Source/web/DateTimeChooserImpl.h b/Source/web/DateTimeChooserImpl.h
index f43b3d41f6e08c716c480c38813c06ebdf459cae..c66af8588c9aa3c5eaae9578d794efc65a516114 100644
--- a/Source/web/DateTimeChooserImpl.h
+++ b/Source/web/DateTimeChooserImpl.h
@@ -46,8 +46,9 @@ class ChromeClientImpl;
 
 class DateTimeChooserImpl FINAL : public WebCore::DateTimeChooser, public WebCore::PagePopupClient {
 public:
-    static PassRefPtr<DateTimeChooserImpl> create(ChromeClientImpl*, WebCore::DateTimeChooserClient*, const WebCore::DateTimeChooserParameters&);
+    static PassRefPtrWillBeRawPtr<DateTimeChooserImpl> create(ChromeClientImpl*, WebCore::DateTimeChooserClient*, const WebCore::DateTimeChooserParameters&);
     virtual ~DateTimeChooserImpl();
+    virtual void trace(WebCore::Visitor*) OVERRIDE;
 
     // DateTimeChooser functions:
     virtual void endChooser() OVERRIDE;
@@ -64,7 +65,7 @@ private:
     virtual void didClosePopup() OVERRIDE;
 
     ChromeClientImpl* m_chromeClient;

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

Just to confirm: This raw pointer is safe because DateTimeChooser shouldn't
outlive ChromeClient, right?

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

Yes.

-    WebCore::DateTimeChooserClient* m_client;
+    RawPtrWillBeMember<WebCore::DateTimeChooserClient> m_client;
     WebCore::PagePopup* m_popup;

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

This raw pointer looks safe since it's explicitly cleared when the popup is
closed.

     WebCore::DateTimeChooserParameters m_parameters;
     OwnPtr<WebCore::Locale> m_locale;