From d8bf859890742b50714cc257c5a9b364f3a5b43b Mon Sep 17 00:00:00 2001 From: Seva Lotoshnikov Date: Thu, 10 Apr 2014 18:54:47 -0700 Subject: [PATCH] Minor cleanup FirefoxBinary.java Signed-off-by: Andreas Tolfsen --- .../src/org/openqa/selenium/firefox/FirefoxBinary.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/java/client/src/org/openqa/selenium/firefox/FirefoxBinary.java b/java/client/src/org/openqa/selenium/firefox/FirefoxBinary.java index 9ea84e20f1f9b..8cb1241939ade 100644 --- a/java/client/src/org/openqa/selenium/firefox/FirefoxBinary.java +++ b/java/client/src/org/openqa/selenium/firefox/FirefoxBinary.java @@ -199,7 +199,7 @@ public void createProfile(String profileName) throws IOException { /** * Waits for the process to execute, returning the command output taken from the profile's * execution. - * + * * @throws InterruptedException if we are interrupted while waiting for the process to launch * @throws IOException if there is a problem with reading the input stream of the launching * process @@ -210,7 +210,7 @@ public void waitFor() throws InterruptedException, IOException { /** * Gets all console output of the binary. Output retrieval is non-destructive and non-blocking. - * + * * @return the console output of the executed binary. * @throws IOException */ @@ -227,6 +227,7 @@ public void clean(FirefoxProfile profile, File profileDir) throws IOException { try { waitFor(); } catch (InterruptedException e) { + process.destroy(); throw new WebDriverException(e); } }