debugger: fix order of special actions to start debugging

Change-Id: I0190012c4fe2ea916947561cb76e7974410fd6a1
Reviewed-by: Eike Ziller <eike.ziller@nokia.com>

2 files changed, 28 insertions, 23 deletions

diff --git a/src/plugins/debugger/debuggerconstants.h b/src/plugins/debugger/debuggerconstants.h
index a3cdc77500..bf2260608c 100644
--- a/src/plugins/debugger/debuggerconstants.h
+++ b/src/plugins/debugger/debuggerconstants.h
@@ -47,8 +47,9 @@ const char C_CPPDEBUGGER[]          = "Gdb Debugger";
 const char C_QMLDEBUGGER[]          = "Qml/JavaScript Debugger";
 
 // Menu Groups
-const char G_START_CPP[]            = "Debugger.Group.Start.Cpp";
-const char G_START_QML[]            = "Debugger.Group.Start.Cpp";
+const char G_START_LOCAL[]          = "Debugger.Group.Start.Local";
+const char G_START_REMOTE[]         = "Debugger.Group.Start.Remote";
+const char G_START_QML[]            = "Debugger.Group.Start.Qml";
 
 // Project Explorer run mode (RUN/DEBUG)
 const char DEBUGMODE[]              = "Debugger.DebugMode";
diff --git a/src/plugins/debugger/debuggerplugin.cpp b/src/plugins/debugger/debuggerplugin.cpp
index 81e083ddc2..ff5edc942c 100644
--- a/src/plugins/debugger/debuggerplugin.cpp
+++ b/src/plugins/debugger/debuggerplugin.cpp
@@ -2961,7 +2961,8 @@ void DebuggerPluginPrivate::extensionsInitialized()
     // "Start Debugging" sub-menu
     // groups:
     //   G_DEFAULT_ONE
-    //   G_START_CPP
+    //   G_START_LOCAL
+    //   G_START_REMOTE
     //   G_START_QML
 
     Command *cmd = 0;
@@ -2982,45 +2983,44 @@ void DebuggerPluginPrivate::extensionsInitialized()
     ModeManager *modeManager = ModeManager::instance();
     modeManager->addAction(m_visibleStartAction, Constants::P_ACTION_DEBUG);
 
-    cmd = am->registerAction(m_startExternalAction,
-        "Debugger.StartExternal", globalcontext);
-    cmd->setAttribute(Command::CA_Hide);
-    mstart->addAction(cmd, Constants::G_START_CPP);
-
     cmd = am->registerAction(m_attachExternalAction,
         "Debugger.AttachExternal", globalcontext);
     cmd->setAttribute(Command::CA_Hide);
-    mstart->addAction(cmd, Constants::G_START_CPP);
+    mstart->addAction(cmd, Constants::G_START_LOCAL);
+
+    cmd = am->registerAction(m_startExternalAction,
+        "Debugger.StartExternal", globalcontext);
+    cmd->setAttribute(Command::CA_Hide);
+    mstart->addAction(cmd, Constants::G_START_LOCAL);
 
     cmd = am->registerAction(m_attachCoreAction,
         "Debugger.AttachCore", globalcontext);
+    cmd->setAttribute(Command::CA_Hide);
+    mstart->addAction(cmd, Constants::G_START_LOCAL);
 
+    cmd = am->registerAction(m_attachRemoteAction,
+        "Debugger.AttachRemote", globalcontext);
     cmd->setAttribute(Command::CA_Hide);
-    mstart->addAction(cmd, Constants::G_START_CPP);
+    mstart->addAction(cmd, Constants::G_START_REMOTE);
 
     cmd = am->registerAction(m_startRemoteAction,
         "Debugger.StartRemote", globalcontext);
     cmd->setAttribute(Command::CA_Hide);
-    mstart->addAction(cmd, Constants::G_START_CPP);
-
-    cmd = am->registerAction(m_attachRemoteAction,
-        "Debugger.AttachRemote", globalcontext);
-    cmd->setAttribute(Command::CA_Hide);
-    mstart->addAction(cmd, Constants::G_START_CPP);
+    mstart->addAction(cmd, Constants::G_START_REMOTE);
 
 
 #ifdef WITH_LLDB
     cmd = am->registerAction(m_startRemoteLldbAction,
         "Debugger.RemoteLldb", globalcontext);
     cmd->setAttribute(Command::CA_Hide);
-    mstart->addAction(cmd, Constants::G_START_CPP);
+    mstart->addAction(cmd, Constants::G_START_REMOTE);
 #endif
 
     if (m_startRemoteCdbAction) {
         cmd = am->registerAction(m_startRemoteCdbAction,
              "Debugger.AttachRemoteCdb", globalcontext);
         cmd->setAttribute(Command::CA_Hide);
-        mstart->addAction(cmd, Constants::G_START_CPP);
+        mstart->addAction(cmd, Constants::G_START_REMOTE);
     }
 
     QAction *sep = new QAction(mstart);
@@ -3391,15 +3391,19 @@ bool DebuggerPlugin::initialize(const QStringList &arguments, QString *errorMess
     Core::ActionManager *am = core->actionManager();
     ActionContainer *mstart = am->actionContainer(PE::M_DEBUG_STARTDEBUGGING);
 
-    mstart->appendGroup(Constants::G_START_CPP);
+    mstart->appendGroup(Constants::G_START_LOCAL);
+    mstart->appendGroup(Constants::G_START_REMOTE);
     mstart->appendGroup(Constants::G_START_QML);
 
-    // add cpp separator
+    // Separators
     QAction *sep = new QAction(mstart);
     sep->setSeparator(true);
-    Command *cmd = am->registerAction(sep,
-        "Debugger.Start.Cpp", globalcontext);
-    mstart->addAction(cmd, Constants::G_START_CPP);
+    Command *cmd = am->registerAction(sep, "Debugger.Local.Cpp", globalcontext);
+    mstart->addAction(cmd, Constants::G_START_LOCAL);
+    sep = new QAction(mstart);
+    sep->setSeparator(true);
+    cmd = am->registerAction(sep, "Debugger.Remote.Cpp", globalcontext);
+    mstart->addAction(cmd, Constants::G_START_REMOTE);
 
     return theDebuggerCore->initialize(arguments, errorMessage);
 }