Debugger: Simplify raise-on-stop logic

Also slightly change meaning (now all stops may cause raising).

Change-Id: Ie3fee7371ab119e4e9747dc44a4d8ef2cd62c8d8
Reviewed-by: Eike Ziller <eike.ziller@digia.com>

src/plugins/debugger/debuggerengine.cpp

@@ -49,6 +49,7 @@
 
 #include <coreplugin/editormanager/editormanager.h>
 #include <coreplugin/editormanager/ieditor.h>
+#include <coreplugin/icore.h>
 #include <coreplugin/progressmanager/progressmanager.h>
 #include <coreplugin/progressmanager/futureprogress.h>
 
@@ -944,7 +945,7 @@ void DebuggerEngine::notifyInferiorSpontaneousStop()
     showStatusMessage(tr("Stopped."));
     setState(InferiorStopOk);
     if (debuggerCore()->boolSetting(RaiseOnInterrupt))
-        emit raiseWindow();
+        ICore::raiseWindow(debuggerCore()->mainWindow());
 }
 
 void DebuggerEngine::notifyInferiorStopFailed()

src/plugins/debugger/debuggerengine.h

@@ -283,7 +283,6 @@ signals:
      * a server start script should be used, but none is given.
      */
     void requestRemoteSetup();
-    void raiseWindow();
 
 protected:
     // The base notify*() function implementation should be sufficient

src/plugins/debugger/debuggermainwindow.cpp

@@ -29,7 +29,6 @@
 
 #include "debuggermainwindow.h"
 #include "debuggercore.h"
-#include "debuggerengine.h"
 #include "debuggerrunconfigurationaspect.h"
 
 #include <utils/appmainwindow.h>
@@ -131,8 +130,6 @@ public:
     Project *m_previousProject;
     Target *m_previousTarget;
     RunConfiguration *m_previousRunConfiguration;
-
-    DebuggerEngine *m_engine;
 };
 
 DebuggerMainWindowPrivate::DebuggerMainWindowPrivate(DebuggerMainWindow *mw)
@@ -150,7 +147,6 @@ DebuggerMainWindowPrivate::DebuggerMainWindowPrivate(DebuggerMainWindow *mw)
     , m_previousProject(0)
     , m_previousTarget(0)
     , m_previousRunConfiguration(0)
-    , m_engine(0)
 {
     m_debugToolBarLayout->setMargin(0);
     m_debugToolBarLayout->setSpacing(0);
@@ -266,15 +262,6 @@ DebuggerMainWindow::~DebuggerMainWindow()
     delete d;
 }
 
-void DebuggerMainWindow::setCurrentEngine(DebuggerEngine *engine)
-{
-    if (d->m_engine)
-        disconnect(d->m_engine, SIGNAL(raiseWindow()), ICore::mainWindow(), SLOT(raiseWindow()));
-    d->m_engine = engine;
-    if (d->m_engine)
-        connect(d->m_engine, SIGNAL(raiseWindow()), ICore::mainWindow(), SLOT(raiseWindow()));
-}
-
 DebuggerLanguages DebuggerMainWindow::activeDebugLanguages() const
 {
     return d->m_activeDebugLanguages;

src/plugins/debugger/debuggermainwindow.h

@@ -69,8 +69,6 @@ public:
     DebuggerMainWindow();
     ~DebuggerMainWindow();
 
-    void setCurrentEngine(DebuggerEngine *engine);
-
     // Debugger toolbars are registered with this function.
     void setToolBar(DebuggerLanguage language, QWidget *widget);
 

src/plugins/debugger/debuggerplugin.cpp

@@ -2109,7 +2109,6 @@ void DebuggerPluginPrivate::connectEngine(DebuggerEngine *engine)
     m_inspectorWindow->setModel(engine->watchModel());
 
     mainWindow()->setEngineDebugLanguages(engine->startParameters().languages);
-    mainWindow()->setCurrentEngine(engine);
 }
 
 static void changeFontSize(QWidget *widget, qreal size)