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_pushbuttontp.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_pushbuttontp.cpp')
| -rw-r--r-- | xfa/fwl/theme/cfwl_pushbuttontp.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xfa/fwl/theme/cfwl_pushbuttontp.cpp b/xfa/fwl/theme/cfwl_pushbuttontp.cpp index 17c02aef44..cf9991e0f4 100644 --- a/xfa/fwl/theme/cfwl_pushbuttontp.cpp +++ b/xfa/fwl/theme/cfwl_pushbuttontp.cpp @@ -19,9 +19,7 @@ CFWL_PushButtonTP::CFWL_PushButtonTP() : m_pThemeData(new PBThemeData) { SetThemeData(0); } -CFWL_PushButtonTP::~CFWL_PushButtonTP() { - delete m_pThemeData; -} +CFWL_PushButtonTP::~CFWL_PushButtonTP() {} bool CFWL_PushButtonTP::IsValidWidget(IFWL_Widget* pWidget) { return pWidget && pWidget->GetClassID() == FWL_Type::PushButton; @@ -155,7 +153,7 @@ void CFWL_PushButtonTP::SetThemeData(uint32_t dwID) { m_pThemeData->clrFill[4] = ArgbEncode(255, 245, 244, 234); } } -int32_t CFWL_PushButtonTP::GetColorID(uint32_t dwStates) { +int32_t CFWL_PushButtonTP::GetColorID(uint32_t dwStates) const { int32_t color = 0; if (dwStates & CFWL_PartState_Disabled) color += 4; |