Remove always-false argument from CPLW_EditImpl::SetAutoScroll()
Change-Id: If9d8f9b4116193cf1ce627b73e0b23cac7a0c08f
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/82194
Auto-Submit: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
diff --git a/fpdfsdk/pwl/cpwl_edit.cpp b/fpdfsdk/pwl/cpwl_edit.cpp
index 8478511..513ee55 100644
--- a/fpdfsdk/pwl/cpwl_edit.cpp
+++ b/fpdfsdk/pwl/cpwl_edit.cpp
@@ -142,7 +142,7 @@
m_pEditImpl->SetMultiLine(HasFlag(PES_MULTILINE), false);
m_pEditImpl->SetAutoReturn(HasFlag(PES_AUTORETURN), false);
m_pEditImpl->SetAutoFontSize(HasFlag(PWS_AUTOFONTSIZE), false);
- m_pEditImpl->SetAutoScroll(HasFlag(PES_AUTOSCROLL), false);
+ m_pEditImpl->SetAutoScroll(HasFlag(PES_AUTOSCROLL));
m_pEditImpl->EnableUndo(HasFlag(PES_UNDO));
if (HasFlag(PES_TEXTOVERFLOW)) {
diff --git a/fpdfsdk/pwl/cpwl_edit_impl.cpp b/fpdfsdk/pwl/cpwl_edit_impl.cpp
index 705678e..5be9483 100644
--- a/fpdfsdk/pwl/cpwl_edit_impl.cpp
+++ b/fpdfsdk/pwl/cpwl_edit_impl.cpp
@@ -783,10 +783,8 @@
Paint();
}
-void CPWL_EditImpl::SetAutoScroll(bool bAuto, bool bPaint) {
+void CPWL_EditImpl::SetAutoScroll(bool bAuto) {
m_bEnableScroll = bAuto;
- if (bPaint)
- Paint();
}
void CPWL_EditImpl::SetTextOverflow(bool bAllowed, bool bPaint) {
diff --git a/fpdfsdk/pwl/cpwl_edit_impl.h b/fpdfsdk/pwl/cpwl_edit_impl.h
index 8b2bb56..f1e3bb0 100644
--- a/fpdfsdk/pwl/cpwl_edit_impl.h
+++ b/fpdfsdk/pwl/cpwl_edit_impl.h
@@ -81,7 +81,7 @@
void SetMultiLine(bool bMultiLine, bool bPaint);
void SetAutoReturn(bool bAuto, bool bPaint);
void SetAutoFontSize(bool bAuto, bool bPaint);
- void SetAutoScroll(bool bAuto, bool bPaint);
+ void SetAutoScroll(bool bAuto);
void SetFontSize(float fFontSize);
void SetTextOverflow(bool bAllowed, bool bPaint);
void OnMouseDown(const CFX_PointF& point, bool bShift, bool bCtrl);