Skip to content

Commit a67dcd6

Browse files
committed
Remove unnecessary implements statements. No logical changes.
1 parent 4555973 commit a67dcd6

File tree

6 files changed

+13
-25
lines changed

6 files changed

+13
-25
lines changed

java/client/src/org/openqa/selenium/chrome/ChromeDriver.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020

2121
import org.openqa.selenium.Capabilities;
2222
import org.openqa.selenium.OutputType;
23-
import org.openqa.selenium.TakesScreenshot;
2423
import org.openqa.selenium.WebDriver;
2524
import org.openqa.selenium.WebDriverException;
2625
import org.openqa.selenium.remote.DriverCommand;
@@ -95,7 +94,7 @@
9594
*
9695
* @see ChromeDriverService#createDefaultService
9796
*/
98-
public class ChromeDriver extends RemoteWebDriver implements TakesScreenshot {
97+
public class ChromeDriver extends RemoteWebDriver {
9998

10099
/**
101100
* Creates a new ChromeDriver using the {@link ChromeDriverService#createDefaultService default}

java/client/src/org/openqa/selenium/firefox/FirefoxDriver.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,10 @@
2929
import com.google.common.collect.Maps;
3030
import com.google.common.collect.Sets;
3131

32-
import org.openqa.selenium.Beta;
3332
import org.openqa.selenium.Capabilities;
3433
import org.openqa.selenium.OutputType;
3534
import org.openqa.selenium.Platform;
3635
import org.openqa.selenium.Proxy;
37-
import org.openqa.selenium.TakesScreenshot;
3836
import org.openqa.selenium.WebDriverException;
3937
import org.openqa.selenium.browserlaunchers.Proxies;
4038
import org.openqa.selenium.firefox.internal.MarionetteConnection;
@@ -58,7 +56,6 @@
5856

5957
import java.io.File;
6058
import java.io.IOException;
61-
import java.util.Map;
6259
import java.util.Set;
6360
import java.util.concurrent.TimeUnit;
6461

@@ -74,7 +71,7 @@
7471
* When the driver starts, it will make a copy of the profile it is using, rather than using that
7572
* profile directly. This allows multiple instances of firefox to be started.
7673
*/
77-
public class FirefoxDriver extends RemoteWebDriver implements TakesScreenshot, Killable {
74+
public class FirefoxDriver extends RemoteWebDriver implements Killable {
7875
public static final String BINARY = "firefox_binary";
7976
public static final String PROFILE = "firefox_profile";
8077

java/client/src/org/openqa/selenium/ie/InternetExplorerDriver.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,13 @@
1717

1818
package org.openqa.selenium.ie;
1919

20+
import static org.openqa.selenium.remote.CapabilityType.PROXY;
21+
2022
import com.google.common.base.Throwables;
2123

2224
import org.openqa.selenium.Capabilities;
2325
import org.openqa.selenium.OutputType;
2426
import org.openqa.selenium.Platform;
25-
import org.openqa.selenium.TakesScreenshot;
2627
import org.openqa.selenium.WebDriverException;
2728
import org.openqa.selenium.browserlaunchers.WindowsProxyManager;
2829
import org.openqa.selenium.remote.CapabilityType;
@@ -34,9 +35,7 @@
3435

3536
import java.io.File;
3637

37-
import static org.openqa.selenium.remote.CapabilityType.PROXY;
38-
39-
public class InternetExplorerDriver extends RemoteWebDriver implements TakesScreenshot {
38+
public class InternetExplorerDriver extends RemoteWebDriver {
4039

4140
/**
4241
* Capability that defines whether to ignore the browser zoom level or not.

java/client/src/org/openqa/selenium/safari/SafariDriver.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,19 +17,14 @@
1717

1818
package org.openqa.selenium.safari;
1919

20-
import com.google.common.base.Optional;
21-
22-
import org.json.JSONException;
2320
import org.openqa.selenium.Capabilities;
2421
import org.openqa.selenium.OutputType;
25-
import org.openqa.selenium.TakesScreenshot;
2622
import org.openqa.selenium.WebDriverException;
2723
import org.openqa.selenium.remote.DesiredCapabilities;
2824
import org.openqa.selenium.remote.DriverCommand;
2925
import org.openqa.selenium.remote.FileDetector;
3026
import org.openqa.selenium.remote.RemoteWebDriver;
3127

32-
import java.io.File;
3328
import java.io.IOException;
3429

3530
/**
@@ -38,7 +33,7 @@
3833
*
3934
* This driver can be configured using the {@link SafariOptions} class.
4035
*/
41-
public class SafariDriver extends RemoteWebDriver implements TakesScreenshot {
36+
public class SafariDriver extends RemoteWebDriver {
4237

4338
/**
4439
* Initializes a new SafariDriver} class with default {@link SafariOptions}.

java/client/test/org/openqa/selenium/testing/drivers/SauceDriver.java

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,23 +17,22 @@
1717

1818
package org.openqa.selenium.testing.drivers;
1919

20-
import java.net.MalformedURLException;
21-
import java.net.URL;
20+
import com.google.common.base.Preconditions;
21+
import com.google.common.base.Strings;
22+
import com.google.common.base.Throwables;
2223

2324
import org.openqa.selenium.Capabilities;
2425
import org.openqa.selenium.OutputType;
2526
import org.openqa.selenium.Platform;
26-
import org.openqa.selenium.TakesScreenshot;
2727
import org.openqa.selenium.ie.InternetExplorerDriver;
2828
import org.openqa.selenium.remote.DesiredCapabilities;
2929
import org.openqa.selenium.remote.DriverCommand;
3030
import org.openqa.selenium.remote.RemoteWebDriver;
3131

32-
import com.google.common.base.Preconditions;
33-
import com.google.common.base.Strings;
34-
import com.google.common.base.Throwables;
32+
import java.net.MalformedURLException;
33+
import java.net.URL;
3534

36-
public class SauceDriver extends RemoteWebDriver implements TakesScreenshot {
35+
public class SauceDriver extends RemoteWebDriver {
3736

3837
private static final String SAUCE_JOB_NAME_ENV_NAME = "SAUCE_JOB_NAME";
3938
private static final String SELENIUM_VERSION_ENV_NAME = "SAUCE_SELENIUM_VERSION";

java/client/test/org/openqa/selenium/testing/drivers/TestChromeDriver.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121

2222
import org.openqa.selenium.Capabilities;
2323
import org.openqa.selenium.OutputType;
24-
import org.openqa.selenium.TakesScreenshot;
2524
import org.openqa.selenium.chrome.ChromeDriverService;
2625
import org.openqa.selenium.chrome.ChromeOptions;
2726
import org.openqa.selenium.remote.DesiredCapabilities;
@@ -37,7 +36,7 @@
3736
* entire test suite. We do not use {@link org.openqa.selenium.chrome.ChromeDriver} since that starts and stops the service
3837
* with each instance (and that is too expensive for our purposes).
3938
*/
40-
public class TestChromeDriver extends RemoteWebDriver implements TakesScreenshot {
39+
public class TestChromeDriver extends RemoteWebDriver {
4140
private static ChromeDriverService service;
4241

4342
public TestChromeDriver() {

0 commit comments

Comments
 (0)