diff options
author | Tom Sepez <tsepez@chromium.org> | 2018-10-04 19:15:30 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-04 19:15:30 +0000 |
commit | 7c40157c9d8d72b91dcde3e79f2a083e4319e2eb (patch) | |
tree | bc70311c8b91540546dc4df2ec8b81938814935e | |
parent | 38772e9f7456021df78eaea3258c63e8c39cf309 (diff) | |
download | pdfium-7c40157c9d8d72b91dcde3e79f2a083e4319e2eb.tar.xz |
Remove CPWL_EditCtrl::m_pEdit null checks.
It is always non-null. Move initializers to header while we're at it.
Change-Id: I64f59ab1d5ffee9256f87e4712e46fb71f14ef08
Reviewed-on: https://pdfium-review.googlesource.com/c/43473
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
-rw-r--r-- | fpdfsdk/pwl/cpwl_edit_ctrl.cpp | 18 | ||||
-rw-r--r-- | fpdfsdk/pwl/cpwl_edit_ctrl.h | 13 |
2 files changed, 9 insertions, 22 deletions
diff --git a/fpdfsdk/pwl/cpwl_edit_ctrl.cpp b/fpdfsdk/pwl/cpwl_edit_ctrl.cpp index 579a587175..33a83cc349 100644 --- a/fpdfsdk/pwl/cpwl_edit_ctrl.cpp +++ b/fpdfsdk/pwl/cpwl_edit_ctrl.cpp @@ -14,14 +14,11 @@ #include "fpdfsdk/pwl/cpwl_scroll_bar.h" #include "fpdfsdk/pwl/cpwl_wnd.h" #include "public/fpdf_fwlevent.h" +#include "third_party/base/ptr_util.h" -CPWL_EditCtrl::CPWL_EditCtrl() - : m_pEdit(new CPWL_EditImpl), - m_pEditCaret(nullptr), - m_bMouseDown(false), - m_nCharSet(FX_CHARSET_Default) {} +CPWL_EditCtrl::CPWL_EditCtrl() : m_pEdit(pdfium::MakeUnique<CPWL_EditImpl>()) {} -CPWL_EditCtrl::~CPWL_EditCtrl() {} +CPWL_EditCtrl::~CPWL_EditCtrl() = default; void CPWL_EditCtrl::OnCreate(CreateParams* pParamsToAdjust) { pParamsToAdjust->eCursorType = FXCT_VBEAM; @@ -29,7 +26,6 @@ void CPWL_EditCtrl::OnCreate(CreateParams* pParamsToAdjust) { void CPWL_EditCtrl::OnCreated() { SetFontSize(GetCreationParams().fFontSize); - m_pEdit->SetFontMap(GetFontMap()); m_pEdit->SetNotify(this); m_pEdit->Initialize(); @@ -52,16 +48,10 @@ void CPWL_EditCtrl::SetCursor() { } WideString CPWL_EditCtrl::GetSelectedText() { - if (m_pEdit) - return m_pEdit->GetSelectedText(); - - return WideString(); + return m_pEdit->GetSelectedText(); } void CPWL_EditCtrl::ReplaceSelection(const WideString& text) { - if (!m_pEdit) - return; - m_pEdit->ClearSelection(); m_pEdit->InsertText(text, FX_CHARSET_Default); } diff --git a/fpdfsdk/pwl/cpwl_edit_ctrl.h b/fpdfsdk/pwl/cpwl_edit_ctrl.h index a21c8a2700..741ce9c668 100644 --- a/fpdfsdk/pwl/cpwl_edit_ctrl.h +++ b/fpdfsdk/pwl/cpwl_edit_ctrl.h @@ -9,6 +9,7 @@ #include <memory> +#include "core/fxcrt/fx_codepage.h" #include "core/fxcrt/fx_string.h" #include "fpdfsdk/pwl/cpwl_wnd.h" @@ -73,24 +74,20 @@ class CPWL_EditCtrl : public CPWL_Wnd { void CutText(); void InsertWord(uint16_t word, int32_t nCharset); void InsertReturn(); - bool IsWndHorV(); - void Delete(); void Backspace(); - void GetCaretInfo(CFX_PointF* ptHead, CFX_PointF* ptFoot) const; - void SetEditCaret(bool bVisible); - std::unique_ptr<CPWL_EditImpl> m_pEdit; - CPWL_Caret* m_pEditCaret; - bool m_bMouseDown; + std::unique_ptr<CPWL_EditImpl> const m_pEdit; + CPWL_Caret* m_pEditCaret = nullptr; + bool m_bMouseDown = false; private: void CreateEditCaret(const CreateParams& cp); - int32_t m_nCharSet; + int32_t m_nCharSet = FX_CHARSET_Default; }; #endif // FPDFSDK_PWL_CPWL_EDIT_CTRL_H_ |