Convert more TransformPoint calls to Transform

This Cl converts several uses of TransformPoint to use Transform(CFX_PointF).

Change-Id: I9bc3c484e0a4304b904584218bd9e59dec7db727
Reviewed-on: https://pdfium-review.googlesource.com/2791
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Nicolás Peña <npm@chromium.org>

3 files changed, 25 insertions, 28 deletions

diff --git a/core/fxge/ge/cfx_pathdata.cpp b/core/fxge/ge/cfx_pathdata.cpp
index 11c421e468..e06eadd6ce 100644
--- a/core/fxge/ge/cfx_pathdata.cpp
+++ b/core/fxge/ge/cfx_pathdata.cpp
@@ -452,26 +452,24 @@ bool CFX_PathData::IsRect(const CFX_Matrix* pMatrix,
     return false;
   }
 
-  FX_FLOAT x[5];
-  FX_FLOAT y[5];
+  CFX_PointF points[5];
   for (size_t i = 0; i < m_Points.size(); i++) {
-    x[i] = m_Points[i].m_PointX;
-    y[i] = m_Points[i].m_PointY;
-    pMatrix->TransformPoint(x[i], y[i]);
+    points[i] = pMatrix->Transform(
+        CFX_PointF(m_Points[i].m_PointX, m_Points[i].m_PointY));
 
     if (i == 0)
       continue;
     if (m_Points[i].m_Type != FXPT_TYPE::LineTo)
       return false;
-    if (x[i] != x[i - 1] && y[i] != y[i - 1])
+    if (points[i].x != points[i - 1].x && points[i].y != points[i - 1].y)
       return false;
   }
 
   if (pRect) {
-    pRect->left = x[0];
-    pRect->right = x[2];
-    pRect->bottom = y[0];
-    pRect->top = y[2];
+    pRect->left = points[0].x;
+    pRect->right = points[2].x;
+    pRect->bottom = points[0].y;
+    pRect->top = points[2].y;
     pRect->Normalize();
   }
   return true;
diff --git a/core/fxge/ge/cfx_renderdevice.cpp b/core/fxge/ge/cfx_renderdevice.cpp
index 86e74cafc4..203fecf061 100644
--- a/core/fxge/ge/cfx_renderdevice.cpp
+++ b/core/fxge/ge/cfx_renderdevice.cpp
@@ -490,15 +490,14 @@ bool CFX_RenderDevice::DrawPathWithBlend(const CFX_PathData* pPathData,
   uint8_t fill_alpha = (fill_mode & 3) ? FXARGB_A(fill_color) : 0;
   const std::vector<FX_PATHPOINT>& pPoints = pPathData->GetPoints();
   if (stroke_alpha == 0 && pPoints.size() == 2) {
-    FX_FLOAT x1 = pPoints[0].m_PointX;
-    FX_FLOAT y1 = pPoints[0].m_PointY;
-    FX_FLOAT x2 = pPoints[1].m_PointX;
-    FX_FLOAT y2 = pPoints[1].m_PointY;
+    CFX_PointF pos1(pPoints[0].m_PointX, pPoints[0].m_PointY);
+    CFX_PointF pos2(pPoints[1].m_PointX, pPoints[1].m_PointY);
     if (pObject2Device) {
-      pObject2Device->TransformPoint(x1, y1);
-      pObject2Device->TransformPoint(x2, y2);
+      pos1 = pObject2Device->Transform(pos1);
+      pos2 = pObject2Device->Transform(pos2);
     }
-    DrawCosmeticLine(x1, y1, x2, y2, fill_color, fill_mode, blend_type);
+    DrawCosmeticLine(pos1.x, pos1.y, pos2.x, pos2.y, fill_color, fill_mode,
+                     blend_type);
     return true;
   }
 
diff --git a/core/fxge/win32/cfx_psrenderer.cpp b/core/fxge/win32/cfx_psrenderer.cpp
index 32da090ed6..a0e675e23c 100644
--- a/core/fxge/win32/cfx_psrenderer.cpp
+++ b/core/fxge/win32/cfx_psrenderer.cpp
@@ -116,12 +116,11 @@ void CFX_PSRenderer::OutputPath(const CFX_PathData* pPathData,
   for (size_t i = 0; i < size; i++) {
     FXPT_TYPE type = pPathData->GetType(i);
     bool closing = pPathData->IsClosingFigure(i);
-    FX_FLOAT x = pPathData->GetPointX(i);
-    FX_FLOAT y = pPathData->GetPointY(i);
+    CFX_PointF pos(pPathData->GetPointX(i), pPathData->GetPointY(i));
     if (pObject2Device)
-      pObject2Device->TransformPoint(x, y);
+      pos = pObject2Device->Transform(pos);
 
-    buf << x << " " << y;
+    buf << pos.x << " " << pos.y;
     switch (type) {
       case FXPT_TYPE::MoveTo:
         buf << " m ";
@@ -132,15 +131,16 @@ void CFX_PSRenderer::OutputPath(const CFX_PathData* pPathData,
           buf << "h ";
         break;
       case FXPT_TYPE::BezierTo: {
-        FX_FLOAT x1 = pPathData->GetPointX(i + 1);
-        FX_FLOAT x2 = pPathData->GetPointX(i + 2);
-        FX_FLOAT y1 = pPathData->GetPointY(i + 1);
-        FX_FLOAT y2 = pPathData->GetPointY(i + 2);
+        CFX_PointF pos1(pPathData->GetPointX(i + 1),
+                        pPathData->GetPointY(i + 1));
+        CFX_PointF pos2(pPathData->GetPointX(i + 2),
+                        pPathData->GetPointY(i + 2));
         if (pObject2Device) {
-          pObject2Device->TransformPoint(x1, y1);
-          pObject2Device->TransformPoint(x2, y2);
+          pos1 = pObject2Device->Transform(pos1);
+          pos2 = pObject2Device->Transform(pos2);
         }
-        buf << " " << x1 << " " << y1 << " " << x2 << " " << y2 << " c";
+        buf << " " << pos1.x << " " << pos1.y << " " << pos2.x << " " << pos2.y
+            << " c";
         if (closing)
           buf << " h";
         buf << "\n";