1766802 - With the Remote Agent starting earlier the "session.new" command has to wait for the browser window

Status: RESOLVED FIXED

(Remote Protocol :: WebDriver BiDi, defect, P2)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 10/03 - 10/13)]

With bug 1759169 landed the Remote Agent gets started before any browser window is open. If a direct BiDi connection requests a new session right after starting the browsre the command immediately returns and as such can cause trouble with following-up BiDi commands that require a browser window:

0:15.59 INFO STDOUT: 1651131463434	WebDriver BiDi	DEBUG	ffd9734f-c718-4690-b6fe-1fb99caf5999 -> {"id":2,"method":"browsingContext.getTree","params":{"maxDepth":0}}
 0:15.60 INFO STDOUT: 1651131463437	RemoteAgent	TRACE	Registered MessageHandlerFrame actors
 0:15.60 INFO STDOUT: 1651131463438	RemoteAgent	TRACE	Created MessageHandler ROOT for session ffd9734f-c718-4690-b6fe-1fb99caf5999
 0:15.60 INFO STDOUT: 1651131463439	RemoteAgent	TRACE	Received command browsingContext.getTree for destination ROOT
 0:15.60 INFO STDOUT: 1651131463440	RemoteAgent	TRACE	Module root/browsingContext.jsm found for ROOT
 0:15.60 INFO STDOUT: 1651131463440	WebDriver BiDi	DEBUG	ffd9734f-c718-4690-b6fe-1fb99caf5999 <- {"id":2,"result":{"contexts":[]}}

Similar to CDP we should wait for sessionstore-windows-restored which can be done with await RemoteAgent.browserStartupFinished;.

This currently blocks my work for creating a test on bug 1719505.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1759169

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 10/03 - 10/13)]

Attachment: Bug 1766802 - [remote] Delay startup of WebDriver BiDi until all initial application windows have been restored.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/67124d2fa796
[remote] Delay startup of WebDriver BiDi until all initial application windows have been restored. r=webdriver-reviewers,jdescottes

Comment 4

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/67124d2fa796