Only call super-expensive Invalidate on scrollbar toggling

otherwise even using backspace in an annotated area will cause super slow
behaviour as each keystroke causes a full page render

This became a problem after

commit 0761f81643a6890457e9ef7d913ab5c88c2593a4
Date:   Thu Dec 19 18:50:58 2013 +0000
    123792: complete annotations on text ranges feature

but underlying problem was always there ready to trigger.

For this case only render the full page if the state
of comments scrollbars *toggles*, i.e. if there wasn't
scrollbars and there ends up still with no scrollbars
avoid the (bad) hack of invalidating the page

Change-Id: Ic7fd432d2317b0f5ad8d8773636f99fe75d660d6

1 files changed, 3 insertions, 2 deletions

diff --git a/sw/source/uibase/docvw/PostItMgr.cxx b/sw/source/uibase/docvw/PostItMgr.cxx
index 7af692705115..b97b1863a2ec 100644
--- a/sw/source/uibase/docvw/PostItMgr.cxx
+++ b/sw/source/uibase/docvw/PostItMgr.cxx
@@ -767,7 +767,8 @@ void SwPostItMgr::LayoutPostIts()
             }
             else
             {
-                bUpdate = true;
+                if (mPages[n]->bScrollbar)
+                    bUpdate = true;
                 mPages[n]->bScrollbar = false;
             }
         }
@@ -803,7 +804,7 @@ void SwPostItMgr::LayoutPostIts()
         // notes scrollbar is otherwise not drawn correctly for some cases
         // scrollbar area is enough
         if (bUpdate)
-            mpEditWin->Invalidate();
+            mpEditWin->Invalidate(); /*This is a super expensive relayout and render of the entire page*/
 
         mbLayouting = false;
     }