iOS: Move updateAutomaticKitList() trigger closer to its user

This removes the IosPlugin::kitsRestored() member that does not
fully fit the standard plugin implementation pattern.

Change-Id: Iedd79e5326b85b189673c19e1920e21c34cd03bb
Reviewed-by: Eike Ziller <eike.ziller@qt.io>

src/plugins/ios/iosconfigurations.cpp

@@ -337,6 +337,15 @@ void IosConfigurations::initialize()
     m_instance = new IosConfigurations(nullptr);
 }
 
+void IosConfigurations::kitsRestored()
+{
+    disconnect(KitManager::instance(), &KitManager::kitsLoaded,
+               this, &IosConfigurations::kitsRestored);
+    IosConfigurations::updateAutomaticKitList();
+    connect(QtVersionManager::instance(), &QtVersionManager::qtVersionsChanged,
+            IosConfigurations::instance(), &IosConfigurations::updateAutomaticKitList);
+}
+
 bool IosConfigurations::ignoreAllDevices()
 {
     return m_instance->m_ignoreAllDevices;
@@ -386,6 +395,8 @@ IosConfigurations::IosConfigurations(QObject *parent)
     : QObject(parent)
 {
     load();
+    connect(KitManager::instance(), &KitManager::kitsLoaded,
+            this, &IosConfigurations::kitsRestored);
 }
 
 void IosConfigurations::load()

src/plugins/ios/iosconfigurations.h

@@ -129,6 +129,7 @@ private:
     IosConfigurations(QObject *parent);
     void load();
     void save();
+    void kitsRestored();
     void updateSimulators();
     static void setDeveloperPath(const Utils::FileName &devPath);
     void initializeProvisioningData();

src/plugins/ios/iosplugin.cpp

@@ -42,13 +42,10 @@
 
 #include <projectexplorer/deployconfiguration.h>
 #include <projectexplorer/devicesupport/devicemanager.h>
-#include <projectexplorer/kitmanager.h>
 #include <projectexplorer/runconfiguration.h>
 
 #include <qmakeprojectmanager/qmakeprojectmanagerconstants.h>
 
-#include <qtsupport/qtversionmanager.h>
-
 using namespace ProjectExplorer;
 using namespace QtSupport;
 
@@ -127,20 +124,8 @@ bool IosPlugin::initialize(const QStringList &arguments, QString *errorMessage)
     RunControl::registerWorker<Internal::IosQmlProfilerSupport>
             (ProjectExplorer::Constants::QML_PROFILER_RUN_MODE, constraint);
 
-    connect(KitManager::instance(), &KitManager::kitsLoaded,
-            this, &IosPlugin::kitsRestored);
-
     return true;
 }
 
-void IosPlugin::kitsRestored()
-{
-    disconnect(KitManager::instance(), &KitManager::kitsLoaded,
-               this, &IosPlugin::kitsRestored);
-    IosConfigurations::updateAutomaticKitList();
-    connect(QtVersionManager::instance(), &QtVersionManager::qtVersionsChanged,
-            IosConfigurations::instance(), &IosConfigurations::updateAutomaticKitList);
-}
-
 } // namespace Internal
 } // namespace Ios

src/plugins/ios/iosplugin.h

@@ -43,8 +43,6 @@ private:
     bool initialize(const QStringList &arguments, QString *errorMessage) final;
     void extensionsInitialized() final {}
 
-    void kitsRestored();
-
     class IosPluginPrivate *d = nullptr;
 };