Workaround marionette 3.1.0 switch_to_window bug in Firefox 79#24992
Workaround marionette 3.1.0 switch_to_window bug in Firefox 79#24992stephenmcgruer merged 3 commits intomasterfrom
Conversation
|
I intend to move this to a method on the protocol, and the documentation needs some work. It's not clear to me still what exactly we're working around here; https://bugzilla.mozilla.org/show_bug.cgi?id=1588424 appears to state in https://bugzilla.mozilla.org/show_bug.cgi?id=1588424#c1 that geckodriver should support this already, and https://bugzilla.mozilla.org/show_bug.cgi?id=1588424#c7 implies we should just be on geckodriver 0.27.0 and maybe this wouldn't be a problem then? But wanted to get a Firefox Stable run triggered, so starting with that :) |
|
geckodriver isn't involved here, we're using the underlying marionette connection directly. It's just that marionette is very close to WebDriver in a way that e.g. CDP isn't. |
Ah, that makes sense. Thanks. |
|
Stable run succeeded, and tests all are green except for an azure pipelines infra error. Merging. |
Fixes #24924