Cleanup default FWL params part I

xfa/fwl/core/cfwl_pushbutton.cpp

Index: xfa/fwl/core/cfwl_pushbutton.cpp
diff --git a/xfa/fwl/core/cfwl_pushbutton.cpp b/xfa/fwl/core/cfwl_pushbutton.cpp
index 5f0e2fb17c2e6982eceddc76352bbbf866181d15..71011c4420b1566a1291312ebac7ca03a0976487 100644
--- a/xfa/fwl/core/cfwl_pushbutton.cpp
+++ b/xfa/fwl/core/cfwl_pushbutton.cpp
@@ -51,12 +51,12 @@ void CFWL_PushButton::GetWidgetRect(CFX_RectF& rect, bool bAutoSize) {
   CFWL_Widget::GetWidgetRect(rect, true);
 }
 
-void CFWL_PushButton::SetStates(uint32_t dwStates, bool bSet) {
-  if ((dwStates & FWL_WGTSTATE_Disabled) && bSet) {
+void CFWL_PushButton::SetStates(uint32_t dwStates) {

[Tom Sepez, 2016/12/06 22:38:26]

sure this never got called as false?

[dsinclair, 2016/12/07 15:03:27]

That call would have been changed to a RemoveState at the call site. This code
doesn't do anything with a false other then proxy through to
CFWL_Widget::SetStates

+  if (dwStates & FWL_WGTSTATE_Disabled) {
     m_pProperties->m_dwStates = FWL_WGTSTATE_Disabled;
     return;
   }
-  CFWL_Widget::SetStates(dwStates, bSet);
+  CFWL_Widget::SetStates(dwStates);
 }
 
 void CFWL_PushButton::Update() {