Revert "Convert Origins to points"

This reverts commit da83d3a5cc09c4056310b3cf299dbbccd5c70d11.

Reason for revert: Reverting chain to see if fixes Chrome roll.

Original change's description:
> Convert Origins to points
>
> This CL converts various OriginX, OriginY pairs into CFX_PointF objects.
>
> Change-Id: I9141f7fc713c710b2014d4fdcdec7dc93501f844
> Reviewed-on: https://pdfium-review.googlesource.com/2575
> Commit-Queue: dsinclair <dsinclair@chromium.org>
> Reviewed-by: Nicolás Peña <npm@chromium.org>
>

TBR=tsepez@chromium.org,dsinclair@chromium.org,npm@chromium.org,pdfium-reviews@googlegroups.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Change-Id: I949fb4ec712e2587e7d0ef0191c34db198b61dcc
Reviewed-on: https://pdfium-review.googlesource.com/2696
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>

2 files changed, 32 insertions, 30 deletions

diff --git a/xfa/fgas/layout/fgas_rtfbreak.cpp b/xfa/fgas/layout/fgas_rtfbreak.cpp
index 293bbc6f5b..62e33fe9c2 100644
--- a/xfa/fgas/layout/fgas_rtfbreak.cpp
+++ b/xfa/fgas/layout/fgas_rtfbreak.cpp
@@ -1307,7 +1307,8 @@ int32_t CFX_RTFBreak::GetDisplayPos(const FX_RTFTEXTOBJ* pText,
               ptOffset.y = fFontSize * (1000 - rtBBox.height) / 2000.0f;
             }
           }
