QTest::CrashHandler: move the enum definition to the header

It's common between the two implementations, though currently completely
unused on Windows because we don't start debuggers.

Change-Id: I7a42db2db35cf6191693fffd3e4e46d482032630
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Ahmad Samir <a.samirh78@gmail.com>

src/testlib/qtestcrashhandler_p.h

@@ -27,6 +27,15 @@
 QT_BEGIN_NAMESPACE
 namespace QTest {
 namespace CrashHandler {
+    enum DebuggerProgram {
+        None,
+        Gdb,
+        Lldb,
+#ifdef Q_OS_WIN
+        Cdb,
+#endif
+    };
+
     bool alreadyDebugging();
     void blockUnixSignals();
 

src/testlib/qtestcrashhandler_unix.cpp

@@ -244,8 +244,6 @@ bool alreadyDebugging()
 #endif
 }
 
-namespace {
-enum DebuggerProgram { None, Gdb, Lldb };
 static bool hasSystemCrashReporter()
 {
 #if defined(Q_OS_MACOS)
@@ -254,7 +252,6 @@ static bool hasSystemCrashReporter()
     return false;
 #endif
 }
-} // unnamed namespaced
 
 void maybeDisableCoreDump()
 {

src/testlib/qtestcrashhandler_win.cpp

@@ -33,7 +33,6 @@ void maybeDisableCoreDump()
 {
 }
 
-enum DebuggerProgram { None, Gdb, Lldb, Cdb };
 static DebuggerProgram debugger = None;
 void prepareStackTrace()
 {