diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2017-08-29 10:00:38 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2017-08-29 13:40:53 +0200 |
| commit | 4b44a42b6a54ddf57635fcdb9cf9c18c5e631ff1 (patch) | |
| tree | e5c306a7722cb0700006df4ced1194b8c78906bb | |
| parent | 6748074004a173876cfd54bff67e24865ba41acb (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>
| -rw-r--r-- | vcl/source/window/status.cxx | 12 |
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() |