diff options
| author | Tom Sepez <tsepez@chromium.org> | 2017-09-15 13:26:32 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-09-15 21:25:46 +0000 |
| commit | bf15730b2c9577d4efd2124d84c6c8ccc5f66b43 (patch) | |
| tree | b03fb50d97a3233094437746260c0927464be5ec /fpdfsdk/formfiller/cffl_formfiller.cpp | |
| parent | bb2f7e73bef47b4e865118cb52170563d849b4de (diff) | |
| download | pdfium-bf15730b2c9577d4efd2124d84c6c8ccc5f66b43.tar.xz | |
Move CPWL_CREATPARAM to CPWL_Wnd::CreateParams
This nesting makes so much more sense when I read the code.
Fix member naming for this in CPWL_Wnd.
Pass in/out CreateParams arg as pointer.
Kill Reset() method, only called during destroy and nothing needs
explicit cleanup.
Change-Id: If50e403e11c131e7656da09c0cb36866223d64c6
Reviewed-on: https://pdfium-review.googlesource.com/14070
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/formfiller/cffl_formfiller.cpp')
| -rw-r--r-- | fpdfsdk/formfiller/cffl_formfiller.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fpdfsdk/formfiller/cffl_formfiller.cpp b/fpdfsdk/formfiller/cffl_formfiller.cpp index e2a8c4e931..f405e3578d 100644 --- a/fpdfsdk/formfiller/cffl_formfiller.cpp +++ b/fpdfsdk/formfiller/cffl_formfiller.cpp @@ -299,8 +299,8 @@ bool CFFL_FormFiller::IsValid() const { return m_bValid; } -PWL_CREATEPARAM CFFL_FormFiller::GetCreateParam() { - PWL_CREATEPARAM cp; +CPWL_Wnd::CreateParams CFFL_FormFiller::GetCreateParam() { + CPWL_Wnd::CreateParams cp; cp.pParentWnd = nullptr; cp.pProvider.Reset(this); cp.rcRectWnd = GetPDFWindowRect(); @@ -356,7 +356,7 @@ CPWL_Wnd* CFFL_FormFiller::GetPDFWindow(CPDFSDK_PageView* pPageView, return pWnd; if (!found) { - PWL_CREATEPARAM cp = GetCreateParam(); + CPWL_Wnd::CreateParams cp = GetCreateParam(); cp.pAttachedWidget.Reset(m_pWidget.Get()); CFFL_PrivateData* pPrivateData = new CFFL_PrivateData; |