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_pushbutton.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_pushbutton.cpp')
| -rw-r--r-- | fpdfsdk/formfiller/cffl_pushbutton.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fpdfsdk/formfiller/cffl_pushbutton.cpp b/fpdfsdk/formfiller/cffl_pushbutton.cpp index f09a8f7d8d..7310da6761 100644 --- a/fpdfsdk/formfiller/cffl_pushbutton.cpp +++ b/fpdfsdk/formfiller/cffl_pushbutton.cpp @@ -15,9 +15,8 @@ CFFL_PushButton::CFFL_PushButton(CPDFSDK_FormFillEnvironment* pApp, CFFL_PushButton::~CFFL_PushButton() {} -CPWL_Wnd* CFFL_PushButton::NewPDFWindow(const PWL_CREATEPARAM& cp) { - CPWL_PushButton* pWnd = new CPWL_PushButton(); +CPWL_Wnd* CFFL_PushButton::NewPDFWindow(const CPWL_Wnd::CreateParams& cp) { + auto* pWnd = new CPWL_PushButton(); pWnd->Create(cp); - return pWnd; } |