commit 9d7fdcd6b5c7b5a9fe781bb99b135fba7d04ddec
author Caolán McNamara <caolanm@redhat.com> Thu Mar 19 11:51:26 2020 +0000
committer Caolán McNamara <caolanm@redhat.com> Thu Mar 19 13:59:37 2020 +0100
tree 0b738817ffb99abaa756b3815f88042ce77027fc
parent 3705a2904d50210e3515723ad24bbbb6a72eb37f

redraw problems seen on scroll in SvTreeListBox

SvTreeListBox::Invalidate override Control::Invalidate and additionally calls
Invalidate on its SvImpLBox pImpl which calls SetClipRegion on m_pView which is
the original SvTreeListBox, which is all very confusing, revert SvTreeListBox
to how it ~always was.

Change-Id: I03cf89e7ed7cf0069cc70d4250a3bb797e3487e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90745
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

vcl/source/treelist/svimpbox.cxx

diff --git a/vcl/source/treelist/svimpbox.cxx b/vcl/source/treelist/svimpbox.cxx
index 50672b2..bc40721 100644
--- a/vcl/source/treelist/svimpbox.cxx
+++ b/vcl/source/treelist/svimpbox.cxx
@@ -394,11 +394,13 @@ void SvImpLBox::PageDown( sal_uInt16 nDelta )
    ShowCursor( false );

    m_nFlags &= ~LBoxFlags::Filling;
    m_pView->Update();
    m_pStartEntry = pNext;

    if( nRealDelta >= m_nVisibleCount )
    {
        m_pView->Invalidate( GetVisibleArea() );
        m_pView->Update();
    }
    else
    {
@@ -430,10 +432,12 @@ void SvImpLBox::PageUp( sal_uInt16 nDelta )
    m_nFlags &= ~LBoxFlags::Filling;
    ShowCursor( false );

    m_pView->Update();
    m_pStartEntry = pPrev;
    if( nRealDelta >= m_nVisibleCount )
    {
        m_pView->Invalidate( GetVisibleArea() );
        m_pView->Update();
    }
    else
    {
@@ -3264,6 +3268,7 @@ IMPL_LINK(SvImpLBox, MyUserEvent, void*, pArg, void )
    if( !pArg )
    {
        m_pView->Invalidate();
        m_pView->Update();
    }
    else
    {