Ssh: Use base QtcProcess' useTerminal for SshRemoteProcess

Still trying to ramp down non-QtcProcess interfaces.

Change-Id: I6b4ae6788f5021959ff3eb35bbade2157a6d67fa
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Cristian Adam <cristian.adam@qt.io>
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>

src/libs/ssh/sshremoteprocess.cpp

@@ -57,7 +57,6 @@ struct SshRemoteProcess::SshRemoteProcessPrivate
     QString remoteCommand;
     QStringList connectionArgs;
     QString displayName;
-    bool useTerminal = false;
 };
 
 SshRemoteProcess::SshRemoteProcess(const QString &command, const QStringList &connectionArgs,
@@ -101,11 +100,6 @@ SshRemoteProcess::~SshRemoteProcess()
     delete d;
 }
 
-void SshRemoteProcess::requestTerminal()
-{
-    d->useTerminal = true;
-}
-
 void SshRemoteProcess::requestX11Forwarding(const QString &displayName)
 {
     d->displayName = displayName;
@@ -122,7 +116,7 @@ Utils::CommandLine SshRemoteProcess::fullLocalCommandLine() const
 
     if (!d->displayName.isEmpty())
         cmd.addArg("-X");
-    if (d->useTerminal)
+    if (useTerminal())
         cmd.addArg("-tt");
 
     cmd.addArg("-q");

src/libs/ssh/sshremoteprocess.h

@@ -43,7 +43,6 @@ class QSSH_EXPORT SshRemoteProcess : public SshProcess
 public:
     ~SshRemoteProcess();
 
-    void requestTerminal();
     void requestX11Forwarding(const QString &displayName);
     void start();
 

src/libs/ssh/sshremoteprocessrunner.cpp

@@ -128,8 +128,7 @@ void SshRemoteProcessRunner::handleConnected()
             this, &SshRemoteProcessRunner::handleStdout);
     connect(d->m_process.get(), &SshRemoteProcess::readyReadStandardError,
             this, &SshRemoteProcessRunner::handleStderr);
-    if (d->m_runInTerminal)
+    d->m_process->setUseTerminal(d->m_runInTerminal);
-        d->m_process->requestTerminal();
     d->m_process->start();
 }
 

src/libs/utils/qtcprocess.cpp

@@ -824,6 +824,11 @@ void QtcProcess::setUseTerminal(bool on)
     d->m_process->setUseTerminal(on);
 }
 
+bool QtcProcess::useTerminal() const
+{
+    return d->m_process->useTerminal();
+}
+
 void QtcProcess::setRunAsRoot(bool on)
 {
     d->m_process->setRunAsRoot(on);

src/libs/utils/qtcprocess.h

@@ -104,9 +104,11 @@ public:
     void setUseCtrlCStub(bool enabled);
     void setLowPriority();
     void setDisableUnixTerminal();
-    void setUseTerminal(bool on);
     void setRunAsRoot(bool on);
 
+    void setUseTerminal(bool on);
+    bool useTerminal() const;
+
     void start();
     void terminate();
     void interrupt();

src/plugins/projectexplorer/devicesupport/sshdeviceprocess.cpp

@@ -194,7 +194,7 @@ void SshDeviceProcess::handleConnected()
     if (!display.isEmpty())
         d->process->requestX11Forwarding(display);
     if (runInTerminal()) {
-        d->process->requestTerminal();
+        d->process->setUseTerminal(true);
         connect(&d->consoleProcess, &ConsoleProcess::errorOccurred,
                 this, &DeviceProcess::error);
         connect(&d->consoleProcess, &ConsoleProcess::processStarted,