EnumWindows not finding hWnd from PID if program is started using QProcess

[Waoweens]

@Christian-Ehrlicher qpr->processId() returns 16228 and task manager also says 16228

qDebug() << qpr->processId()

[Christian Ehrlicher]

@Waoweens said in EnumWindows not finding hWnd from PID if program is started using QProcess:

GetWindowThreadProcessId

So this doesn't seem to return what you're expecting. Don't think Qt and the task manager lies...

See e.g. https://stackoverflow.com/questions/61246529/pid-returned-from-getwindowthreadprocessid-call-doesnt-match-the-pid-from-taskm

[Waoweens]

@Christian-Ehrlicher I modified it to list every open window

static BOOL CALLBACK enumWindowCallback(HWND hWnd, LPARAM lparam) {
    DWORD pid;
    GetWindowThreadProcessId(hWnd, &pid);
    qDebug() << hWnd << pid
    return TRUE;
}
//...
EnumWindows(enumWindowCallback, 0);

If the program is launched manually, it appears on the list.
If the program is started with QProcess, it does not appear on the list.

[Christian Ehrlicher]

@Waoweens said in EnumWindows not finding hWnd from PID if program is started using QProcess:

If the program is launched manually, it appears on the list.

How do you know the PID in this case? Also from the task manager?

[hskoglund]

Hi, it could be that QProcess launches your program not as a top-level window but as a child, and then EnumWindows() never sees it :-(
Perhaps QProcess startDetached() works better?

[Christian Ehrlicher]

@hskoglund said in EnumWindows not finding hWnd from PID if program is started using QProcess:

Hi, it could be that QProcess launches your program not as a top-level window but as a child

How should it work otherwise - the Qt process is starting the process and therefore it's the creator. startDetached might work but must not.

The msdn documentation also tells which windows are evaluated: "Retrieves the identifier of the thread that created the specified window and, optionally, the identifier of the process that created the window."

[Waoweens]

@Christian-Ehrlicher said in EnumWindows not finding hWnd from PID if program is started using QProcess:

@Waoweens said in EnumWindows not finding hWnd from PID if program is started using QProcess:

If the program is launched manually, it appears on the list.

How do you know the PID in this case? Also from the task manager?

Yes, task manager.

@hskoglund said in EnumWindows not finding hWnd from PID if program is started using QProcess:

Hi, it could be that QProcess launches your program not as a top-level window but as a child, and then EnumWindows() never sees it :-(
Perhaps QProcess startDetached() works better?

using startDetached() makes processId() return 0.

[Christian Ehrlicher]

@Waoweens said in EnumWindows not finding hWnd from PID if program is started using QProcess:

using startDetached() makes processId() return 0.

As properly documented...

I don't understand the need at all for this.

[Waoweens]

@Christian-Ehrlicher said in EnumWindows not finding hWnd from PID if program is started using QProcess:

I don't understand the need at all for this.

I want to embed an external program in a QWidget, but to do that you need the hWid of the program.

QWindow *embed = QWindow::fromWinId(hWid);
QWidget *widget = QWidget::createWindowContainer(embed)

[Christian Ehrlicher]

Then you've to find another WinAPI function or modify the external program to return the winId to the caller.

[Waoweens]

Looks like adding a delay of 1 second makes it works, so maybe it was too fast?
But now the external program closes after 1 second. I removed qpr->terminate but now the QMainWindow wont start unless the external program is closed first.
Is there a way to make them both run at the same time?

[Waoweens]

Looks like using the QProcess::stateChanged signal (and removing terminate()) works.
It uses QThread::msleep() so its not ideal, but its fine for me.

HWND myHWND;
static BOOL CALLBACK enumWindowCallback(HWND hWnd, LPARAM lparam) {
    DWORD pid;
    GetWindowThreadProcessId(hWnd, &pid);

    if (pid == lparam) {
        myHWND = hWnd;
        return FALSE;
    }
    return TRUE;
}

ProgramStarter::ProgramStarter(QObject *parent)
    : QObject{parent}
{
    QProcess *qpr = new QProcess(this);

    connect(qpr, &QProcess::stateChanged, this, [=](QProcess::ProcessState newState){
        if (newState == QProcess::Running) {
            QThread::msleep(250);
            EnumWindows(enumWindowCallback, qpr->processId());
        }
    });

    QString executable = "C:\\path\\to\\program.exe";
    qpr->start(executable);

    qDebug() << myHWND;
}