diff options
author | A. Wilcox <AWilcox@Wilcox-Tech.com> | 2017-12-29 02:44:11 -0600 |
---|---|---|
committer | A. Wilcox <AWilcox@Wilcox-Tech.com> | 2017-12-29 02:44:11 -0600 |
commit | cf5e3c7c71d481c4c5005c8353587dccea4238fa (patch) | |
tree | 4a9437eb05b6d5f97506a2cdd8541f6431a45206 /user/plasma-workspace/fix-crash-on-wallpaper-change.patch | |
parent | 54252e3f3ad2451193df2e01af5d2acfad1db83e (diff) | |
download | packages-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.patch | 58 |
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 + |