Oilpan: Prepare moving DateTimeChooser to Oilpan

Source/web/DateTimeChooserImpl.h

 /*
  * 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 28 matching lines @@
 class PagePopup;
 class DateTimeChooserClient;
 }
 
 namespace blink {
 
 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;
 
 private:
     DateTimeChooserImpl(ChromeClientImpl*, WebCore::DateTimeChooserClient*, const WebCore::DateTimeChooserParameters&);
     // PagePopupClient functions:
     virtual WebCore::IntSize contentSize() OVERRIDE;
     virtual void writeDocument(WebCore::SharedBuffer*) OVERRIDE;
     virtual WebCore::Locale& locale() OVERRIDE;
     virtual void setValueAndClosePopup(int, const String&) OVERRIDE;
     virtual void setValue(const String&) OVERRIDE;
     virtual void closePopup() OVERRIDE;
     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;
 };
 
 }
 
 #endif // ENABLE(INPUT_MULTIPLE_FIELDS_UI)
 
 #endif // DateTimeChooserImpl_h