From 9861bb24b33346a73401bd5df1c5f243b245f427 Mon Sep 17 00:00:00 2001 From: Gülşah Köse Date: Mon, 23 Mar 2020 15:33:40 +0300 Subject: tdf#131254 Prevent extra eventqueue empty call. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit That call disables advance timing setting when we back to previous slide. Its comment line says "Process initial events and skip any animations that are started when the slide is shown" but it can still skip any animations. It seems that is an unnneeded call here. Change-Id: Iec23409da8285f5b7a8bff3d9c4f507a997ae885 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90913 Tested-by: Jenkins Reviewed-by: Gülşah Köse (cherry picked from commit 80f386f787ad59936ead2022e6d932a6d441c6e9) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91119 Reviewed-by: Andras Timar --- slideshow/source/engine/effectrewinder.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/slideshow/source/engine/effectrewinder.cxx b/slideshow/source/engine/effectrewinder.cxx index d8889024108b..1a1dc0636363 100644 --- a/slideshow/source/engine/effectrewinder.cxx +++ b/slideshow/source/engine/effectrewinder.cxx @@ -412,7 +412,7 @@ void EffectRewinder::asynchronousRewind ( // Process initial events and skip any animations that are started // when the slide is shown. mbNonUserTriggeredMainSequenceEffectSeen = false; - mrEventQueue.forceEmpty(); + if (mbNonUserTriggeredMainSequenceEffectSeen) { mrUserEventQueue.callSkipEffectEventHandler(); -- cgit v1.2.3