[Libreoffice-commits] core.git: sc/source

[Caolán McNamara]

 sc/source/ui/inc/tabview.hxx   |    6 ------
 sc/source/ui/view/gridwin4.cxx |    2 ++
 sc/source/ui/view/tabview.cxx  |    4 ----
 sc/source/ui/view/tabview3.cxx |   10 ----------
 sc/source/ui/view/tabview4.cxx |    7 ++++++-
 5 files changed, 8 insertions(+), 21 deletions(-)

New commits:
commit 642d64d8fe54b7577fb4184f1ad6e0e8b3f809c4
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Tue Jul 15 16:42:42 2014 +0100

    scrolling very slow in calc
    
    even on a short spreadsheet scrolling up and down leaves
    the first/last row (depending on the direction of scroll)
    unchanged until the scrolling stops.
    
    http://people.freedesktop.org/~mst/calc_4.2_scrolling.webm
    
    On larger document there are rendering artifacts during scrolling which go away
    after scrolling for me and mstahl, but a bunch of people can show us piles of
    horribly broken spreadsheets after scrolling, esp wheel scrolling
    
    Revert "fdo#75026: Sometimes we need to update grid view...
    
    while not being active."
    
    This reverts commit 52cc88d6191ba0c4b6477e5c4b9c5d0f0228030d.
    
    Revert "fdo#68961: Check visible range during scrolling, and re-paint if necessary."
    
    This reverts commit e36c8a674845ab19577fc06d44b780549757e1e7.
    
    Revert "Repaint grid view when the visible area changes."
    
    This reverts commit b54c1a53b4d400b1c2d282c186af1fa8f151894e.
    
    Conflicts:
    	sc/source/ui/app/scmod.cxx
    
    Revert "Update visible ranges when updating the scroll bars."
    
    This reverts commit 391a57ef65687f2e373bac8d410e551aafa780ec.
    
    Change-Id: Ie170308cba18a9a74c7c72daf07dfa0a4ef7bd13
    Reviewed-on: https://gerrit.libreoffice.org/10350
    Tested-by: Michael Stahl <mstahl at redhat.com>
    Reviewed-by: Kohei Yoshida <libreoffice at kohei.us>
    Tested-by: Kohei Yoshida <libreoffice at kohei.us>

diff --git a/sc/source/ui/inc/tabview.hxx b/sc/source/ui/inc/tabview.hxx
index a5d7392..891af9e 100644
--- a/sc/source/ui/inc/tabview.hxx
+++ b/sc/source/ui/inc/tabview.hxx
@@ -204,12 +204,6 @@ private:
 
     void            PaintRangeFinderEntry (ScRangeFindData* pData, SCTAB nTab);
 
-    /**
-     * Check the visible grid area to see if the visible range has changed. If
-     * so, update the stored visible range, and re-paint the grid area.
-     */
-    void UpdateGrid();
-
 protected:
     void            UpdateHeaderWidth( const ScVSplitPos* pWhich = NULL,
                                         const SCROW* pPosY = NULL );
diff --git a/sc/source/ui/view/gridwin4.cxx b/sc/source/ui/view/gridwin4.cxx
index fa697ee5..a60fb82 100644
--- a/sc/source/ui/view/gridwin4.cxx
+++ b/sc/source/ui/view/gridwin4.cxx
@@ -390,6 +390,8 @@ void ScGridWindow::Draw( SCCOL nX1, SCROW nY1, SCCOL nX2, SCROW nY2, ScUpdateMod
 
     OSL_ENSURE( ValidCol(nX2) && ValidRow(nY2), "GridWin Draw Bereich zu gross" );
 
+    UpdateVisibleRange();
+
     if (nX2 < maVisibleRange.mnCol1 || nY2 < maVisibleRange.mnRow1)
         return;
                     // unsichtbar
diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index aafd114..02b99b9 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -1198,8 +1198,6 @@ void ScTabView::ScrollX( long nDeltaX, ScHSplitPos eWhich, bool bUpdBars )
         if (pColOutline[eWhich]) pColOutline[eWhich]->ScrollPixel( nDiff );
         if (bUpdBars)
             UpdateScrollBars();
-        else
-            UpdateGrid();
     }
 
     if (nDeltaX==1 || nDeltaX==-1)
@@ -1285,8 +1283,6 @@ void ScTabView::ScrollY( long nDeltaY, ScVSplitPos eWhich, bool bUpdBars )
         if (pRowOutline[eWhich]) pRowOutline[eWhich]->ScrollPixel( nDiff );
         if (bUpdBars)
             UpdateScrollBars();
-        else
-            UpdateGrid();
     }
 
     if (nDeltaY==1 || nDeltaY==-1)
diff --git a/sc/source/ui/view/tabview3.cxx b/sc/source/ui/view/tabview3.cxx
index 7053037..50066ba 100644
--- a/sc/source/ui/view/tabview3.cxx
+++ b/sc/source/ui/view/tabview3.cxx
@@ -2089,16 +2089,6 @@ void ScTabView::PaintRangeFinderEntry (ScRangeFindData* pData, const SCTAB nTab)
     }
 }
 
-void ScTabView::UpdateGrid()
-{
-    if (!UpdateVisibleRange())
-        // Visible range hasn't changed. No need to re-paint.
-        return;
-
-    SC_MOD()->AnythingChanged();                // if visible area has changed
-    PaintGrid();
-}
-
 void ScTabView::PaintRangeFinder( long nNumber )
 {
     ScInputHandler* pHdl = SC_MOD()->GetInputHdl( aViewData.GetViewShell() );
diff --git a/sc/source/ui/view/tabview4.cxx b/sc/source/ui/view/tabview4.cxx
index 2f72a20..8528431 100644
--- a/sc/source/ui/view/tabview4.cxx
+++ b/sc/source/ui/view/tabview4.cxx
@@ -421,7 +421,12 @@ void ScTabView::UpdateScrollBars()
     }
 
     //      set visible area for online spelling
-    UpdateGrid();
+
+    if ( aViewData.IsActive() )
+    {
+        if (UpdateVisibleRange())
+            SC_MOD()->AnythingChanged();                // if visible area has changed
+    }
 }
 
 #ifndef HDR_SLIDERSIZE