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/cfwl_datetimepicker.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/cfwl_datetimepicker.cpp')
| -rw-r--r-- | xfa/fwl/core/cfwl_datetimepicker.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/xfa/fwl/core/cfwl_datetimepicker.cpp b/xfa/fwl/core/cfwl_datetimepicker.cpp index f794317279..973e06ea4b 100644 --- a/xfa/fwl/core/cfwl_datetimepicker.cpp +++ b/xfa/fwl/core/cfwl_datetimepicker.cpp @@ -8,6 +8,7 @@ #include <memory> +#include "third_party/base/ptr_util.h" #include "xfa/fwl/core/fwl_error.h" #include "xfa/fwl/core/ifwl_datetimepicker.h" #include "xfa/fwl/core/ifwl_widget.h" @@ -28,11 +29,9 @@ CFWL_DateTimePicker::~CFWL_DateTimePicker() {} void CFWL_DateTimePicker::Initialize() { ASSERT(!m_pIface); - std::unique_ptr<IFWL_DateTimePicker> pDateTimePicker(new IFWL_DateTimePicker( - m_pApp, m_pProperties->MakeWidgetImpProperties(&m_DateTimePickerDP))); - pDateTimePicker->Initialize(); + m_pIface = pdfium::MakeUnique<IFWL_DateTimePicker>( + m_pApp, m_pProperties->MakeWidgetImpProperties(&m_DateTimePickerDP)); - m_pIface = std::move(pDateTimePicker); CFWL_Widget::Initialize(); } |