rb: Share method to communicate to running service

Author: p0deje

rb/lib/selenium/webdriver/chrome/service.rb

@@ -54,12 +54,7 @@ def start_process
         end
 
         def stop_server
-          Net::HTTP.start(@host, @port) do |http|
-            http.open_timeout = STOP_TIMEOUT / 2
-            http.read_timeout = STOP_TIMEOUT / 2
-
-            http.get("/shutdown")
-          end
+          connect_to_server { |http| http.get("/shutdown") }
         end
 
         def connect_until_stable

rb/lib/selenium/webdriver/common/service.rb

@@ -71,6 +71,15 @@ def uri
 
       private
 
+      def connect_to_server
+        Net::HTTP.start(@host, @port) do |http|
+          http.open_timeout = STOP_TIMEOUT / 2
+          http.read_timeout = STOP_TIMEOUT / 2
+
+          yield http
+        end
+      end
+
       def find_free_port
         @port = PortProber.above(@port)
       end

rb/lib/selenium/webdriver/edge/service.rb

@@ -46,12 +46,7 @@ def self.default_service(*extra_args)
         private
 
         def stop_server
-          Net::HTTP.start(@host, @port) do |http|
-            http.open_timeout = STOP_TIMEOUT / 2
-            http.read_timeout = STOP_TIMEOUT / 2
-
-            http.head("/shutdown")
-          end
+          connect_to_server { |http| http.head("/shutdown") }
         end
 
         def start_process

rb/lib/selenium/webdriver/firefox/service.rb

@@ -68,12 +68,7 @@ def stop_process
         end
 
         def stop_server
-          Net::HTTP.start(@host, @port) do |http|
-            http.open_timeout = STOP_TIMEOUT / 2
-            http.read_timeout = STOP_TIMEOUT / 2
-
-            http.head("/shutdown")
-          end
+          connect_to_server { |http| http.head("/shutdown") }
         end
 
         def connect_until_stable

rb/lib/selenium/webdriver/phantomjs/service.rb

@@ -67,12 +67,7 @@ def stop_process
         end
 
         def stop_server
-          Net::HTTP.start(@host, @port) do |http|
-            http.open_timeout = STOP_TIMEOUT / 2
-            http.read_timeout = STOP_TIMEOUT / 2
-
-            http.get("/shutdown")
-          end
+          connect_to_server { |http| http.get("/shutdown") }
         end
 
         def connect_until_stable