Animations: Drop checks for duplicate jobs

We cannot return the same job multiple times from transition(). The
calling code takes ownership of the job, and even retaining a pointer to
it in the animation would be a problem. There are no animations that
return the same job multiple times, though.

Task-number: QTBUG-90401
Change-Id: I3a113c624a20dceee745cd6a4fca6f3634bdbdd1
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>

1 files changed, 9 insertions, 10 deletions

diff --git a/src/quick/util/qquickbehavior.cpp b/src/quick/util/qquickbehavior.cpp
index ceda5589b2..70e283bdb0 100644
--- a/src/quick/util/qquickbehavior.cpp
+++ b/src/quick/util/qquickbehavior.cpp
@@ -308,21 +308,20 @@ void QQuickBehavior::write(const QVariant &value)
     actions << action;
 
     QList<QQmlProperty> after;
-    QAbstractAnimationJob *prev = d->animationInstance;
-    d->animationInstance = d->animation->transition(actions, after, QQuickAbstractAnimation::Forward);
-
-    if (d->animationInstance && d->animation->threadingModel() == QQuickAbstractAnimation::RenderThread)
-        d->animationInstance = new QQuickAnimatorProxyJob(d->animationInstance, d->animation);
-
-    if (prev && prev != d->animationInstance)
-        delete prev;
+    auto *newInstance = d->animation->transition(actions, after, QQuickAbstractAnimation::Forward);
+    Q_ASSERT(newInstance != d->animationInstance);
+    delete d->animationInstance;
+    d->animationInstance = newInstance;
 
     if (d->animationInstance) {
-        if (d->animationInstance != prev)
-            d->animationInstance->addAnimationChangeListener(d, QAbstractAnimationJob::StateChange);
+        if (d->animation->threadingModel() == QQuickAbstractAnimation::RenderThread)
+            d->animationInstance = new QQuickAnimatorProxyJob(d->animationInstance, d->animation);
+
+        d->animationInstance->addAnimationChangeListener(d, QAbstractAnimationJob::StateChange);
         d->animationInstance->start();
         d->blockRunningChanged = false;
     }
+
     if (!after.contains(d->property))
         QQmlPropertyPrivate::write(d->property, value, QQmlPropertyData::BypassInterceptor | QQmlPropertyData::DontRemoveBinding);
 }