Cleanup CFWL_ThemePart data.

xfa/fwl/theme/cfwl_formtp.cpp

Index: xfa/fwl/theme/cfwl_formtp.cpp
diff --git a/xfa/fwl/theme/cfwl_formtp.cpp b/xfa/fwl/theme/cfwl_formtp.cpp
index 4a18ea0cc40ab41383641f16a94debab8f6d58e9..9ee78ec33d11d54270f1fad7271f7c2ca35b0ba1 100644
--- a/xfa/fwl/theme/cfwl_formtp.cpp
+++ b/xfa/fwl/theme/cfwl_formtp.cpp
@@ -133,7 +133,7 @@ FX_BOOL CFWL_FormTP::DrawBackground(CFWL_ThemeBackground* pParams) {
     }
     case CFWL_Part::MaximizeBox: {
       DrawMaximizeBox(pParams->m_pGraphics, &pParams->m_rtPart, eState,
-                      pParams->m_dwData, &pParams->m_matrix, iActive);
+                      pParams->m_bMaximize, &pParams->m_matrix, iActive);

[dsinclair, 2016/05/04 16:31:01]

This maybe incorrect. Should I be setting maximize true everywhere we assigned
the pointer previously?

[Tom Sepez, 2016/05/04 17:41:18]

Heh, except when the stack was aligned to a 32bit boundary.  Might as well do
that, also the case where we set it to "i" in the loop.  

[dsinclair, 2016/05/04 18:07:39]

Done.

       break;
     }
     case CFWL_Part::Icon: {
@@ -519,7 +519,7 @@ void CFWL_FormTP::DrawMinimizeBox(CFX_Graphics* pGraphics,
 void CFWL_FormTP::DrawMaximizeBox(CFX_Graphics* pGraphics,
                                   const CFX_RectF* pRect,
                                   FWLTHEME_STATE eState,
-                                  FX_BOOL bMax,
+                                  bool bMax,
                                   CFX_Matrix* pMatrix,
                                   int32_t iActive) {
   DrawMinMaxBoxCommon(pGraphics, pRect, eState, pMatrix);