diff options
author | Tom Sepez <tsepez@chromium.org> | 2018-06-27 17:54:50 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-27 17:54:50 +0000 |
commit | d78a964c31f303e828453ee189d4998558bac8e1 (patch) | |
tree | fe74133529c24a276616bbb2e34a0986da1bbe4a /xfa/fxfa/cxfa_ffpushbutton.cpp | |
parent | 9e12f14814722c0c0d46d4968f636b5e1a72a1e7 (diff) | |
download | pdfium-d78a964c31f303e828453ee189d4998558bac8e1.tar.xz |
Remove CXFA_FField::UnloadWidget().
It is only called during the destructor, and does things that
the destructor would automatically do, like calling into the
superclass. So just use destructors.
Change-Id: I1196817798eeca99914b44ddee1f75781e682211
Reviewed-on: https://pdfium-review.googlesource.com/36231
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffpushbutton.cpp')
-rw-r--r-- | xfa/fxfa/cxfa_ffpushbutton.cpp | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/xfa/fxfa/cxfa_ffpushbutton.cpp b/xfa/fxfa/cxfa_ffpushbutton.cpp index 295422dcf3..0753a8349a 100644 --- a/xfa/fxfa/cxfa_ffpushbutton.cpp +++ b/xfa/fxfa/cxfa_ffpushbutton.cpp @@ -28,9 +28,7 @@ CXFA_FFPushButton::CXFA_FFPushButton(CXFA_Node* pNode, CXFA_Button* button) : CXFA_FFField(pNode), button_(button) {} -CXFA_FFPushButton::~CXFA_FFPushButton() { - CXFA_FFPushButton::UnloadWidget(); -} +CXFA_FFPushButton::~CXFA_FFPushButton() = default; void CXFA_FFPushButton::RenderWidget(CXFA_Graphics* pGS, const CFX_Matrix& matrix, @@ -88,15 +86,6 @@ void CXFA_FFPushButton::UpdateWidgetProperty() { m_pNormalWidget->ModifyStylesEx(dwStyleEx, 0xFFFFFFFF); } -void CXFA_FFPushButton::UnloadWidget() { - m_pRolloverTextLayout.reset(); - m_pDownTextLayout.reset(); - m_pRollProvider.reset(); - m_pDownProvider.reset(); - m_pOldDelegate = nullptr; - CXFA_FFField::UnloadWidget(); -} - bool CXFA_FFPushButton::PerformLayout() { CXFA_FFWidget::PerformLayout(); CFX_RectF rtWidget = GetRectWithoutRotate(); |