summaryrefslogtreecommitdiff
path: root/xfa/fxfa/app/xfa_ffwidget.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-02-07 09:04:28 -0500
committerChromium commit bot <commit-bot@chromium.org>2017-02-07 14:33:17 +0000
commitc222907f453e8a0e6376a86f89354eedb8285854 (patch)
tree8ac6942b7fe65f7eb6dbc951a6a1f77aa9e18afc /xfa/fxfa/app/xfa_ffwidget.cpp
parent76da8841aef9a60c1c65c646a9f943c25861bc33 (diff)
downloadpdfium-c222907f453e8a0e6376a86f89354eedb8285854.tar.xz
Cleanup out params in XFA layout code.
This CL converts some of the out parameters in the XFA layout code to pointers instead of references. Change-Id: I6246b91b975e7bc08f8cfb040de9dfdc3c3bedee Reviewed-on: https://pdfium-review.googlesource.com/2531 Commit-Queue: dsinclair <dsinclair@chromium.org> Reviewed-by: Nicolás Peña <npm@chromium.org>
Diffstat (limited to 'xfa/fxfa/app/xfa_ffwidget.cpp')
-rw-r--r--xfa/fxfa/app/xfa_ffwidget.cpp12
1 files changed, 5 insertions, 7 deletions
diff --git a/xfa/fxfa/app/xfa_ffwidget.cpp b/xfa/fxfa/app/xfa_ffwidget.cpp
index 29b10dd195..95cfe2e59f 100644
--- a/xfa/fxfa/app/xfa_ffwidget.cpp
+++ b/xfa/fxfa/app/xfa_ffwidget.cpp
@@ -46,15 +46,13 @@ const CFWL_App* CXFA_FFWidget::GetFWLApp() {
}
void CXFA_FFWidget::GetWidgetRect(CFX_RectF& rtWidget) {
- if ((m_dwStatus & XFA_WidgetStatus_RectCached) == 0) {
- m_dwStatus |= XFA_WidgetStatus_RectCached;
- GetRect(m_rtWidget);
- }
+ if ((m_dwStatus & XFA_WidgetStatus_RectCached) == 0)
+ RecacheWidgetRect();
rtWidget = m_rtWidget;
}
-CFX_RectF CXFA_FFWidget::ReCacheWidgetRect() {
+CFX_RectF CXFA_FFWidget::RecacheWidgetRect() {
m_dwStatus |= XFA_WidgetStatus_RectCached;
- GetRect(m_rtWidget);
+ m_rtWidget = GetRect(false);
return m_rtWidget;
}
void CXFA_FFWidget::GetRectWithoutRotate(CFX_RectF& rtWidget) {
@@ -136,7 +134,7 @@ bool CXFA_FFWidget::LoadWidget() {
}
void CXFA_FFWidget::UnloadWidget() {}
bool CXFA_FFWidget::PerformLayout() {
- ReCacheWidgetRect();
+ RecacheWidgetRect();
return true;
}
bool CXFA_FFWidget::UpdateFWLData() {