summaryrefslogtreecommitdiff
path: root/vcl
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2017-08-29 10:00:38 +0100
committerCaolán McNamara <caolanm@redhat.com>2017-08-29 13:40:53 +0200
commit4b44a42b6a54ddf57635fcdb9cf9c18c5e631ff1 (patch)
treee5c306a7722cb0700006df4ced1194b8c78906bb /vcl
parent6748074004a173876cfd54bff67e24865ba41acb (diff)
Resolves: tdf#111865 ensure draw after SetProgressValue uses new value
and always erase the background so we can go backwards as well as forwards Change-Id: Ibdf215320d0eeb3392cd21d3e1fd5c0535e6cd31 Reviewed-on: https://gerrit.libreoffice.org/41675 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'vcl')
-rw-r--r--vcl/source/window/status.cxx12
1 files changed, 6 insertions, 6 deletions
diff --git a/vcl/source/window/status.cxx b/vcl/source/window/status.cxx
index 131483888f31..3829a5aa99bf 100644
--- a/vcl/source/window/status.cxx
+++ b/vcl/source/window/status.cxx
@@ -1373,15 +1373,15 @@ void StatusBar::SetProgressValue( sal_uInt16 nNewPercent )
SAL_WARN_IF( !mbProgressMode, "vcl", "StatusBar::SetProgressValue(): no progress mode" );
SAL_WARN_IF( nNewPercent > 100, "vcl", "StatusBar::SetProgressValue(): nPercent > 100" );
- if ( mbProgressMode
- && IsReallyVisible()
- && (!mnPercent || (mnPercent != nNewPercent)) )
+ bool bInvalidate = mbProgressMode && IsReallyVisible() && (!mnPercent || (mnPercent != nNewPercent));
+
+ mnPercent = nNewPercent;
+
+ if (bInvalidate)
{
- bool bNeedErase = ImplGetSVData()->maNWFData.mbProgressNeedsErase;
- Invalidate(maPrgsFrameRect, bNeedErase ? InvalidateFlags::NONE : InvalidateFlags::NoErase);
+ Invalidate(maPrgsFrameRect);
Update();
}
- mnPercent = nNewPercent;
}
void StatusBar::EndProgressMode()