[MSPAINT] Add selectionModel.moveSelection and use it

Refactoring. CORE-19094

base/applications/mspaint/canvas.cpp

@@ -770,16 +770,6 @@ VOID CCanvasWindow::EndSelectionDrag(POINT ptImage)
     Invalidate(FALSE);
 }
 
-VOID CCanvasWindow::MoveSelection(INT xDelta, INT yDelta)
-{
-    if (!selectionModel.m_bShow)
-        return;
-
-    selectionModel.TakeOff();
-    ::OffsetRect(&selectionModel.m_rc, xDelta, yDelta);
-    Invalidate(FALSE);
-}
-
 LRESULT CCanvasWindow::OnCtlColorEdit(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled)
 {
     SetTextColor((HDC)wParam, paletteModel.GetFgColor());

base/applications/mspaint/canvas.h

@@ -52,7 +52,6 @@ public:
     VOID CanvasToImage(POINT& pt);
     VOID CanvasToImage(RECT& rc);
     VOID GetImageRect(RECT& rc);
-    VOID MoveSelection(INT xDelta, INT yDelta);
     VOID getNewZoomRect(CRect& rcView, INT newZoom, CPoint ptTarget);
     VOID zoomTo(INT newZoom, LONG left = 0, LONG top = 0);
 

base/applications/mspaint/selectionmodel.cpp

@@ -551,6 +551,16 @@ void SelectionModel::drawFrameOnCanvas(HDC hCanvasDC)
     drawSizeBoxes(hCanvasDC, &rcSelection, TRUE);
 }
 
+void SelectionModel::moveSelection(INT xDelta, INT yDelta)
+{
+    if (!m_bShow)
+        return;
+
+    TakeOff();
+    ::OffsetRect(&m_rc, xDelta, yDelta);
+    canvasWindow.Invalidate();
+}
+
 void SelectionModel::StretchSelection(BOOL bShrink)
 {
     if (!m_bShow)

base/applications/mspaint/selectionmodel.h

@@ -41,6 +41,7 @@ public:
     void DeleteSelection();
     HITTEST hitTest(POINT ptCanvas);
     void drawFrameOnCanvas(HDC hCanvasDC);
+    void moveSelection(INT xDelta, INT yDelta);
 
     HBITMAP GetSelectionContents();
     void DrawFramePoly(HDC hDCImage);

base/applications/mspaint/winproc.cpp

@@ -529,16 +529,16 @@ LRESULT CMainWindow::OnKeyDown(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bH
             break;
 
         case VK_LEFT:
-            canvasWindow.MoveSelection(-1, 0);
+            selectionModel.moveSelection(-1, 0);
             break;
         case VK_RIGHT:
-            canvasWindow.MoveSelection(+1, 0);
+            selectionModel.moveSelection(+1, 0);
             break;
         case VK_UP:
-            canvasWindow.MoveSelection(0, -1);
+            selectionModel.moveSelection(0, -1);
             break;
         case VK_DOWN:
-            canvasWindow.MoveSelection(0, +1);
+            selectionModel.moveSelection(0, +1);
             break;
         default:
             break;