Turn caret movement into an enum class

This CL changes caret movement into an enum class and removes the
MoveNone flag.

Change-Id: I8a01b057942d45ceae9a6c3b95cfe3b8dc9ce9a5
Reviewed-on: https://pdfium-review.googlesource.com/11274
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>

2 files changed, 19 insertions, 24 deletions

diff --git a/xfa/fde/cfde_txtedtengine.cpp b/xfa/fde/cfde_txtedtengine.cpp
index e69f299e30..31cbfc0773 100644
--- a/xfa/fde/cfde_txtedtengine.cpp
+++ b/xfa/fde/cfde_txtedtengine.cpp
@@ -265,9 +265,7 @@ int32_t CFDE_TxtEdtEngine::SetCaretPos(int32_t nIndex, bool bBefore) {
   return m_nCaret;
 }
 
-int32_t CFDE_TxtEdtEngine::MoveCaretPos(FDE_TXTEDTMOVECARET eMoveCaret,
-                                        bool bShift,
-                                        bool bCtrl) {
+int32_t CFDE_TxtEdtEngine::MoveCaretPos(FDE_CaretMove eMoveCaret, bool bShift) {
   if (IsLocked() || !pdfium::IndexInBounds(m_PagePtrArray, m_nCaretPage))
     return 0;
 
@@ -284,46 +282,44 @@ int32_t CFDE_TxtEdtEngine::MoveCaretPos(FDE_TXTEDTMOVECARET eMoveCaret,
   }
 
   switch (eMoveCaret) {
-    case MC_Left: {
+    case FDE_CaretMove::Left: {
       bool bBefore = true;
       int32_t nIndex = MoveBackward(bBefore);
       if (nIndex >= 0)
         UpdateCaretRect(nIndex, bBefore);
       break;
     }
-    case MC_Right: {
+    case FDE_CaretMove::Right: {
       bool bBefore = true;
       int32_t nIndex = MoveForward(bBefore);
       if (nIndex >= 0)
         UpdateCaretRect(nIndex, bBefore);
       break;
     }
-    case MC_Up: {
+    case FDE_CaretMove::Up: {
       CFX_PointF ptCaret;
       if (MoveUp(ptCaret))
         UpdateCaretIndex(ptCaret);
       break;
     }
-    case MC_Down: {
+    case FDE_CaretMove::Down: {
       CFX_PointF ptCaret;
       if (MoveDown(ptCaret))
         UpdateCaretIndex(ptCaret);
       break;
     }
-    case MC_LineStart:
+    case FDE_CaretMove::LineStart:
       MoveLineStart();
       break;
-    case MC_LineEnd:
+    case FDE_CaretMove::LineEnd:
       MoveLineEnd();
       break;
-    case MC_Home:
+    case FDE_CaretMove::Home:
       MoveHome();
       break;
-    case MC_End:
+    case FDE_CaretMove::End:
       MoveEnd();
       break;
-    default:
-      break;
   }
   if (bShift && m_nAnchorPos != -1 && (m_nAnchorPos != m_nCaret)) {
     AddSelRange(std::min(m_nAnchorPos, m_nCaret), abs(m_nAnchorPos - m_nCaret));
diff --git a/xfa/fde/cfde_txtedtengine.h b/xfa/fde/cfde_txtedtengine.h
index 31de49b4d2..9d472ce65a 100644
--- a/xfa/fde/cfde_txtedtengine.h
+++ b/xfa/fde/cfde_txtedtengine.h
@@ -38,16 +38,15 @@ class IFX_CharIter;
 #define FDE_TEXTEDITLAYOUT_CombText (1L << 4)
 #define FDE_TEXTEDITLAYOUT_LastLineHeight (1L << 8)
 
-enum FDE_TXTEDTMOVECARET {
-  MC_MoveNone = 0,
-  MC_Left,
-  MC_Right,
-  MC_Up,
-  MC_Down,
-  MC_LineStart,
-  MC_LineEnd,
-  MC_Home,
-  MC_End,
+enum class FDE_CaretMove {
+  Left,
+  Right,
+  Up,
+  Down,
+  LineStart,
+  LineEnd,
+  Home,
+  End,
 };
 
 struct FDE_TXTEDTPARAMS {
@@ -109,7 +108,7 @@ class CFDE_TxtEdtEngine {
     return IsLocked() ? 0 : m_nCaret + (m_bBefore ? 0 : 1);
   }
   int32_t SetCaretPos(int32_t nIndex, bool bBefore);
-  int32_t MoveCaretPos(FDE_TXTEDTMOVECARET eMoveCaret, bool bShift, bool bCtrl);
+  int32_t MoveCaretPos(FDE_CaretMove eMoveCaret, bool bShift);
 
   int32_t Insert(const CFX_WideString& str);
   void Delete(bool bBackspace);