commit 8b6b33c3b8fc02eefeca1a0b1e66ec1e71d0a81e
author Tom Sepez <tsepez@chromium.org> Tue Jan 29 20:30:18 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue Jan 29 20:30:18 2019 +0000
tree 70b67519d3211f16b9edd529a4d8bd89bdde075b
parent 7eeb78e285b6d8f2a072540e0a6308f22b4dd1d1

Remove unused FWL stretch handler mechanism.

CFWL_Combobox::DrawStretchHandler() is uncalled. Removing that
makes CFWL_Part::StrechHandler unused. Removing that makes
CFWL_ComboboxTP:::DrawStretchHandler() unused.
Change-Id: If1d39b883d423676020f62acb5077da123e8af93
Reviewed-on: https://pdfium-review.googlesource.com/c/49393
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

xfa/fwl/cfwl_combobox.cpp

diff --git a/xfa/fwl/cfwl_combobox.cpp b/xfa/fwl/cfwl_combobox.cpp
index 84f1be3..4760ab2 100644
--- a/xfa/fwl/cfwl_combobox.cpp
+++ b/xfa/fwl/cfwl_combobox.cpp
@@ -213,18 +213,6 @@
     m_pEdit->ModifyStylesEx(dwStylesExAdded, dwStylesExRemoved);
 }
 
-void CFWL_ComboBox::DrawStretchHandler(CXFA_Graphics* pGraphics,
-                                       const CFX_Matrix* pMatrix) {
-  CFWL_ThemeBackground param;
-  param.m_pGraphics = pGraphics;
-  param.m_iPart = CFWL_Part::StretchHandler;
-  param.m_dwStates = CFWL_PartState_Normal;
-  param.m_pWidget = this;
-  if (pMatrix)
-    param.m_matrix.Concat(*pMatrix);
-  m_pProperties->m_pThemeProvider->DrawBackground(param);
-}
-
 void CFWL_ComboBox::ShowDropList(bool bActivate) {
   if (IsDropListVisible() == bActivate)
     return;

xfa/fwl/cfwl_combobox.h

diff --git a/xfa/fwl/cfwl_combobox.h b/xfa/fwl/cfwl_combobox.h
index c9e092f..4142b83 100644
--- a/xfa/fwl/cfwl_combobox.h
+++ b/xfa/fwl/cfwl_combobox.h
@@ -88,8 +88,6 @@
 
   CFX_RectF GetBBox() const;
   void EditModifyStylesEx(uint32_t dwStylesExAdded, uint32_t dwStylesExRemoved);
-
-  void DrawStretchHandler(CXFA_Graphics* pGraphics, const CFX_Matrix* pMatrix);
   void ShowDropList(bool bActivate);
 
   CFWL_ComboEdit* GetComboEdit() const { return m_pEdit.get(); }

xfa/fwl/cfwl_themepart.h

diff --git a/xfa/fwl/cfwl_themepart.h b/xfa/fwl/cfwl_themepart.h
index 942fd71..46be205 100644
--- a/xfa/fwl/cfwl_themepart.h
+++ b/xfa/fwl/cfwl_themepart.h
@@ -40,7 +40,6 @@
   MaximizeBox,
   NarrowCaption,
   RBtn,
-  StretchHandler,
   Thumb,
   ThumbBackArrow,
   ThumbForeArrow,

xfa/fwl/theme/cfwl_comboboxtp.cpp

diff --git a/xfa/fwl/theme/cfwl_comboboxtp.cpp b/xfa/fwl/theme/cfwl_comboboxtp.cpp
index ff179b8..6876965 100644
--- a/xfa/fwl/theme/cfwl_comboboxtp.cpp
+++ b/xfa/fwl/theme/cfwl_comboboxtp.cpp
@@ -48,25 +48,11 @@
       DrawDropDownButton(pParams, pParams.m_dwStates, pParams.m_matrix);
       break;
     }
-    case CFWL_Part::StretchHandler: {
-      DrawStretchHandler(pParams, 0, pParams.m_matrix);
-      break;
-    }
     default:
       break;
   }
 }
 
-void CFWL_ComboBoxTP::DrawStretchHandler(const CFWL_ThemeBackground& pParams,
-                                         uint32_t dwStates,
-                                         const CFX_Matrix& matrix) {
-  CXFA_GEPath path;
-  path.AddRectangle(pParams.m_rtPart.left, pParams.m_rtPart.top,
-                    pParams.m_rtPart.width - 1, pParams.m_rtPart.height);
-  pParams.m_pGraphics->SetFillColor(CXFA_GEColor(ArgbEncode(0xff, 0xff, 0, 0)));
-  pParams.m_pGraphics->FillPath(&path, FXFILL_WINDING, &pParams.m_matrix);
-}
-
 void CFWL_ComboBoxTP::DrawDropDownButton(const CFWL_ThemeBackground& pParams,
                                          uint32_t dwStates,
                                          const CFX_Matrix& matrix) {

xfa/fwl/theme/cfwl_comboboxtp.h

diff --git a/xfa/fwl/theme/cfwl_comboboxtp.h b/xfa/fwl/theme/cfwl_comboboxtp.h
index 0be02aa..b3a75a5 100644
--- a/xfa/fwl/theme/cfwl_comboboxtp.h
+++ b/xfa/fwl/theme/cfwl_comboboxtp.h
@@ -21,9 +21,6 @@
   void DrawDropDownButton(const CFWL_ThemeBackground& pParams,
                           uint32_t dwStates,
                           const CFX_Matrix& matrix);
-  void DrawStretchHandler(const CFWL_ThemeBackground& pParams,
-                          uint32_t dwStates,
-                          const CFX_Matrix& matrix);
 };
 
 #endif  // XFA_FWL_THEME_CFWL_COMBOBOXTP_H_