Fold CXFA_ImageRenderer::Start into constructor

This CL moves the code to initialize the variables into the constructor
and changes the Start command into the StartDIB command.

Change-Id: Iba3c1dcd9a872b4e2a1eadf87050f39c73b8e704
Reviewed-on: https://pdfium-review.googlesource.com/23050
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>

1 files changed, 17 insertions, 22 deletions

diff --git a/xfa/fxfa/cxfa_imagerenderer.cpp b/xfa/fxfa/cxfa_imagerenderer.cpp
index 14f8b25af1..7423037cb5 100644
--- a/xfa/fxfa/cxfa_imagerenderer.cpp
+++ b/xfa/fxfa/cxfa_imagerenderer.cpp
@@ -13,28 +13,23 @@
 #include "core/fxge/dib/cfx_imagetransformer.h"
 #include "third_party/base/ptr_util.h"
 
-CXFA_ImageRenderer::CXFA_ImageRenderer() : m_pDevice(nullptr) {}
+CXFA_ImageRenderer::CXFA_ImageRenderer(
+    CFX_RenderDevice* pDevice,
+    const RetainPtr<CFX_DIBSource>& pDIBSource,
+    FX_ARGB bitmap_argb,
+    int bitmap_alpha,
+    const CFX_Matrix* pImage2Device,
+    uint32_t flags)
+    : m_pDevice(pDevice),
+      m_ImageMatrix(*pImage2Device),
+      m_pDIBSource(pDIBSource),
+      m_BitmapAlpha(bitmap_alpha),
+      m_FillArgb(bitmap_argb),
+      m_Flags(flags) {}
 
 CXFA_ImageRenderer::~CXFA_ImageRenderer() {}
 
-bool CXFA_ImageRenderer::Start(CFX_RenderDevice* pDevice,
-                               const RetainPtr<CFX_DIBSource>& pDIBSource,
-                               FX_ARGB bitmap_argb,
-                               int bitmap_alpha,
-                               const CFX_Matrix* pImage2Device,
-                               uint32_t flags,
-                               int blendType) {
-  m_pDevice = pDevice;
-  m_pDIBSource = pDIBSource;
-  m_FillArgb = bitmap_argb;
-  m_BitmapAlpha = bitmap_alpha;
-  m_ImageMatrix = *pImage2Device;
-  m_Flags = flags;
-  m_BlendType = blendType;
-  return StartDIBSource();
-}
-
-bool CXFA_ImageRenderer::StartDIBSource() {
+bool CXFA_ImageRenderer::Start() {
   if (m_pDevice->StartDIBitsWithBlend(m_pDIBSource, m_BitmapAlpha, m_FillArgb,
                                       &m_ImageMatrix, m_Flags, &m_DeviceHandle,
                                       m_BlendType)) {
@@ -228,9 +223,9 @@ void CXFA_ImageRenderer::CompositeDIBitmap(
   if (!pCloneConvert)
     return;
 
-  CXFA_ImageRenderer imageRender;
-  if (!imageRender.Start(m_pDevice, pCloneConvert, m_FillArgb, m_BitmapAlpha,
-                         &m_ImageMatrix, m_Flags)) {
+  CXFA_ImageRenderer imageRender(m_pDevice, pCloneConvert, m_FillArgb,
+                                 m_BitmapAlpha, &m_ImageMatrix, m_Flags);
+  if (!imageRender.Start()) {
     return;
   }
   while (imageRender.Continue())