Convert enum to enum class for Widget::MoveLoopEscapeBehavior

Use enum class instead of enum for Widget::MoveLoopEscapeBehavior.
enum class is more type safety.

Bug: 940736
Change-Id: I93a798be271d83179a86cb778a772b2ed70a4872
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547229
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#829239}

chrome/browser/ui/views/tabs/tab_drag_controller.cc

@@ -1415,8 +1415,8 @@ void TabDragController::RunMoveLoop(const gfx::Vector2d& drag_offset) {
           : views::Widget::MoveLoopSource::kTouch;
   const views::Widget::MoveLoopEscapeBehavior escape_behavior =
       is_dragging_new_browser_
-          ? views::Widget::MOVE_LOOP_ESCAPE_BEHAVIOR_HIDE
-          : views::Widget::MOVE_LOOP_ESCAPE_BEHAVIOR_DONT_HIDE;
+          ? views::Widget::MoveLoopEscapeBehavior::kHide
+          : views::Widget::MoveLoopEscapeBehavior::kDontHide;
   views::Widget::MoveLoopResult result = move_loop_widget_->RunMoveLoop(
       drag_offset, move_loop_source, escape_behavior);
   content::NotificationService::current()->Notify(

ui/views/widget/desktop_aura/desktop_window_tree_host_win.cc

@@ -421,7 +421,7 @@ Widget::MoveLoopResult DesktopWindowTreeHostWin::RunMoveLoop(
     Widget::MoveLoopSource source,
     Widget::MoveLoopEscapeBehavior escape_behavior) {
   const bool hide_on_escape =
-      escape_behavior == Widget::MOVE_LOOP_ESCAPE_BEHAVIOR_HIDE;
+      escape_behavior == Widget::MoveLoopEscapeBehavior::kHide;
   return message_handler_->RunMoveLoop(drag_offset, hide_on_escape)
              ? Widget::MOVE_LOOP_SUCCESSFUL
              : Widget::MOVE_LOOP_CANCELED;

ui/views/widget/desktop_aura/window_move_client_platform.cc

@@ -24,7 +24,7 @@ wm::WindowMoveResult WindowMoveClientPlatform::RunMoveLoop(
       move_source == wm::WindowMoveSource::WINDOW_MOVE_SOURCE_MOUSE
           ? Widget::MoveLoopSource::kMouse
           : Widget::MoveLoopSource::kTouch,
-      Widget::MoveLoopEscapeBehavior::MOVE_LOOP_ESCAPE_BEHAVIOR_HIDE);
+      Widget::MoveLoopEscapeBehavior::kHide);
 
   return move_loop_result == Widget::MOVE_LOOP_SUCCESSFUL ? wm::MOVE_SUCCESSFUL
                                                           : wm::MOVE_CANCELED;

ui/views/widget/widget.h

@@ -122,12 +122,12 @@ class VIEWS_EXPORT Widget : public internal::NativeWidgetDelegate,
   };
 
   // Behavior when escape is pressed during a move loop.
-  enum MoveLoopEscapeBehavior {
+  enum class MoveLoopEscapeBehavior {
     // Indicates the window should be hidden.
-    MOVE_LOOP_ESCAPE_BEHAVIOR_HIDE,
+    kHide,
 
     // Indicates the window should not be hidden.
-    MOVE_LOOP_ESCAPE_BEHAVIOR_DONT_HIDE,
+    kDontHide,
   };
 
   // Type of visibility change transition that should animate.