diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-21 21:52:50 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-21 21:52:50 +0000 |
| commit | 64e16079bb87085fdff5d6268267fd01eacbc5d9 (patch) | |
| tree | 0c13b2e9fb686dc18cdf49f4091e22f089eea65c /xfa/fxfa/cxfa_ffimage.cpp | |
| parent | f9958ed00c5f288d95ab4c6c1d90fc639d5c0f8d (diff) | |
| download | pdfium-64e16079bb87085fdff5d6268267fd01eacbc5d9.tar.xz | |
Cleanup CXFA_ImageData
This CL removes the second constructor parameter which is always true
and removes all related conditionals. The methods have been changed to
const and return instead of using out parameters.
Change-Id: Id6d2aa533a212e961e3b8169eef57b7e868d503e
Reviewed-on: https://pdfium-review.googlesource.com/19010
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffimage.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffimage.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxfa/cxfa_ffimage.cpp b/xfa/fxfa/cxfa_ffimage.cpp index e7dc715eb7..063d3aef5b 100644 --- a/xfa/fxfa/cxfa_ffimage.cpp +++ b/xfa/fxfa/cxfa_ffimage.cpp @@ -61,10 +61,10 @@ void CXFA_FFImage::RenderWidget(CXFA_Graphics* pGS, iVertAlign = paraData.GetVerticalAlign(); } - int32_t iAspect = m_pDataAcc->GetFormValueData().GetImageData().GetAspect(); int32_t iImageXDpi = 0; int32_t iImageYDpi = 0; m_pDataAcc->GetImageDpi(iImageXDpi, iImageYDpi); - XFA_DrawImage(pGS, rtImage, mtRotate, pDIBitmap, iAspect, iImageXDpi, - iImageYDpi, iHorzAlign, iVertAlign); + XFA_DrawImage(pGS, rtImage, mtRotate, pDIBitmap, + m_pDataAcc->GetFormValueData().GetImageData().GetAspect(), + iImageXDpi, iImageYDpi, iHorzAlign, iVertAlign); } |