summaryrefslogtreecommitdiff
path: root/user/plasma-workspace/fix-crash-on-wallpaper-change.patch
diff options
context:
space:
mode:
authorA. Wilcox <AWilcox@Wilcox-Tech.com>2017-12-29 02:44:11 -0600
committerA. Wilcox <AWilcox@Wilcox-Tech.com>2017-12-29 02:44:11 -0600
commitcf5e3c7c71d481c4c5005c8353587dccea4238fa (patch)
tree4a9437eb05b6d5f97506a2cdd8541f6431a45206 /user/plasma-workspace/fix-crash-on-wallpaper-change.patch
parent54252e3f3ad2451193df2e01af5d2acfad1db83e (diff)
downloadpackages-cf5e3c7c71d481c4c5005c8353587dccea4238fa.tar.gz
packages-cf5e3c7c71d481c4c5005c8353587dccea4238fa.tar.bz2
packages-cf5e3c7c71d481c4c5005c8353587dccea4238fa.tar.xz
packages-cf5e3c7c71d481c4c5005c8353587dccea4238fa.zip
user/plasma-workspace: fix crash on wallpaper change
Diffstat (limited to 'user/plasma-workspace/fix-crash-on-wallpaper-change.patch')
-rw-r--r--user/plasma-workspace/fix-crash-on-wallpaper-change.patch58
1 files changed, 58 insertions, 0 deletions
diff --git a/user/plasma-workspace/fix-crash-on-wallpaper-change.patch b/user/plasma-workspace/fix-crash-on-wallpaper-change.patch
new file mode 100644
index 000000000..078a48e3a
--- /dev/null
+++ b/user/plasma-workspace/fix-crash-on-wallpaper-change.patch
@@ -0,0 +1,58 @@
+From a44d84ef47492ca60ee608996b5ab1f2849ef16e Mon Sep 17 00:00:00 2001
+From: David Edmundson <kde@davidedmundson.co.uk>
+Date: Mon, 14 Aug 2017 10:04:19 +0100
+Subject: Don't change fillMode of image just before deleting it
+
+It is still set for when a source is applied for the next
+transistion.
+
+Summary:
+This saves evaluating a new paintedSize and doing a bunch of evaluations
+which are not used.
+
+More importantly it works round a Qt animation crash.
+
+fadeAnim.running is bound to otherImage.status !== loading
+
+with the intention of delaying animations until the images are really
+loaded.
+
+By setting the fill mode and then the source, we go to loading and back,
+effectively restarting the animation from within the animation.
+This leads to a crash.
+
+CCBUG: 381105
+
+Test Plan:
+End user result is the same.
+
+Reviewers: #plasma, broulik, mart
+
+Reviewed By: #plasma, broulik, mart
+
+Subscribers: plasma-devel
+
+Tags: #plasma
+
+Differential Revision: https://phabricator.kde.org/D7248
+---
+ wallpapers/image/imagepackage/contents/ui/main.qml | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/wallpapers/image/imagepackage/contents/ui/main.qml b/wallpapers/image/imagepackage/contents/ui/main.qml
+index fc22176..0539f37 100644
+--- a/wallpapers/image/imagepackage/contents/ui/main.qml
++++ b/wallpapers/image/imagepackage/contents/ui/main.qml
+@@ -193,8 +193,8 @@ Item {
+ }
+ ScriptAction {
+ script: {
+- otherImage.fillMode = fillMode;
+ otherImage.source = "";
++ otherImage.fillMode = fillMode;
+ }
+ }
+ }
+--
+cgit v0.11.2
+