Make IXFA_DocEnvironment::PopupMenu() pass a CFX_PointF by const-ref. Change the entire call stack pass by const-ref, instead of by value. Change-Id: I0abac640145957b31a692c2667ba16bc1f20b59b Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/57492 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/fpdfsdk/cpdfsdk_formfillenvironment.cpp b/fpdfsdk/cpdfsdk_formfillenvironment.cpp index e6221ac..94bd808 100644 --- a/fpdfsdk/cpdfsdk_formfillenvironment.cpp +++ b/fpdfsdk/cpdfsdk_formfillenvironment.cpp
@@ -406,7 +406,7 @@ bool CPDFSDK_FormFillEnvironment::PopupMenu(CPDFXFA_Page* page, FPDF_WIDGET hWidget, int menuFlag, - CFX_PointF pt) { + const CFX_PointF& pt) { return m_pInfo && m_pInfo->FFI_PopupMenu && m_pInfo->FFI_PopupMenu(m_pInfo, FPDFPageFromIPDFPage(page), hWidget, menuFlag, pt.x, pt.y);
diff --git a/fpdfsdk/cpdfsdk_formfillenvironment.h b/fpdfsdk/cpdfsdk_formfillenvironment.h index ce357d4..0a902c9 100644 --- a/fpdfsdk/cpdfsdk_formfillenvironment.h +++ b/fpdfsdk/cpdfsdk_formfillenvironment.h
@@ -134,7 +134,7 @@ bool PopupMenu(CPDFXFA_Page* page, FPDF_WIDGET hWidget, int menuFlag, - CFX_PointF pt); + const CFX_PointF& pt); void EmailTo(FPDF_FILEHANDLER* fileHandler, FPDF_WIDESTRING pTo,
diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp index db3de42..8dd06a5 100644 --- a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp +++ b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp
@@ -238,7 +238,7 @@ } bool CPDFXFA_DocEnvironment::PopupMenu(CXFA_FFWidget* hWidget, - CFX_PointF ptPopup) { + const CFX_PointF& ptPopup) { if (!hWidget) return false;
diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.h b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.h index 974de88..7c86bd81 100644 --- a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.h +++ b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.h
@@ -33,7 +33,7 @@ float fMaxPopup, const CFX_RectF& rtAnchor, CFX_RectF* pPopupRect) override; - bool PopupMenu(CXFA_FFWidget* hWidget, CFX_PointF ptPopup) override; + bool PopupMenu(CXFA_FFWidget* hWidget, const CFX_PointF& ptPopup) override; // dwFlags XFA_PAGEVIEWEVENT_Added, XFA_PAGEVIEWEVENT_Removing void PageViewEvent(CXFA_FFPageView* pPageView, uint32_t dwFlags) override;
diff --git a/xfa/fxfa/fxfa.h b/xfa/fxfa/fxfa.h index b5a6d1a..83fb3cc 100644 --- a/xfa/fxfa/fxfa.h +++ b/xfa/fxfa/fxfa.h
@@ -235,7 +235,7 @@ float fMaxPopup, const CFX_RectF& rtAnchor, CFX_RectF* pPopupRect) = 0; - virtual bool PopupMenu(CXFA_FFWidget* hWidget, CFX_PointF ptPopup) = 0; + virtual bool PopupMenu(CXFA_FFWidget* hWidget, const CFX_PointF& ptPopup) = 0; virtual void PageViewEvent(CXFA_FFPageView* pPageView, uint32_t dwFlags) = 0; virtual void WidgetPostAdd(CXFA_FFWidget* hWidget) = 0; virtual void WidgetPreRemove(CXFA_FFWidget* hWidget) = 0;