Deleting deprecated code

SeleniumHQ · May 9, 2018 · 5a8a2c1 · 5a8a2c1
1 parent f7eeab8
commit 5a8a2c1
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 24 deletions.
diff --git a/java/client/src/org/openqa/selenium/remote/http/HttpClient.java b/java/client/src/org/openqa/selenium/remote/http/HttpClient.java
@@ -46,15 +46,6 @@ public interface HttpClient {
    */
   HttpResponse execute(HttpRequest request) throws IOException;
 
-  /**
-	 * Closes the connections associated with this client.
-	 *
-	 * @throws  IOException  if an I/O error occurs.
-   * @deprecated This responsibility moved to Factory
-	 */
-  @Deprecated
-  void close() throws IOException;
-
   interface Factory {
 
     static Factory createDefault() {

diff --git a/java/client/src/org/openqa/selenium/remote/internal/ApacheHttpClient.java b/java/client/src/org/openqa/selenium/remote/internal/ApacheHttpClient.java
@@ -290,9 +290,4 @@ private static synchronized HttpClientFactory getDefaultHttpClientFactory() {
       return defaultClientFactory;
     }
   }
-
-  @Deprecated
-  @Override
-  public void close() throws IOException {
-  }
 }
diff --git a/java/client/src/org/openqa/selenium/remote/internal/OkHttpClient.java b/java/client/src/org/openqa/selenium/remote/internal/OkHttpClient.java
@@ -113,11 +113,6 @@ public HttpResponse execute(HttpRequest request) throws IOException {
     return toReturn;
   }
 
-  @Override
-  public void close() {
-    // No-op
-  }
-
   public static class Factory implements HttpClient.Factory {
 
     private final ConnectionPool pool = new ConnectionPool();

diff --git a/java/client/test/org/openqa/selenium/remote/ProtocolHandshakeTest.java b/java/client/test/org/openqa/selenium/remote/ProtocolHandshakeTest.java
@@ -384,11 +384,6 @@ public HttpResponse execute(HttpRequest request) {
       return response;
     }
 
-    @Override
-    public void close() {
-      // Does nothing
-    }
-
     String getRequestPayload() {
       return payload;
     }