Convert FDE_TXTEDIT_LINEEND to a private enum class

This CL converts the enum to an enum class and updates the usage as
needed.

Change-Id: I7ba883e7d1debee00d3d9baafe941620109df8b8
Reviewed-on: https://pdfium-review.googlesource.com/11511
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>

2 files changed, 25 insertions, 22 deletions

diff --git a/xfa/fde/cfde_txtedtengine.cpp b/xfa/fde/cfde_txtedtengine.cpp
index a786a9a6d4..72beac7718 100644
--- a/xfa/fde/cfde_txtedtengine.cpp
+++ b/xfa/fde/cfde_txtedtengine.cpp
@@ -29,13 +29,6 @@ enum FDE_TXTEDT_MODIFY_RET {
   FDE_TXTEDT_MODIFY_RET_S_Normal = 0,
 };
 
-enum FDE_TXTEDIT_LINEEND {
-  FDE_TXTEDIT_LINEEND_Auto,
-  FDE_TXTEDIT_LINEEND_CRLF,
-  FDE_TXTEDIT_LINEEND_CR,
-  FDE_TXTEDIT_LINEEND_LF,
-};
-
 class InsertOperation : public IFDE_TxtEdtDoRecord {
  public:
   InsertOperation(CFDE_TxtEdtEngine* pEngine,
@@ -146,7 +139,7 @@ CFDE_TxtEdtEngine::CFDE_TxtEdtEngine()
       m_nCaretPage(0),
       m_nLimit(0),
       m_wcAliasChar(L'*'),
-      m_nFirstLineEnd(FDE_TXTEDIT_LINEEND_Auto),
+      m_FirstLineEnding(LineEnding::kAuto),
       m_bBefore(true),
       m_bLock(false),
       m_bAutoLineEnd(true) {}
@@ -888,20 +881,20 @@ bool CFDE_TxtEdtEngine::ReplaceParagEnd(wchar_t*& lpText,
           nLength--;
           bPreIsCR = false;
           if (m_bAutoLineEnd) {
-            m_nFirstLineEnd = FDE_TXTEDIT_LINEEND_CRLF;
+            m_FirstLineEnding = LineEnding::kCRLF;
             m_bAutoLineEnd = false;
           }
         } else {
           lpText[i] = L'\n';
           if (m_bAutoLineEnd) {
-            m_nFirstLineEnd = FDE_TXTEDIT_LINEEND_LF;
+            m_FirstLineEnding = LineEnding::kLF;
             m_bAutoLineEnd = false;
           }
         }
       } break;
       default: {
         if (bPreIsCR && m_bAutoLineEnd) {
-          m_nFirstLineEnd = FDE_TXTEDIT_LINEEND_CR;
+          m_FirstLineEnding = LineEnding::kCR;
           m_bAutoLineEnd = false;
         }
         bPreIsCR = false;
@@ -912,14 +905,16 @@ bool CFDE_TxtEdtEngine::ReplaceParagEnd(wchar_t*& lpText,
 }
 
 void CFDE_TxtEdtEngine::RecoverParagEnd(CFX_WideString& wsText) const {
-  wchar_t wc = (m_nFirstLineEnd == FDE_TXTEDIT_LINEEND_CR) ? L'\n' : L'\r';
-  if (m_nFirstLineEnd == FDE_TXTEDIT_LINEEND_CRLF) {
+  if (m_FirstLineEnding == LineEnding::kCR)
+    return;
+
+  if (m_FirstLineEnding == LineEnding::kCRLF) {
     std::vector<int32_t> PosArr;
     int32_t nLength = wsText.GetLength();
     wchar_t* lpPos = const_cast<wchar_t*>(wsText.c_str());
     for (int32_t i = 0; i < nLength; i++, lpPos++) {
       if (*lpPos == L'\n') {
-        *lpPos = wc;
+        *lpPos = L'\r';
         PosArr.push_back(i);
       }
     }
@@ -945,13 +940,14 @@ void CFDE_TxtEdtEngine::RecoverParagEnd(CFX_WideString& wsText) const {
     }
     wsTemp.ReleaseBuffer(nLength + nCount);
     wsText = wsTemp;
-  } else {
-    int32_t nLength = wsText.GetLength();
-    wchar_t* lpBuf = const_cast<wchar_t*>(wsText.c_str());
-    for (int32_t i = 0; i < nLength; i++, lpBuf++) {
-      if (*lpBuf == L'\n')
-        *lpBuf = wc;
-    }
+    return;
+  }
+
+  int32_t nLength = wsText.GetLength();
+  wchar_t* lpBuf = const_cast<wchar_t*>(wsText.c_str());
+  for (int32_t i = 0; i < nLength; i++, lpBuf++) {
+    if (*lpBuf == L'\n')
+      *lpBuf = L'\r';
   }
 }
 
diff --git a/xfa/fde/cfde_txtedtengine.h b/xfa/fde/cfde_txtedtengine.h
index 40251159c0..3edd4dc872 100644
--- a/xfa/fde/cfde_txtedtengine.h
+++ b/xfa/fde/cfde_txtedtengine.h
@@ -144,6 +144,13 @@ class CFDE_TxtEdtEngine {
     int32_t nCharIndex;
   };
 
+  enum class LineEnding {
+    kAuto,
+    kCRLF,
+    kCR,
+    kLF,
+  };
+
   int32_t CountPages() const {
     return m_nLineCount == 0 ? 0 : ((m_nLineCount - 1) / m_nPageLineCount) + 1;
   }
@@ -205,7 +212,7 @@ class CFDE_TxtEdtEngine {
   CFX_RectF m_rtCaret;
   int32_t m_nLimit;
   wchar_t m_wcAliasChar;
-  int32_t m_nFirstLineEnd;
+  LineEnding m_FirstLineEnding;
   bool m_bBefore;
   bool m_bLock;
   bool m_bAutoLineEnd;