EditorManager: Make closeEditor public

And use it where appropriate

Change-Id: I0f37b8aada6eaa9be6743724b91a59173a01cb0c
Reviewed-by: hjk <hjk121@nokiamail.com>
Reviewed-by: David Schulz <david.schulz@digia.com>

src/plugins/coreplugin/editormanager/editormanager.cpp

@@ -1009,11 +1009,11 @@ void EditorManager::setCurrentEditorFromContextChange()
     setCurrentEditor(newCurrent);
 }
 
-void EditorManager::closeEditor(Core::IEditor *editor)
+void EditorManager::closeEditor(Core::IEditor *editor, bool askAboutModifiedEditors)
 {
     if (!editor)
         return;
-    closeEditors(QList<IEditor *>() << editor);
+    closeEditors(QList<IEditor *>() << editor, askAboutModifiedEditors);
 }
 
 void EditorManager::closeEditor(DocumentModel::Entry *entry)

src/plugins/coreplugin/editormanager/editormanager.h

@@ -142,6 +142,7 @@ public:
     bool saveEditor(IEditor *editor);
 
     bool closeEditors(const QList<IEditor *> &editorsToClose, bool askAboutModifiedEditors = true);
+    void closeEditor(IEditor *editor, bool askAboutModifiedEditors = true);
 
     MakeWritableResult makeFileWritable(IDocument *document);
 
@@ -268,7 +269,6 @@ private:
     static Internal::EditorView *viewForEditor(IEditor *editor);
     static Internal::SplitterOrView *findRoot(const Internal::EditorView *view, int *rootIndex = 0);
 
-    void closeEditor(IEditor *editor);
     void closeView(Internal::EditorView *view);
     void emptyView(Internal::EditorView *view);
     static void splitNewWindow(Internal::EditorView *view);

src/plugins/cppeditor/cppdoxygen_test.cpp

@@ -115,7 +115,7 @@ TestCase::TestCase(const QByteArray &input)
 
 TestCase::~TestCase()
 {
-    EditorManager::instance()->closeEditors(QList<Core::IEditor *>() << editor, false);
+    EditorManager::instance()->closeEditor(editor, false);
     QCoreApplication::processEvents(); // process any pending events
 
     // Remove the test file from the code-model

src/plugins/cpptools/cppmodelmanager_test.cpp

@@ -540,8 +540,7 @@ void CppToolsPlugin::test_modelmanager_gc_if_last_cppeditor_closed()
     QVERIFY(mm->snapshot().contains(file));
 
     // Close file/editor
-    const QList<Core::IEditor*> editorsToClose = QList<Core::IEditor*>() << editor;
+    em->closeEditor(editor, /*askAboutModifiedEditors=*/ false);
-    em->closeEditors(editorsToClose, /*askAboutModifiedEditors=*/ false);
 
     // Check: File is removed from the snapshpt
     QVERIFY(!mm->workingCopy().contains(file));

src/plugins/cvs/cvsplugin.cpp

@@ -1212,8 +1212,7 @@ bool CvsPlugin::describe(const QString &repositoryPath,
 void CvsPlugin::submitCurrentLog()
 {
     m_submitActionTriggered = true;
-    EditorManager::instance()->closeEditors(QList<IEditor*>()
+    EditorManager::instance()->closeEditor(EditorManager::currentEditor());
-        << EditorManager::currentEditor());
 }
 
 // Run CVS. At this point, file arguments must be relative to

src/plugins/debugger/disassembleragent.cpp

@@ -124,10 +124,8 @@ DisassemblerAgentPrivate::DisassemblerAgentPrivate()
 
 DisassemblerAgentPrivate::~DisassemblerAgentPrivate()
 {
-    if (editor) {
+    if (editor)
-        EditorManager *editorManager = EditorManager::instance();
+        EditorManager::instance()->closeEditor(editor);
-        editorManager->closeEditors(QList<IEditor *>() << editor);
-    }
     editor = 0;
     delete locationMark;
     qDeleteAll(breakpointMarks);

src/plugins/debugger/sourceagent.cpp

@@ -76,10 +76,8 @@ SourceAgentPrivate::SourceAgentPrivate()
 
 SourceAgentPrivate::~SourceAgentPrivate()
 {
-    if (editor) {
+    if (editor)
-        EditorManager *editorManager = EditorManager::instance();
+        EditorManager::instance()->closeEditor(editor);
-        editorManager->closeEditors(QList<IEditor *>() << editor);
-    }
     editor = 0;
     delete locationMark;
 }

src/plugins/fakevim/fakevimplugin.cpp

@@ -1865,13 +1865,10 @@ void FakeVimPluginPrivate::handleDelayedQuit(bool forced, IEditor *editor)
     // This tries to simulate vim behaviour. But the models of vim and
     // Qt Creator core do not match well...
     EditorManager *editorManager = ICore::editorManager();
-    if (editorManager->hasSplitter()) {
+    if (editorManager->hasSplitter())
         triggerAction(Core::Constants::REMOVE_CURRENT_SPLIT);
-    } else {
+    else
-        QList<IEditor *> editors;
+        editorManager->closeEditor(editor, !forced);
-        editors.append(editor);
-        editorManager->closeEditors(editors, !forced);
-    }
 }
 
 void FakeVimPluginPrivate::handleDelayedQuitAll(bool forced)

src/plugins/git/gitsubmiteditor.cpp

@@ -181,7 +181,7 @@ void GitSubmitEditor::updateFileModel()
     } else {
         VcsBase::VcsBaseOutputWindow::instance()->append(errorMessage);
         m_forceClose = true;
-        Core::EditorManager::instance()->closeEditors(QList<IEditor*>() << this);
+        Core::EditorManager::instance()->closeEditor(this);
     }
 }
 

src/plugins/perforce/perforceplugin.cpp

@@ -1296,8 +1296,7 @@ void PerforcePlugin::describe(const QString & source, const QString &n)
 void PerforcePlugin::submitCurrentLog()
 {
     m_submitActionTriggered = true;
-    Core::EditorManager *em = Core::EditorManager::instance();
+    Core::EditorManager::instance()->closeEditor(Core::EditorManager::currentEditor());
-    em->closeEditors(QList<Core::IEditor*>() << Core::EditorManager::currentEditor());
 }
 
 void PerforcePlugin::cleanCommitMessageFile()