Skip to content

Commit

Permalink
Replace Throwables.propagate with equivalent rethrows.
Browse files Browse the repository at this point in the history
Committed on behalf of the Guava team, who are trying to eliminate
Throwables.propagate, especially for trivial throws like this one.
  • Loading branch information
juangj committed Apr 7, 2016
1 parent ae3e1bf commit 67d3dc4
Showing 1 changed file with 4 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,10 @@

package org.openqa.selenium.testing.drivers;

import com.google.common.base.Throwables;
import static java.util.concurrent.TimeUnit.SECONDS;

import com.google.common.io.Files;

import org.openqa.selenium.Build;
import org.openqa.selenium.Capabilities;
import org.openqa.selenium.net.NetworkUtils;
Expand All @@ -38,8 +39,6 @@
import java.util.List;
import java.util.logging.Logger;

import static java.util.concurrent.TimeUnit.SECONDS;

public class OutOfProcessSeleniumServer {

private static final Logger log = Logger.getLogger(OutOfProcessSeleniumServer.class.getName());
Expand Down Expand Up @@ -97,9 +96,9 @@ public OutOfProcessSeleniumServer start() {
log.info("Server is ready");
} catch (UrlChecker.TimeoutException e) {
log.severe("Server failed to start: " + e.getMessage());
throw Throwables.propagate(e);
throw new RuntimeException(e);
} catch (MalformedURLException e) {
throw Throwables.propagate(e);
throw new RuntimeException(e);
}

return this;
Expand Down

0 comments on commit 67d3dc4

Please sign in to comment.