commit a76ed5c80d4b6b7ccc73ddf690bb502aad5baa43
author Tom Sepez <tsepez@chromium.org> Wed Jun 09 23:10:56 2021 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jun 09 23:10:56 2021 +0000
tree b15e9a94721f7e3a5097caf9feae785c2102cf72
parent bb6b584fe5a581ecf3d4683cb378c51be86ab2f2

Make most of CStretchEngine members be private.

Requires adding one test-only method.

Change-Id: I32ee0688aae62a14096ff25b96199e43fba4725d
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/81752
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

core/fxge/dib/cstretchengine.h

diff --git a/core/fxge/dib/cstretchengine.h b/core/fxge/dib/cstretchengine.h
index fc23e85..c61522e 100644
--- a/core/fxge/dib/cstretchengine.h
+++ b/core/fxge/dib/cstretchengine.h
@@ -32,11 +32,15 @@
   ~CStretchEngine();
 
   bool Continue(PauseIndicatorIface* pPause);
-
   bool StartStretchHorz();
   bool ContinueStretchHorz(PauseIndicatorIface* pPause);
   void StretchVert();
 
+  const FXDIB_ResampleOptions& GetResampleOptionsForTest() const {
+    return m_ResampleOptions;
+  }
+
+ private:
   class CWeightTable {
    public:
     CWeightTable();

core/fxge/dib/cstretchengine_unittest.cpp

diff --git a/core/fxge/dib/cstretchengine_unittest.cpp b/core/fxge/dib/cstretchengine_unittest.cpp
index 055ed25..c3e3b8d 100644
--- a/core/fxge/dib/cstretchengine_unittest.cpp
+++ b/core/fxge/dib/cstretchengine_unittest.cpp
@@ -25,8 +25,8 @@
   dib_source->Load(nullptr, stream.Get());
   CStretchEngine engine(nullptr, FXDIB_Format::k8bppRgb, 500, 500, clip_rect,
                         dib_source, FXDIB_ResampleOptions());
-  EXPECT_TRUE(engine.m_ResampleOptions.bInterpolateBilinear);
-  EXPECT_FALSE(engine.m_ResampleOptions.bHalftone);
-  EXPECT_FALSE(engine.m_ResampleOptions.bNoSmoothing);
-  EXPECT_FALSE(engine.m_ResampleOptions.bLossy);
+  EXPECT_TRUE(engine.GetResampleOptionsForTest().bInterpolateBilinear);
+  EXPECT_FALSE(engine.GetResampleOptionsForTest().bHalftone);
+  EXPECT_FALSE(engine.GetResampleOptionsForTest().bNoSmoothing);
+  EXPECT_FALSE(engine.GetResampleOptionsForTest().bLossy);
 }