diff options
author | Tom Sepez <tsepez@chromium.org> | 2018-10-26 23:56:27 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-26 23:56:27 +0000 |
commit | b7c4a0243e82f6c4ff91cd90b5bae336100c3c70 (patch) | |
tree | 782b8b225dbc01d78c0dff7b126f90f606d13fb1 /fpdfsdk/formfiller | |
parent | 314380e271d0a1e3117c67cb5a417cbc5b5b81b3 (diff) | |
download | pdfium-b7c4a0243e82f6c4ff91cd90b5bae336100c3c70.tar.xz |
Pass CreateParams to CPWL_Wnd constructor.chromium/3595chromium/3594
The previous reshuffling means we always have the create params
earlier, so pass them in at create time.
Clean up two forward declarations that were not needed as
noticed in the process.
Change-Id: I5d3861f3c1e3508e0d25950e919859244fac5f8b
Reviewed-on: https://pdfium-review.googlesource.com/c/44710
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/formfiller')
-rw-r--r-- | fpdfsdk/formfiller/cffl_checkbox.cpp | 4 | ||||
-rw-r--r-- | fpdfsdk/formfiller/cffl_combobox.cpp | 4 | ||||
-rw-r--r-- | fpdfsdk/formfiller/cffl_listbox.cpp | 4 | ||||
-rw-r--r-- | fpdfsdk/formfiller/cffl_pushbutton.cpp | 4 | ||||
-rw-r--r-- | fpdfsdk/formfiller/cffl_radiobutton.cpp | 5 | ||||
-rw-r--r-- | fpdfsdk/formfiller/cffl_textfield.cpp | 4 |
6 files changed, 13 insertions, 12 deletions
diff --git a/fpdfsdk/formfiller/cffl_checkbox.cpp b/fpdfsdk/formfiller/cffl_checkbox.cpp index fbf6c34bf8..a40cfc6143 100644 --- a/fpdfsdk/formfiller/cffl_checkbox.cpp +++ b/fpdfsdk/formfiller/cffl_checkbox.cpp @@ -24,8 +24,8 @@ CFFL_CheckBox::~CFFL_CheckBox() {} std::unique_ptr<CPWL_Wnd> CFFL_CheckBox::NewPWLWindow( const CPWL_Wnd::CreateParams& cp, std::unique_ptr<CPWL_Wnd::PrivateData> pAttachedData) { - auto pWnd = pdfium::MakeUnique<CPWL_CheckBox>(std::move(pAttachedData)); - pWnd->Realize(cp); + auto pWnd = pdfium::MakeUnique<CPWL_CheckBox>(cp, std::move(pAttachedData)); + pWnd->Realize(); pWnd->SetCheck(m_pWidget->IsChecked()); return std::move(pWnd); } diff --git a/fpdfsdk/formfiller/cffl_combobox.cpp b/fpdfsdk/formfiller/cffl_combobox.cpp index 358596e526..f163c78887 100644 --- a/fpdfsdk/formfiller/cffl_combobox.cpp +++ b/fpdfsdk/formfiller/cffl_combobox.cpp @@ -47,9 +47,9 @@ CPWL_Wnd::CreateParams CFFL_ComboBox::GetCreateParam() { std::unique_ptr<CPWL_Wnd> CFFL_ComboBox::NewPWLWindow( const CPWL_Wnd::CreateParams& cp, std::unique_ptr<CPWL_Wnd::PrivateData> pAttachedData) { - auto pWnd = pdfium::MakeUnique<CPWL_ComboBox>(std::move(pAttachedData)); + auto pWnd = pdfium::MakeUnique<CPWL_ComboBox>(cp, std::move(pAttachedData)); pWnd->AttachFFLData(this); - pWnd->Realize(cp); + pWnd->Realize(); CFFL_InteractiveFormFiller* pFormFiller = m_pFormFillEnv->GetInteractiveFormFiller(); diff --git a/fpdfsdk/formfiller/cffl_listbox.cpp b/fpdfsdk/formfiller/cffl_listbox.cpp index e601c8305e..0b67ab0ba0 100644 --- a/fpdfsdk/formfiller/cffl_listbox.cpp +++ b/fpdfsdk/formfiller/cffl_listbox.cpp @@ -42,9 +42,9 @@ CPWL_Wnd::CreateParams CFFL_ListBox::GetCreateParam() { std::unique_ptr<CPWL_Wnd> CFFL_ListBox::NewPWLWindow( const CPWL_Wnd::CreateParams& cp, std::unique_ptr<CPWL_Wnd::PrivateData> pAttachedData) { - auto pWnd = pdfium::MakeUnique<CPWL_ListBox>(std::move(pAttachedData)); + auto pWnd = pdfium::MakeUnique<CPWL_ListBox>(cp, std::move(pAttachedData)); pWnd->AttachFFLData(this); - pWnd->Realize(cp); + pWnd->Realize(); pWnd->SetFillerNotify(m_pFormFillEnv->GetInteractiveFormFiller()); for (int32_t i = 0, sz = m_pWidget->CountOptions(); i < sz; i++) diff --git a/fpdfsdk/formfiller/cffl_pushbutton.cpp b/fpdfsdk/formfiller/cffl_pushbutton.cpp index d7a421886c..4f80593973 100644 --- a/fpdfsdk/formfiller/cffl_pushbutton.cpp +++ b/fpdfsdk/formfiller/cffl_pushbutton.cpp @@ -20,7 +20,7 @@ CFFL_PushButton::~CFFL_PushButton() = default; std::unique_ptr<CPWL_Wnd> CFFL_PushButton::NewPWLWindow( const CPWL_Wnd::CreateParams& cp, std::unique_ptr<CPWL_Wnd::PrivateData> pAttachedData) { - auto pWnd = pdfium::MakeUnique<CPWL_PushButton>(std::move(pAttachedData)); - pWnd->Realize(cp); + auto pWnd = pdfium::MakeUnique<CPWL_PushButton>(cp, std::move(pAttachedData)); + pWnd->Realize(); return std::move(pWnd); } diff --git a/fpdfsdk/formfiller/cffl_radiobutton.cpp b/fpdfsdk/formfiller/cffl_radiobutton.cpp index ffe1ee7f2b..c8c65dc9fd 100644 --- a/fpdfsdk/formfiller/cffl_radiobutton.cpp +++ b/fpdfsdk/formfiller/cffl_radiobutton.cpp @@ -23,8 +23,9 @@ CFFL_RadioButton::~CFFL_RadioButton() {} std::unique_ptr<CPWL_Wnd> CFFL_RadioButton::NewPWLWindow( const CPWL_Wnd::CreateParams& cp, std::unique_ptr<CPWL_Wnd::PrivateData> pAttachedData) { - auto pWnd = pdfium::MakeUnique<CPWL_RadioButton>(std::move(pAttachedData)); - pWnd->Realize(cp); + auto pWnd = + pdfium::MakeUnique<CPWL_RadioButton>(cp, std::move(pAttachedData)); + pWnd->Realize(); pWnd->SetCheck(m_pWidget->IsChecked()); return std::move(pWnd); } diff --git a/fpdfsdk/formfiller/cffl_textfield.cpp b/fpdfsdk/formfiller/cffl_textfield.cpp index 79f4f3d3e8..76b2ec1a57 100644 --- a/fpdfsdk/formfiller/cffl_textfield.cpp +++ b/fpdfsdk/formfiller/cffl_textfield.cpp @@ -72,9 +72,9 @@ CPWL_Wnd::CreateParams CFFL_TextField::GetCreateParam() { std::unique_ptr<CPWL_Wnd> CFFL_TextField::NewPWLWindow( const CPWL_Wnd::CreateParams& cp, std::unique_ptr<CPWL_Wnd::PrivateData> pAttachedData) { - auto pWnd = pdfium::MakeUnique<CPWL_Edit>(std::move(pAttachedData)); + auto pWnd = pdfium::MakeUnique<CPWL_Edit>(cp, std::move(pAttachedData)); pWnd->AttachFFLData(this); - pWnd->Realize(cp); + pWnd->Realize(); pWnd->SetFillerNotify(m_pFormFillEnv->GetInteractiveFormFiller()); int32_t nMaxLen = m_pWidget->GetMaxLen(); |