Android: Small cleanup in AndroidRunnerWorker

Remove unused constant and forward declaration.
Use nested namespaces.

Change-Id: Iccb7e2a836b38290da38c40532ebafc4b07c25a2
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>

src/plugins/android/androidrunnerworker.cpp

@@ -47,8 +47,7 @@ using namespace std;
 using namespace std::chrono_literals;
 using namespace std::placeholders;
 
-namespace Android {
+namespace Android::Internal {
-namespace Internal {
 
 static const QString pidPollingScript = QStringLiteral("while [ -d /proc/%1 ]; do sleep 1; done");
 static const QRegularExpression userIdPattern("u(\\d+)_a");
@@ -861,5 +860,4 @@ void AndroidRunnerWorker::onProcessIdChanged(const PidUserPair &pidUser)
     }
 }
 
-} // namespace Internal
+} // namespace Android::Internal
-} // namespace Android

src/plugins/android/androidrunnerworker.h

@@ -11,19 +11,11 @@
 #include <utils/environment.h>
 #include <utils/port.h>
 
-namespace Utils {
+namespace Android { class AndroidDeviceInfo; }
-class FilePath;
-class Process;
-}
 namespace ProjectExplorer { class RunWorker; }
+namespace Utils { class Process; }
 
-namespace Android {
+namespace Android::Internal {
-
-class AndroidDeviceInfo;
-
-namespace Internal {
-
-const int MIN_SOCKET_HANDSHAKE_PORT = 20001;
 
 using PidUserPair = std::pair<qint64, qint64>;
 
@@ -109,5 +101,4 @@ private:
     bool m_useAppParamsForQmlDebugger = false;
 };
 
-} // namespace Internal
+} // namespace Android::Internal
-} // namespace Android