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/pwl/cpwl_scroll_bar.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/pwl/cpwl_scroll_bar.cpp')
-rw-r--r-- | fpdfsdk/pwl/cpwl_scroll_bar.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/fpdfsdk/pwl/cpwl_scroll_bar.cpp b/fpdfsdk/pwl/cpwl_scroll_bar.cpp index ea38b0f591..15247293a0 100644 --- a/fpdfsdk/pwl/cpwl_scroll_bar.cpp +++ b/fpdfsdk/pwl/cpwl_scroll_bar.cpp @@ -131,8 +131,8 @@ CFX_ByteString CPWL_SBButton::GetClassName() const { return "CPWL_SBButton"; } -void CPWL_SBButton::OnCreate(PWL_CREATEPARAM& cp) { - cp.eCursorType = FXCT_ARROW; +void CPWL_SBButton::OnCreate(CreateParams* pParamsToAdjust) { + pParamsToAdjust->eCursorType = FXCT_ARROW; } void CPWL_SBButton::DrawThisAppearance(CFX_RenderDevice* pDevice, @@ -329,8 +329,8 @@ CFX_ByteString CPWL_ScrollBar::GetClassName() const { return "CPWL_ScrollBar"; } -void CPWL_ScrollBar::OnCreate(PWL_CREATEPARAM& cp) { - cp.eCursorType = FXCT_ARROW; +void CPWL_ScrollBar::OnCreate(CreateParams* pParamsToAdjust) { + pParamsToAdjust->eCursorType = FXCT_ARROW; } void CPWL_ScrollBar::OnDestroy() { @@ -539,8 +539,8 @@ void CPWL_ScrollBar::NotifyMouseMove(CPWL_Wnd* child, const CFX_PointF& pos) { OnPosButtonMouseMove(pos); } -void CPWL_ScrollBar::CreateButtons(const PWL_CREATEPARAM& cp) { - PWL_CREATEPARAM scp = cp; +void CPWL_ScrollBar::CreateButtons(const CreateParams& cp) { + CreateParams scp = cp; scp.pParentWnd = this; scp.dwBorderWidth = 2; scp.nBorderStyle = BorderStyle::BEVELED; @@ -863,7 +863,7 @@ float CPWL_ScrollBar::FaceToTrue(float fFace) { return fTrue; } -void CPWL_ScrollBar::CreateChildWnd(const PWL_CREATEPARAM& cp) { +void CPWL_ScrollBar::CreateChildWnd(const CreateParams& cp) { CreateButtons(cp); } |