Skip to content

Commit 5331865

Browse files
committed
rc: Don't start Firefox with -silent option, it causes FF38 crash
1 parent 025b61b commit 5331865

File tree

1 file changed

+0
-23
lines changed

1 file changed

+0
-23
lines changed

java/server/src/org/openqa/selenium/server/browserlaunchers/FirefoxChromeLauncher.java

-23
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,6 @@ protected void launch(String url) {
8383
try {
8484
homePage = new ChromeUrlConvert().convert(url);
8585
profilePath = makeCustomProfile(homePage);
86-
populateCustomProfileDirectory(profilePath);
8786

8887
log.info("Launching Firefox...");
8988
process = prepareCommand(
@@ -98,28 +97,6 @@ protected void launch(String url) {
9897
}
9998
}
10099

101-
private void populateCustomProfileDirectory(String profilePath) {
102-
/*
103-
* The first time we launch Firefox with an empty profile directory, Firefox will launch itself,
104-
* populate the profile directory, then kill/relaunch itself, so our process handle goes out of
105-
* date. So, the first time we launch Firefox, we'll start it up at an URL that will immediately
106-
* shut itself down.
107-
*/
108-
CommandLine command = prepareCommand(browserInstallation.launcherFilePath(),
109-
"-profile", profilePath,
110-
"-silent"
111-
);
112-
command.setDynamicLibraryPath(browserInstallation.libraryPath());
113-
log.info("Preparing Firefox profile...");
114-
command.execute();
115-
try {
116-
waitForFullProfileToBeCreated(20 * 1000);
117-
} catch (RuntimeException e) {
118-
command.destroy();
119-
throw e;
120-
}
121-
}
122-
123100
protected CommandLine prepareCommand(String... commands) {
124101
CommandLine command = new CommandLine(commands);
125102
command.setEnvironmentVariable("MOZ_NO_REMOTE", "1");

0 commit comments

Comments
 (0)