commit f84814b3f512d61f87621948d6dadeaa68d39a15
author Tom Sepez <tsepez@chromium.org> Wed Aug 19 19:44:47 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Wed Aug 19 19:44:47 2020 +0000
tree 16caf9e3fd381cb02e7586e0dc708b2b1b8ae6e9
parent 193cfbd675299e49c04b323e7f6ba402a9306da9

Make ~CFWL_CheckBoxTP() be the default destructor.

There isn't any need to invoke clear on the CXFA_GEPath before
destroying it, because CXFA_GEPath::Clear() just invokes Clear()
on its CFX_PathData, and CFX_PathData::Clear() just invokes
std::vector::clear() on its vector of PoD structs. The vector
cleanup happens automatically as part of destruction.


Change-Id: I9167c281c25925ac5930034e05b1f73dde7f3685
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/72916
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

xfa/fwl/theme/cfwl_checkboxtp.cpp

diff --git a/xfa/fwl/theme/cfwl_checkboxtp.cpp b/xfa/fwl/theme/cfwl_checkboxtp.cpp
index 380b50c..301f47f 100644
--- a/xfa/fwl/theme/cfwl_checkboxtp.cpp
+++ b/xfa/fwl/theme/cfwl_checkboxtp.cpp
@@ -31,10 +31,7 @@
 
 CFWL_CheckBoxTP::CFWL_CheckBoxTP() = default;
 
-CFWL_CheckBoxTP::~CFWL_CheckBoxTP() {
-  if (m_pCheckPath)
-    m_pCheckPath->Clear();
-}
+CFWL_CheckBoxTP::~CFWL_CheckBoxTP() = default;
 
 void CFWL_CheckBoxTP::DrawText(const CFWL_ThemeText& pParams) {
   EnsureTTOInitialized();