Core: Make parent argument last in ProgressTimer ctor

Change-Id: Id6513c46c72f0f4729783136ba0e9d779af7d147
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

src/plugins/coreplugin/progressmanager/progressmanager.cpp

@@ -763,7 +763,7 @@ FutureProgress *ProgressManager::addTimedTask(const QFutureInterface<void> &futu
 {
     QFutureInterface<void> dummy(futureInterface); // Need mutable to access .future()
     FutureProgress *fp = m_instance->doAddTask(dummy.future(), title, type, flags);
-    (void) new ProgressTimer(fp, futureInterface, expectedSeconds);
+    (void) new ProgressTimer(futureInterface, expectedSeconds, fp);
     return fp;
 }
 
@@ -779,9 +779,9 @@ void ProgressManager::cancelTasks(Id type)
 }
 
 
-ProgressTimer::ProgressTimer(QObject *parent,
-                             const QFutureInterfaceBase &futureInterface,
-                             int expectedSeconds)
+ProgressTimer::ProgressTimer(const QFutureInterfaceBase &futureInterface,
+                             int expectedSeconds,
+                             QObject *parent)
     : QObject(parent),
       m_futureInterface(futureInterface),
       m_expectedTime(expectedSeconds),

src/plugins/coreplugin/progressmanager/progressmanager.h

@@ -75,7 +75,8 @@ private:
 class CORE_EXPORT ProgressTimer : public QObject
 {
 public:
-    ProgressTimer(QObject *parent, const QFutureInterfaceBase &futureInterface, int expectedSeconds);
+    ProgressTimer(const QFutureInterfaceBase &futureInterface, int expectedSeconds,
+                  QObject *parent = 0);
 
 private:
     void handleTimeout();