commit c6af591b6af8b1edda29222a140af829d4423ad2
author Tom Sepez <tsepez@chromium.org> Mon Apr 26 20:46:00 2021 +0000
committer Chromium commit bot <commit-bot@chromium.org> Mon Apr 26 20:46:00 2021 +0000
tree 4b682262b23c349f83b76983f49776d63dfbf10e
parent e9e352647ead711c384d02ef4773c27588f74bda

Encapsulate CPDF_ExpIntFunc members.

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

core/fpdfapi/page/cpdf_expintfunc.h

diff --git a/core/fpdfapi/page/cpdf_expintfunc.h b/core/fpdfapi/page/cpdf_expintfunc.h
index 95bdab6..be29695 100644
--- a/core/fpdfapi/page/cpdf_expintfunc.h
+++ b/core/fpdfapi/page/cpdf_expintfunc.h
@@ -22,6 +22,12 @@
               std::set<const CPDF_Object*>* pVisited) override;
   bool v_Call(const float* inputs, float* results) const override;
 
+  uint32_t GetOrigOutputs() const { return m_nOrigOutputs; }
+  float GetExponent() const { return m_Exponent; }
+  const std::vector<float>& GetBeginValues() const { return m_BeginValues; }
+  const std::vector<float>& GetEndValues() const { return m_EndValues; }
+
+ private:
   uint32_t m_nOrigOutputs = 0;
   float m_Exponent = 0.0f;
   std::vector<float> m_BeginValues;

core/fxge/skia/fx_skia_device.cpp

diff --git a/core/fxge/skia/fx_skia_device.cpp b/core/fxge/skia/fx_skia_device.cpp
index e7be6fd..d2ba799 100644
--- a/core/fxge/skia/fx_skia_device.cpp
+++ b/core/fxge/skia/fx_skia_device.cpp
@@ -409,13 +409,13 @@
                bool is_encode_reversed) {
   if (pFunc->CountInputs() != 1)
     return false;
-  if (pFunc->m_Exponent != 1)
+  if (pFunc->GetExponent() != 1)
     return false;
-  if (pFunc->m_nOrigOutputs != 3)
+  if (pFunc->GetOrigOutputs() != 3)
     return false;
 
-  auto begin_values = pFunc->m_BeginValues.begin();
-  auto end_values = pFunc->m_EndValues.begin();
+  auto begin_values = pFunc->GetBeginValues().begin();
+  auto end_values = pFunc->GetEndValues().begin();
   if (is_encode_reversed)
     std::swap(begin_values, end_values);