diff options
author | dsinclair <dsinclair@chromium.org> | 2016-11-01 18:48:19 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-11-01 18:48:20 -0700 |
commit | 6fe8795d9022105a0061a0a81be9a49d49fda345 (patch) | |
tree | b1fc8d51560fbf0a9e4a163f887e763f565d5234 /xfa/fwl/core/ifwl_datetimecalendar.cpp | |
parent | 919e48d877e503f1ca250bc37324c9f7dc96d7a9 (diff) | |
download | pdfium-6fe8795d9022105a0061a0a81be9a49d49fda345.tar.xz |
Fold IFWL*::{Initialize|Finalize} into constructor/destructor
This Cl moves the ::Initialize method into the constructors and the
::Finalize method into the destructors. The |m_pDelegate| is made private and
a unique_ptr. A |SetDelegate| and |GetDelegate| method are added.
Review-Url: https://codereview.chromium.org/2464703006
Diffstat (limited to 'xfa/fwl/core/ifwl_datetimecalendar.cpp')
-rw-r--r-- | xfa/fwl/core/ifwl_datetimecalendar.cpp | 17 |
1 files changed, 3 insertions, 14 deletions
diff --git a/xfa/fwl/core/ifwl_datetimecalendar.cpp b/xfa/fwl/core/ifwl_datetimecalendar.cpp index ee04bf4978..a44b558190 100644 --- a/xfa/fwl/core/ifwl_datetimecalendar.cpp +++ b/xfa/fwl/core/ifwl_datetimecalendar.cpp @@ -6,6 +6,7 @@ #include "xfa/fwl/core/ifwl_datetimecalendar.h" +#include "third_party/base/ptr_util.h" #include "xfa/fwl/core/cfwl_widgetmgr.h" #include "xfa/fwl/core/ifwl_datetimepicker.h" #include "xfa/fwl/core/ifwl_formproxy.h" @@ -14,20 +15,8 @@ IFWL_DateTimeCalendar::IFWL_DateTimeCalendar( const IFWL_App* app, const CFWL_WidgetImpProperties& properties, IFWL_Widget* pOuter) - : IFWL_MonthCalendar(app, properties, pOuter) {} - -void IFWL_DateTimeCalendar::Initialize() { - IFWL_MonthCalendar::Initialize(); - - // Delete delegated set by IFWL_MonthCalendar::Initialize. - delete m_pDelegate; - m_pDelegate = new CFWL_DateTimeCalendarImpDelegate(this); -} - -void IFWL_DateTimeCalendar::Finalize() { - delete m_pDelegate; - m_pDelegate = nullptr; - IFWL_MonthCalendar::Finalize(); + : IFWL_MonthCalendar(app, properties, pOuter) { + SetDelegate(pdfium::MakeUnique<CFWL_DateTimeCalendarImpDelegate>(this)); } CFWL_DateTimeCalendarImpDelegate::CFWL_DateTimeCalendarImpDelegate( |