refactor to create a HasMarked, no logic change

Change-Id: I667a8582a740f5fbbcdbcd82907e40e82f6165a1

include/svx/sdr/table/tablecontroller.hxx

@@ -51,6 +51,7 @@ public:
     SVX_DLLPRIVATE virtual bool onMouseButtonUp(const MouseEvent& rMEvt, vcl::Window* pWin) SAL_OVERRIDE;
     SVX_DLLPRIVATE virtual bool onMouseMove(const MouseEvent& rMEvt, vcl::Window* pWin) SAL_OVERRIDE;
 
+    SVX_DLLPRIVATE virtual bool HasMarked() SAL_OVERRIDE;
     SVX_DLLPRIVATE virtual bool DeleteMarked() SAL_OVERRIDE;
 
     SVX_DLLPRIVATE virtual void onSelectionHasChanged() SAL_OVERRIDE;

include/svx/selectioncontroller.hxx

@@ -50,6 +50,7 @@ public:
     virtual void GetState( SfxItemSet& rSet );
     virtual void Execute( SfxRequest& rReq );
 
+    virtual bool HasMarked();
     virtual bool DeleteMarked();
 
     virtual bool GetAttributes(SfxItemSet& rTargetSet, bool bOnlyHardAttr) const;

svx/source/svdraw/selectioncontroller.cxx

@@ -55,6 +55,11 @@ void SelectionController::Execute( SfxRequest& /*rReq*/ )
 {
 }
 
+bool SelectionController::HasMarked()
+{
+    return false;
+}
+
 bool SelectionController::DeleteMarked()
 {
     return false;

svx/source/table/tablecontroller.cxx

@@ -1325,41 +1325,41 @@ void SvxTableController::DistributeRows()
     }
 }
 
+bool SvxTableController::HasMarked()
+{
+    return mbCellSelectionMode && mxTable.is();
+}
+
 bool SvxTableController::DeleteMarked()
 {
-    if( mbCellSelectionMode )
+    if (!HasMarked())
+        return false;
+
+    const bool bUndo = mpModel && mpModel->IsUndoEnabled();
+    if (bUndo)
+        mpModel->BegUndo(ImpGetResStr(STR_TABLE_DELETE_CELL_CONTENTS));
+
+    CellPos aStart, aEnd;
+    getSelectedCells( aStart, aEnd );
+    for( sal_Int32 nRow = aStart.mnRow; nRow <= aEnd.mnRow; nRow++ )
     {
-        if( mxTable.is() )
+        for( sal_Int32 nCol = aStart.mnCol; nCol <= aEnd.mnCol; nCol++ )
         {
-            const bool bUndo = mpModel && mpModel->IsUndoEnabled();
-            if (bUndo)
-                mpModel->BegUndo(ImpGetResStr(STR_TABLE_DELETE_CELL_CONTENTS));
-
-            CellPos aStart, aEnd;
-            getSelectedCells( aStart, aEnd );
-            for( sal_Int32 nRow = aStart.mnRow; nRow <= aEnd.mnRow; nRow++ )
+            CellRef xCell( dynamic_cast< Cell* >( mxTable->getCellByPosition( nCol, nRow ).get() ) );
+            if (xCell.is() && xCell->hasText())
             {
-                for( sal_Int32 nCol = aStart.mnCol; nCol <= aEnd.mnCol; nCol++ )
-                {
-                    CellRef xCell( dynamic_cast< Cell* >( mxTable->getCellByPosition( nCol, nRow ).get() ) );
-                    if (xCell.is() && xCell->hasText())
-                    {
-                        if (bUndo)
-                            xCell->AddUndo();
-                        xCell->SetOutlinerParaObject(0);
-                    }
-                }
+                if (bUndo)
+                    xCell->AddUndo();
+                xCell->SetOutlinerParaObject(0);
             }
-
-            if (bUndo)
-                mpModel->EndUndo();
-
-            UpdateTableShape();
-            return true;
         }
     }
 
-    return false;
+    if (bUndo)
+        mpModel->EndUndo();
+
+    UpdateTableShape();
+    return true;
 }
 
 bool SvxTableController::GetStyleSheet( SfxStyleSheet*& rpStyleSheet ) const