diff options
author | dsinclair <dsinclair@chromium.org> | 2016-11-01 08:33:02 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-11-01 08:33:02 -0700 |
commit | 9ec22175f945d227e71c459e7e0b7c464159c18b (patch) | |
tree | 1c4836aa35a9e8a1da1761562088b1f9df6aa94f /xfa/fwl/core/ifwl_form.cpp | |
parent | 5a22582ca1dbe1d66b18d5253bfa1d202fcc3b4a (diff) | |
download | pdfium-9ec22175f945d227e71c459e7e0b7c464159c18b.tar.xz |
Rename IFWL_Widget::SetDelegate
The ::SetDelegate method was misleading. The primary use was
SetDelegate(nullptr) which returned the current delegate and didn't actually
set anything. When a value was passed it would set the
|m_pCurDelegate| not the |m_pDelegate|.
This Cl breaks ::SetDelegate into ::GetCurrentDelegate and ::SetCurrentDelegate
to make it clear what is happening and that this does not effect the
|m_pDelegate| variable.
Review-Url: https://codereview.chromium.org/2459423003
Diffstat (limited to 'xfa/fwl/core/ifwl_form.cpp')
-rw-r--r-- | xfa/fwl/core/ifwl_form.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/xfa/fwl/core/ifwl_form.cpp b/xfa/fwl/core/ifwl_form.cpp index e46328b634..2ded30e6d6 100644 --- a/xfa/fwl/core/ifwl_form.cpp +++ b/xfa/fwl/core/ifwl_form.cpp @@ -830,10 +830,11 @@ void CFWL_FormImpDelegate::OnProcessMessage(CFWL_Message* pMessage) { if (pSubFocus == pDriver->GetFocus()) { pDriver->SetFocus(nullptr); } else if (pSubFocus->GetStates() & FWL_WGTSTATE_Focused) { - CFWL_MsgKillFocus ms; - IFWL_WidgetDelegate* pDelegate = pSubFocus->SetDelegate(nullptr); - if (pDelegate) + if (IFWL_WidgetDelegate* pDelegate = + pSubFocus->GetCurrentDelegate()) { + CFWL_MsgKillFocus ms; pDelegate->OnProcessMessage(&ms); + } } } m_pOwner->Repaint(&m_pOwner->m_rtRelative); |