-          pCharPos->m_Origin = CFX_PointF(fX + ptOffset.x, fY - ptOffset.y);
+          pCharPos->m_OriginX = fX + ptOffset.x;
+          pCharPos->m_OriginY = fY - ptOffset.y;
         }
         if (!bRTLPiece && dwCharType != FX_CHARTYPE_Combination) {
           if (bVerticalDoc) {
@@ -1324,28 +1325,28 @@ int32_t CFX_RTFBreak::GetDisplayPos(const FX_RTFTEXTOBJ* pText,
               pCharPos->m_AdjustMatrix[1] = 0;
               pCharPos->m_AdjustMatrix[2] = 0;
               pCharPos->m_AdjustMatrix[3] = 1;
-              pCharPos->m_Origin.y += fAscent * iVerScale / 100.0f;
+              pCharPos->m_OriginY += fAscent * iVerScale / 100.0f;
             } else if (iCharRotation == 1) {
               pCharPos->m_AdjustMatrix[0] = 0;
               pCharPos->m_AdjustMatrix[1] = -1;
               pCharPos->m_AdjustMatrix[2] = -1;
               pCharPos->m_AdjustMatrix[3] = 0;
-              pCharPos->m_Origin.x -=
+              pCharPos->m_OriginX -=
                   fDescent + fAscent * iVerScale / 100.0f - fAscent;
             } else if (iCharRotation == 2) {
               pCharPos->m_AdjustMatrix[0] = 1;
               pCharPos->m_AdjustMatrix[1] = 0;
               pCharPos->m_AdjustMatrix[2] = 0;
               pCharPos->m_AdjustMatrix[3] = -1;
-              pCharPos->m_Origin.x += fCharWidth;
-              pCharPos->m_Origin.y += fAscent;
+              pCharPos->m_OriginX += fCharWidth;
+              pCharPos->m_OriginY += fAscent;
             } else {
               pCharPos->m_AdjustMatrix[0] = 0;
               pCharPos->m_AdjustMatrix[1] = 1;
               pCharPos->m_AdjustMatrix[2] = 1;
               pCharPos->m_AdjustMatrix[3] = 0;
-              pCharPos->m_Origin.x += fAscent;
-              pCharPos->m_Origin.y += fCharWidth;
+              pCharPos->m_OriginX += fAscent;
+              pCharPos->m_OriginY += fCharWidth;
             }
           } else {
             if (iCharRotation == 0) {
@@ -1353,27 +1354,27 @@ int32_t CFX_RTFBreak::GetDisplayPos(const FX_RTFTEXTOBJ* pText,
               pCharPos->m_AdjustMatrix[1] = 0;
               pCharPos->m_AdjustMatrix[2] = 0;
               pCharPos->m_AdjustMatrix[3] = 1;
-              pCharPos->m_Origin.y += fAscent * iVerScale / 100.0f - fAscent;
+              pCharPos->m_OriginY += fAscent * iVerScale / 100.0f - fAscent;
             } else if (iCharRotation == 1) {
               pCharPos->m_AdjustMatrix[0] = 0;
               pCharPos->m_AdjustMatrix[1] = -1;
               pCharPos->m_AdjustMatrix[2] = -1;
               pCharPos->m_AdjustMatrix[3] = 0;
-              pCharPos->m_Origin.x -= fDescent;
-              pCharPos->m_Origin.y -= fAscent + fDescent;
+              pCharPos->m_OriginX -= fDescent;
+              pCharPos->m_OriginY -= fAscent + fDescent;
             } else if (iCharRotation == 2) {
               pCharPos->m_AdjustMatrix[0] = 1;
               pCharPos->m_AdjustMatrix[1] = 0;
               pCharPos->m_AdjustMatrix[2] = 0;
               pCharPos->m_AdjustMatrix[3] = -1;
-              pCharPos->m_Origin.x += fCharWidth;
-              pCharPos->m_Origin.y -= fAscent;
+              pCharPos->m_OriginX += fCharWidth;
+              pCharPos->m_OriginY -= fAscent;
             } else {
               pCharPos->m_AdjustMatrix[0] = 0;
               pCharPos->m_AdjustMatrix[1] = 1;
               pCharPos->m_AdjustMatrix[2] = 1;
               pCharPos->m_AdjustMatrix[3] = 0;
-              pCharPos->m_Origin.x += fAscent * iVerScale / 100.0f;
+              pCharPos->m_OriginX += fAscent * iVerScale / 100.0f;
             }
           }
           if (iHorScale != 100 || iVerScale != 100) {
diff --git a/xfa/fgas/layout/fgas_textbreak.cpp b/xfa/fgas/layout/fgas_textbreak.cpp
index b8893b99ae..69773f8eb0 100644
--- a/xfa/fgas/layout/fgas_textbreak.cpp
+++ b/xfa/fgas/layout/fgas_textbreak.cpp
@@ -1447,21 +1447,22 @@ int32_t CFX_TxtBreak::GetDisplayPos(const FX_TXTRUN* pTxtRun,
         }
       }
       if (!bEmptyChar || (bEmptyChar && !bSkipSpace)) {
-        pCharPos->m_Origin = CFX_PointF(fX, fY);
+        pCharPos->m_OriginX = fX;
+        pCharPos->m_OriginY = fY;
         if ((dwStyles & FX_TXTLAYOUTSTYLE_CombText) != 0) {
           int32_t iFormWidth = iCharWidth;
           pFont->GetCharWidth(wForm, iFormWidth, false);
           FX_FLOAT fOffset = fFontSize * (iCharWidth - iFormWidth) / 2000.0f;
           if (bVerticalDoc) {
-            pCharPos->m_Origin.y += fOffset;
+            pCharPos->m_OriginY += fOffset;
           } else {
-            pCharPos->m_Origin.x += fOffset;
+            pCharPos->m_OriginX += fOffset;
           }
         }
         if (chartype == FX_CHARTYPE_Combination) {
           CFX_Rect rtBBox;
           if (pFont->GetCharBBox(wForm, &rtBBox, false)) {
-            pCharPos->m_Origin.y =
+            pCharPos->m_OriginY =
                 fYBase + fFontSize -
                 fFontSize * (FX_FLOAT)rtBBox.height / (FX_FLOAT)iMaxHeight;
           }
@@ -1471,7 +1472,7 @@ int32_t CFX_TxtBreak::GetDisplayPos(const FX_TXTRUN* pTxtRun,
                 FX_CHARTYPE_Combination) {
               CFX_Rect rtBox;
               if (pFont->GetCharBBox(wLast, &rtBox, false)) {
-                pCharPos->m_Origin.y -= fFontSize * rtBox.height / iMaxHeight;
+                pCharPos->m_OriginY -= fFontSize * rtBox.height / iMaxHeight;
               }
             }
           }
@@ -1489,8 +1490,8 @@ int32_t CFX_TxtBreak::GetDisplayPos(const FX_TXTRUN* pTxtRun,
             ptOffset.y = fFontSize * (iAscent - rtBBox.top - 150) / iMaxHeight;
           }
         }
-        pCharPos->m_Origin.x += ptOffset.x;
-        pCharPos->m_Origin.y -= ptOffset.y;
+        pCharPos->m_OriginX += ptOffset.x;
+        pCharPos->m_OriginY -= ptOffset.y;
       }
       if (!bRTLPiece && chartype != FX_CHARTYPE_Combination) {
         if (bVerticalDoc) {
@@ -1507,26 +1508,26 @@ int32_t CFX_TxtBreak::GetDisplayPos(const FX_TXTRUN* pTxtRun,
             pCharPos->m_AdjustMatrix[1] = 0;
             pCharPos->m_AdjustMatrix[2] = 0;
             pCharPos->m_AdjustMatrix[3] = 1;
-            pCharPos->m_Origin.y += fAscent;
+            pCharPos->m_OriginY += fAscent;
           } else if (iCharRotation == 1) {
             pCharPos->m_AdjustMatrix[0] = 0;
             pCharPos->m_AdjustMatrix[1] = -1;
             pCharPos->m_AdjustMatrix[2] = -1;
             pCharPos->m_AdjustMatrix[3] = 0;
-            pCharPos->m_Origin.x -= fDescent;
+            pCharPos->m_OriginX -= fDescent;
           } else if (iCharRotation == 2) {
             pCharPos->m_AdjustMatrix[0] = 1;
             pCharPos->m_AdjustMatrix[1] = 0;
             pCharPos->m_AdjustMatrix[2] = 0;
             pCharPos->m_AdjustMatrix[3] = -1;
-            pCharPos->m_Origin.x += fCharWidth;
-            pCharPos->m_Origin.y += fAscent;
+            pCharPos->m_OriginX += fCharWidth;
+            pCharPos->m_OriginY += fAscent;
           } else {
             pCharPos->m_AdjustMatrix[0] = 0;
             pCharPos->m_AdjustMatrix[1] = 1;
             pCharPos->m_AdjustMatrix[2] = 1;
             pCharPos->m_AdjustMatrix[3] = 0;
-            pCharPos->m_Origin.x += fAscent;
+            pCharPos->m_OriginX += fAscent;
           }
         } else {
           if (iCharRotation == 0) {
@@ -1539,21 +1540,21 @@ int32_t CFX_TxtBreak::GetDisplayPos(const FX_TXTRUN* pTxtRun,
             pCharPos->m_AdjustMatrix[1] = -1;
             pCharPos->m_AdjustMatrix[2] = -1;
             pCharPos->m_AdjustMatrix[3] = 0;
-            pCharPos->m_Origin.x -= fDescent;
-            pCharPos->m_Origin.y -= fAscent + fDescent;
+            pCharPos->m_OriginX -= fDescent;
+            pCharPos->m_OriginY -= fAscent + fDescent;
           } else if (iCharRotation == 2) {
             pCharPos->m_AdjustMatrix[0] = 1;
             pCharPos->m_AdjustMatrix[1] = 0;
             pCharPos->m_AdjustMatrix[2] = 0;
             pCharPos->m_AdjustMatrix[3] = -1;
-            pCharPos->m_Origin.x += fCharWidth;
-            pCharPos->m_Origin.y -= fAscent;
+            pCharPos->m_OriginX += fCharWidth;
+            pCharPos->m_OriginY -= fAscent;
           } else {
             pCharPos->m_AdjustMatrix[0] = 0;
             pCharPos->m_AdjustMatrix[1] = 1;
             pCharPos->m_AdjustMatrix[2] = 1;
             pCharPos->m_AdjustMatrix[3] = 0;
-            pCharPos->m_Origin.x += fAscent;
+            pCharPos->m_OriginX += fAscent;
           }
         }
         if (iHorScale != 100 || iVerScale != 100) {