diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-09-21 15:43:56 -0400 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-09-21 19:57:09 +0000 |
| commit | a5eb9f05b7c3f82630784e043ccf75c4e019b18f (patch) | |
| tree | 5c4adb283ed60a04661b28c6bcbda59aabf2f8fd /xfa/fxfa/cxfa_ffapp.cpp | |
| parent | 65ecca4bc9c8e043b355c459486870ac1518095d (diff) | |
| download | pdfium-a5eb9f05b7c3f82630784e043ccf75c4e019b18f.tar.xz | |
Cleanup WidgetMgr ownership
The CXFA_FFApp can get the FWLWidgetMgr from the CFWL_App and does not
need to store a copy. This removes the need to pass the delegate into
the GetFWLAdapterWidgetMgr() method and we can set the capabilities
directy.
Change-Id: If4794d07fa0fb5dd32b816a21f04e6d7710886ca
Reviewed-on: https://pdfium-review.googlesource.com/14611
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffapp.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffapp.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/xfa/fxfa/cxfa_ffapp.cpp b/xfa/fxfa/cxfa_ffapp.cpp index 7ef266b1e5..8ec23e19c4 100644 --- a/xfa/fxfa/cxfa_ffapp.cpp +++ b/xfa/fxfa/cxfa_ffapp.cpp @@ -78,13 +78,9 @@ CXFA_FWLTheme* CXFA_FFApp::GetFWLTheme() { return m_pFWLTheme.get(); } -CXFA_FWLAdapterWidgetMgr* CXFA_FFApp::GetFWLAdapterWidgetMgr( - CFWL_WidgetMgr* pDelegate) { - if (!m_pAdapterWidgetMgr) { +CXFA_FWLAdapterWidgetMgr* CXFA_FFApp::GetFWLAdapterWidgetMgr() { + if (!m_pAdapterWidgetMgr) m_pAdapterWidgetMgr = pdfium::MakeUnique<CXFA_FWLAdapterWidgetMgr>(); - pDelegate->OnSetCapability(FWL_WGTMGR_DisableForm); - m_pWidgetMgr = pDelegate; - } return m_pAdapterWidgetMgr.get(); } |