Update call sites to httpclient.execute

odedniv · Jan 18, 2018 · f907cd5 · f907cd5
1 parent fe1307e
commit f907cd5
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/java/client/src/org/openqa/selenium/remote/HttpCommandExecutor.java b/java/client/src/org/openqa/selenium/remote/HttpCommandExecutor.java
@@ -161,7 +161,7 @@ public Response execute(Command command) throws IOException {
     HttpRequest httpRequest = commandCodec.encode(command);
     try {
       log(LogType.PROFILER, new HttpProfilerLogEntry(command.getName(), true));
-      HttpResponse httpResponse = client.execute(httpRequest, true);
+      HttpResponse httpResponse = client.execute(httpRequest);
       log(LogType.PROFILER, new HttpProfilerLogEntry(command.getName(), false));
 
       Response response = responseCodec.decode(httpResponse);

diff --git a/java/client/src/org/openqa/selenium/remote/ProtocolHandshake.java b/java/client/src/org/openqa/selenium/remote/ProtocolHandshake.java
@@ -98,7 +98,7 @@ private Optional<Result> createSession(HttpClient client, InputStream newSession
     request.setHeader(CONTENT_TYPE, JSON_UTF_8.toString());
     request.setContent(newSessionBlob);
     long start = System.currentTimeMillis();
-    HttpResponse response = client.execute(request, true);
+    HttpResponse response = client.execute(request);
     long time = System.currentTimeMillis() - start;
 
     // Ignore the content type. It may not have been set. Strictly speaking we're not following the

diff --git a/java/client/test/org/openqa/selenium/environment/webserver/JettyAppServer.java b/java/client/test/org/openqa/selenium/environment/webserver/JettyAppServer.java
@@ -210,7 +210,7 @@ public String create(Page page) {
       HttpRequest request = new HttpRequest(HttpMethod.POST, "/common/createPage");
       request.setHeader(CONTENT_TYPE, JSON_UTF_8.toString());
       request.setContent(data);
-      HttpResponse response = client.execute(request, true);
+      HttpResponse response = client.execute(request);
       return response.getContentString();
     } catch (IOException ex) {
       throw new RuntimeException(ex);

diff --git a/java/client/test/org/openqa/selenium/remote/internal/HttpClientTestBase.java b/java/client/test/org/openqa/selenium/remote/internal/HttpClientTestBase.java
@@ -99,7 +99,7 @@ protected void doGet(HttpServletRequest req, HttpServletResponse resp)
     try {
       HttpClient client = createFactory().createClient(server.getURI().toURL());
       HttpRequest request = new HttpRequest(HttpMethod.GET, "/foo");
-      return client.execute(request, true);
+      return client.execute(request);
     } finally {
       server.stop();
     }

diff --git a/java/client/test/org/openqa/selenium/testing/drivers/GridSupplier.java b/java/client/test/org/openqa/selenium/testing/drivers/GridSupplier.java
@@ -91,7 +91,7 @@ private synchronized void startServers() {
       HttpRequest req = new HttpRequest(HttpMethod.GET, "/status");
       HttpResponse response = null;
       try {
-        response = c.execute(req, true);
+        response = c.execute(req);
       } catch (IOException e) {
         throw new RuntimeException(e);
       }

diff --git a/java/server/src/org/openqa/selenium/remote/server/ProtocolConverter.java b/java/server/src/org/openqa/selenium/remote/server/ProtocolConverter.java
@@ -93,7 +93,7 @@ public void handle(HttpRequest req, HttpResponse resp) throws IOException {
 
   @VisibleForTesting
   HttpResponse makeRequest(HttpRequest request) throws IOException {
-    return client.execute(request, true);
+    return client.execute(request);
   }
 
   private void copyToServletResponse(HttpResponse response, HttpResponse resp)