Index: xfa/fxfa/app/xfa_ffchoicelist.cpp |
diff --git a/xfa/fxfa/app/xfa_ffchoicelist.cpp b/xfa/fxfa/app/xfa_ffchoicelist.cpp |
index d4cfe07202215975447904f046d10bf834c14527..e8c364a6130be03453314e8f2b5ba3c6b65e407d 100644 |
--- a/xfa/fxfa/app/xfa_ffchoicelist.cpp |
+++ b/xfa/fxfa/app/xfa_ffchoicelist.cpp |
@@ -227,16 +227,13 @@ bool CXFA_FFComboBox::GetBBox(CFX_RectF& rtBox, |
} |
bool CXFA_FFComboBox::PtInActiveRect(FX_FLOAT fx, FX_FLOAT fy) { |
- if (!m_pNormalWidget) { |
+ if (!m_pNormalWidget) |
return false; |
- } |
- CFX_RectF rtWidget; |
- ((CFWL_ComboBox*)m_pNormalWidget)->GetBBox(rtWidget); |
- if (rtWidget.Contains(fx, fy)) { |
- return true; |
- } |
- return false; |
+ return static_cast<CFWL_ComboBox*>(m_pNormalWidget) |
+ ->GetBBox() |
+ .Contains(fx, fy); |
} |
+ |
bool CXFA_FFComboBox::LoadWidget() { |
CFWL_ComboBox* pComboBox = new CFWL_ComboBox(GetFWLApp()); |
m_pNormalWidget = (CFWL_Widget*)pComboBox; |
@@ -321,12 +318,11 @@ bool CXFA_FFComboBox::IsDataChanged() { |
CFX_WideString wsText = pFWLcombobox->GetEditText(); |
int32_t iCursel = pFWLcombobox->GetCurSel(); |
if (iCursel >= 0) { |
- CFX_WideString wsSel; |
- pFWLcombobox->GetTextByIndex(iCursel, wsSel); |
- if (wsSel == wsText) { |
+ CFX_WideString wsSel = pFWLcombobox->GetTextByIndex(iCursel); |
+ if (wsSel == wsText) |
m_pDataAcc->GetChoiceListItem(wsText, iCursel, true); |
- } |
} |
+ |
CFX_WideString wsOldValue; |
m_pDataAcc->GetValue(wsOldValue, XFA_VALUEPICTURE_Raw); |
if (wsOldValue != wsText) { |