API for default wait time when getting MainWindow

[JohanLarsson]

Now it is

public Window MainWindow => this.GetMainWindow(TimeSpan.FromSeconds(10));

Which is not very nice and the time is probably too short for real applications.
Static property is an alternative but not very nice.
Another alternative is attribute.

[davishoang96]

I have this GetMainWindow function for the program. I did try TimeSpan.FromSeconds(5) but i'm still getting infinity waiting for the respond. Do you have any idea?

    public static void GetMainWindow()
    {
        try
        {
            Process AttachProcess = WindowInteraction.GetProcess(ProcessForm.targetproc);
            
            App = Application.Attach(AttachProcess.Id);

            MainWindow = App.GetMainWindow(TimeSpan.FromSeconds(5));

        }
        catch
        {
        
            throw new Exception("Cannot get MainWindow");
        }
       
    }

[JohanLarsson]

Sounds like a bug, have you seen anything dumb in the code?

Gu.Wpf.UiAutomation/Gu.Wpf.UiAutomation/Application.cs

Lines 344 to 369 in 2916d66

	/// <summary>
	/// Waits until the main handle is set.
	/// </summary>
	/// <param name="process">The process.</param>
	/// <param name="waitTimeout">An optional timeout. If null is passed, the timeout is infinite.</param>
	public static void WaitForMainWindow(Process process, TimeSpan? waitTimeout = null)
	{
	var start = DateTime.Now;
	do
	{
	if (process.HasExited)
	{
	throw new InvalidOperationException("Process has exited.");
	}
	process.Refresh();
	if (process.MainWindowHandle != IntPtr.Zero)
	{
	return;
	}
	Wait.For(Retry.PollInterval);
	}
	while (!Retry.IsTimeouted(start, waitTimeout ?? TimeSpan.MaxValue));
	throw new TimeoutException("Did not find Process.MainWindowHandle, if startup is slow try with a longer timeout.");
	}

[davishoang96]

I did try
App.WaitForMainWindow(TimeSpan.FromSeconds(1));
Although the time is still longer than 1 secs

[JohanLarsson]

Did it timeout when you did App.WaitForMainWindow(TimeSpan.FromSeconds(1));?

[davishoang96]

Yes it did throw timeout exception

[JohanLarsson]

Do you have API ideas for this? Static property App.WaitForMainWindowTime is not awesome but perhaps not so bad.

[davishoang96]

Yeah I fixed the issue by putting: Application.WaitForMainWindow(AttachProcess, TimeSpan.FromSeconds(1)); before the App = Application.Attach(AttachProcess.Id);

It worked perfectly fine.
Thanks Johan.