diff options
| author | weili <weili@chromium.org> | 2016-08-10 11:29:57 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-08-10 11:29:57 -0700 |
| commit | 4e62b5c7c5a35c7020d581fb65de4b3fc5ad3c1a (patch) | |
| tree | b3a46fd89073a9935bb3821d5f2638effc9617d7 /xfa/fwl/theme/cfwl_monthcalendartp.cpp | |
| parent | d83842e0b9ef6e25ab28642e2a146159ac9f3596 (diff) | |
| download | pdfium-4e62b5c7c5a35c7020d581fb65de4b3fc5ad3c1a.tar.xz | |
Use smart pointers for class owned pointers in xfa/fwl/theme
Use smart pointers instead of raw pointer to make memory management
easier for classes under xfa/fwl/theme.
BUG=pdfium:518
Review-Url: https://codereview.chromium.org/2230813002
Diffstat (limited to 'xfa/fwl/theme/cfwl_monthcalendartp.cpp')
| -rw-r--r-- | xfa/fwl/theme/cfwl_monthcalendartp.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/xfa/fwl/theme/cfwl_monthcalendartp.cpp b/xfa/fwl/theme/cfwl_monthcalendartp.cpp index d04ebb16d2..7faa8e1533 100644 --- a/xfa/fwl/theme/cfwl_monthcalendartp.cpp +++ b/xfa/fwl/theme/cfwl_monthcalendartp.cpp @@ -49,26 +49,25 @@ const int kSeparatorY = kHeaderHeight + kDatesCellHeight + kSeparatorDOffset; } // namespace -CFWL_MonthCalendarTP::CFWL_MonthCalendarTP() { - m_pThemeData = new MCThemeData; +CFWL_MonthCalendarTP::CFWL_MonthCalendarTP() : m_pThemeData(new MCThemeData) { SetThemeData(0); } -CFWL_MonthCalendarTP::~CFWL_MonthCalendarTP() { - delete m_pThemeData; -} +CFWL_MonthCalendarTP::~CFWL_MonthCalendarTP() {} bool CFWL_MonthCalendarTP::IsValidWidget(IFWL_Widget* pWidget) { return pWidget && pWidget->GetClassID() == FWL_Type::MonthCalendar; } + uint32_t CFWL_MonthCalendarTP::SetThemeID(IFWL_Widget* pWidget, uint32_t dwThemeID, FX_BOOL bChildren) { - if (m_pThemeData) { + if (m_pThemeData) SetThemeData(FWL_GetThemeColor(dwThemeID)); - } + return CFWL_WidgetTP::SetThemeID(pWidget, dwThemeID, bChildren); } + FX_BOOL CFWL_MonthCalendarTP::DrawBackground(CFWL_ThemeBackground* pParams) { if (!pParams) return FALSE; |