Skip to content

Commit ef44fef

Browse files
committed
Make sure that legacy firefox used the XpiDriverService
1 parent 73d1654 commit ef44fef

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

java/server/src/org/openqa/selenium/remote/server/ActiveSessionFactory.java

+3-8
Original file line numberDiff line numberDiff line change
@@ -54,17 +54,12 @@ public ActiveSessionFactory() {
5454
StreamSupport.stream(loadDriverProviders().spliterator(), false)
5555
.forEach(p -> builder.put(p::canCreateDriverInstanceFor, new InMemorySession.Factory(p)));
5656

57-
bind(
58-
builder,
59-
"org.openqa.selenium.firefox.FirefoxDriver",
60-
caps -> {
57+
ImmutableMap.<Predicate<Capabilities>, String>builder()
58+
.put(caps -> {
6159
Object marionette = caps.getCapability("marionette");
6260

6361
return marionette instanceof Boolean && !(Boolean) marionette;
64-
},
65-
firefox());
66-
67-
ImmutableMap.<Predicate<Capabilities>, String>builder()
62+
}, "org.openqa.selenium.firefox.XpiDriverService")
6863
.put(browserName(chrome()), "org.openqa.selenium.chrome.ChromeDriverService")
6964
.put(containsKey("chromeOptions"), "org.openqa.selenium.chrome.ChromeDriverService")
7065
.put(browserName(edge()), "org.openqa.selenium.edge.EdgeDriverService")

0 commit comments

Comments
 (0)