Vcs: Separate plugin and QObject parent roles for VcsSubmitEditorFactory

These are different when the factories are used as real members,
as already done in Bazaar.

Change-Id: I3c187896dcbacb5156be1543424ccacb9140a493
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

src/plugins/clearcase/clearcaseplugin.cpp

@@ -439,7 +439,7 @@ ClearCasePluginPrivate::ClearCasePluginPrivate()
 
     new ClearCaseSettingsPage(this);
 
-    new VcsSubmitEditorFactory(submitParameters, [] { return new ClearCaseSubmitEditor; }, this);
+    new VcsSubmitEditorFactory(submitParameters, [] { return new ClearCaseSubmitEditor; }, this, this);
 
     // any editor responds to describe (when clicking a version)
     const auto describeFunc = [this](const QString &source, const QString &changeNr) {

src/plugins/cvs/cvsplugin.cpp

@@ -518,7 +518,7 @@ CvsPluginPrivate::CvsPluginPrivate()
 
     new CvsSettingsPage([this] { configurationChanged(); }, &m_settings, this);
 
-    new VcsSubmitEditorFactory(submitParameters, [] { return new CvsSubmitEditor; }, this);
+    new VcsSubmitEditorFactory(submitParameters, [] { return new CvsSubmitEditor; }, this, this);
 
     const auto describeFunc = [this](const QString &source, const QString &changeNr) {
         QString errorMessage;

src/plugins/git/gitplugin.cpp

@@ -367,7 +367,7 @@ GitPluginPrivate::GitPluginPrivate()
     for (int i = 0; i < editorCount; i++)
         new VcsEditorFactory(editorParameters + i, widgetCreator, describeFunc, this);
 
-    new VcsSubmitEditorFactory(submitParameters, [] { return new GitSubmitEditor; }, this);
+    new VcsSubmitEditorFactory(submitParameters, [] { return new GitSubmitEditor; }, this, this);
 
     const QString prefix = "git";
     m_commandLocator = new CommandLocator("Git", prefix, prefix, this);

src/plugins/mercurial/mercurialplugin.cpp

@@ -256,7 +256,7 @@ MercurialPluginPrivate::MercurialPluginPrivate()
     for (auto &editor : editorParameters)
         new VcsEditorFactory(&editor, widgetCreator, describeFunc, this);
 
-    new VcsSubmitEditorFactory(submitEditorParameters, [] { return new CommitEditor; }, this);
+    new VcsSubmitEditorFactory(submitEditorParameters, [] { return new CommitEditor; }, this, this);
 
     const QString prefix = QLatin1String("hg");
     m_commandLocator = new Core::CommandLocator("Mercurial", prefix, prefix, this);

src/plugins/perforce/perforceplugin.cpp

@@ -381,7 +381,7 @@ PerforcePluginPrivate::PerforcePluginPrivate()
     m_settings.fromSettings(ICore::settings());
 
     // Editor factories
-    new VcsSubmitEditorFactory(submitParameters, [] { return new PerforceSubmitEditor; }, this);
+    new VcsSubmitEditorFactory(submitParameters, [] { return new PerforceSubmitEditor; }, this, this);
 
     const auto describeFunc = [this](const QString &source, const QString &n) {
         describe(source, n);

src/plugins/subversion/subversionplugin.cpp

@@ -388,7 +388,7 @@ SubversionPluginPrivate::SubversionPluginPrivate()
     using namespace Core::Constants;
     Context context(SUBVERSION_CONTEXT);
 
-    new VcsSubmitEditorFactory(submitParameters, [] { return new SubversionSubmitEditor; }, this);
+    new VcsSubmitEditorFactory(submitParameters, [] { return new SubversionSubmitEditor; }, this, this);
 
     const auto describeFunc = [this](const QString &source, const QString &id) {
         describe(source, id);

src/plugins/vcsbase/basevcssubmiteditorfactory.cpp

@@ -43,8 +43,9 @@ const char DIFF_SELECTED[] = "Vcs.DiffSelectedFiles";
 VcsSubmitEditorFactory::VcsSubmitEditorFactory
         (const VcsBaseSubmitEditorParameters &parameters,
          const EditorCreator &editorCreator,
-         VcsBasePluginPrivate *plugin)
-    : IEditorFactory(plugin)
+         VcsBasePluginPrivate *plugin,
+         QObject *parent)
+    : IEditorFactory(parent)
 {
     setId(parameters.id);
     setDisplayName(QLatin1String(parameters.displayName));

src/plugins/vcsbase/basevcssubmiteditorfactory.h

@@ -49,7 +49,8 @@ public:
 
     VcsSubmitEditorFactory(const VcsBaseSubmitEditorParameters &parameters,
                            const EditorCreator &editorCreator,
-                           VcsBasePluginPrivate *plugin);
+                           VcsBasePluginPrivate *plugin,
+                           QObject *parent = nullptr);
 
 private:
     QAction *m_submitAction = nullptr;