selenium-webdriver 3.2.1 → 3.2.2

data/CHANGES

@@ -1,3 +1,13 @@
+3.2.2 (2017-03-01)
+===================
+
+Ruby:
+  * Fix bug for supporting Logger output on Ruby versions < 2.3
+  * Add more logging and adjust output levels
+
+Remote:
+  * Support for creating Remote session with browser name and url parameters
+
 3.2.1 (2017-02-24)
 ===================
 

data/lib/selenium/server.rb

@@ -234,7 +234,10 @@ module Selenium
       @process ||= (
         # extract any additional_args that start with -D as options
         properties = @additional_args.dup - @additional_args.delete_if { |arg| arg[/^-D/] }
-        cp = ChildProcess.build('java', *properties, '-jar', @jar, '-port', @port.to_s, *@additional_args)
+        server_command = ['java'] + properties + ['-jar', @jar, '-port', @port.to_s] + @additional_args
+        cp = ChildProcess.build(*server_command)
+        WebDriver.logger.debug("Executing Process #{server_command}")
+
         io = cp.io
 
         if @log.is_a?(String)

data/lib/selenium/webdriver/chrome.rb

@@ -27,23 +27,11 @@ module Selenium
   module WebDriver
     module Chrome
       def self.driver_path=(path)
-        warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-          [DEPRECATION] `driver_path=` is deprecated. Pass the driver path as an option instead.
-          e.g. Selenium::WebDriver.for :chrome, driver_path: '/path'
-        DEPRECATE
-
         Platform.assert_executable path
         @driver_path = path
       end
 
-      def self.driver_path(warning = true)
-        if warning
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-            [DEPRECATION] `driver_path` is deprecated. Pass the driver path as an option instead.
-            e.g. Selenium::WebDriver.for :chrome, driver_path: '/path'
-          DEPRECATE
-        end
-
+      def self.driver_path
         @driver_path ||= nil
       end
 

data/lib/selenium/webdriver/chrome/bridge.rb

@@ -23,22 +23,22 @@ module Selenium
       # @api private
       class Bridge < Remote::Bridge
         def initialize(opts = {})
-          port = opts.delete(:port) || Service::DEFAULT_PORT
-          service_args = opts.delete(:service_args) || {}
-
-          if opts[:service_log_path]
-            service_args.merge!(service_log_path: opts.delete(:service_log_path))
-          end
+          opts[:desired_capabilities] = create_capabilities(opts)
 
           unless opts.key?(:url)
-            driver_path = opts.delete(:driver_path) || Chrome.driver_path(false)
+            port = opts.delete(:port) || Service::DEFAULT_PORT
+            service_args = opts.delete(:service_args) || {}
+
+            if opts[:service_log_path]
+              service_args.merge!(service_log_path: opts.delete(:service_log_path))
+            end
+
+            driver_path = opts.delete(:driver_path) || Chrome.driver_path
             @service = Service.new(driver_path, port, *extract_service_args(service_args))
             @service.start
             opts[:url] = @service.uri
           end
 
-          opts[:desired_capabilities] = create_capabilities(opts)
-
           super(opts)
         end
 

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

@@ -38,6 +38,8 @@ module Selenium
         def start_process
           server_command = [@executable_path, "--port=#{@port}", *@extra_args]
           @process = ChildProcess.build(*server_command)
+          WebDriver.logger.debug("Executing Process #{server_command}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.leader = true unless Platform.windows?
           @process.start

data/lib/selenium/webdriver/common/logger.rb

@@ -45,16 +45,16 @@ module Selenium
                      :level
 
       def initialize
-        @logger = ::Logger.new($stdout)
-        @logger.progname = 'Selenium'
-        @logger.level = ($DEBUG ? DEBUG : WARN)
-        @logger.formatter = proc do |severity, time, progname, msg|
-          "#{time.strftime('%F %T')} #{severity} #{progname} #{msg}\n"
-        end
+        @logger = create_logger($stdout)
       end
 
       def output=(io)
-        @logger.reopen(io)
+        # `Logger#reopen` was added in Ruby 2.3
+        if @logger.respond_to?(:reopen)
+          @logger.reopen(io)
+        else
+          @logger = create_logger(io)
+        end
       end
 
       #
@@ -66,22 +66,21 @@ module Selenium
         if severity.is_a?(Integer)
           @logger.level = severity
         else
-          _severity = severity.to_s.downcase
-          case _severity
-            when 'debug'.freeze
-              @logger.level = DEBUG
-            when 'info'.freeze
-              @logger.level = INFO
-            when 'warn'.freeze
-              @logger.level = WARN
-            when 'error'.freeze
-              @logger.level = ERROR
-            when 'fatal'.freeze
-              @logger.level = FATAL
-            when 'unknown'.freeze
-              @logger.level = UNKNOWN
-            else
-              raise ArgumentError, "invalid log level: #{severity}"
+          case severity.to_s.downcase
+          when 'debug'.freeze
+            @logger.level = DEBUG
+          when 'info'.freeze
+            @logger.level = INFO
+          when 'warn'.freeze
+            @logger.level = WARN
+          when 'error'.freeze
+            @logger.level = ERROR
+          when 'fatal'.freeze
+            @logger.level = FATAL
+          when 'unknown'.freeze
+            @logger.level = UNKNOWN
+          else
+            raise ArgumentError, "invalid log level: #{severity}"
           end
         end
       end
@@ -104,6 +103,19 @@ module Selenium
           File.new(Platform.null_device, 'w')
         end
       end
+
+      private
+
+      def create_logger(output)
+        logger = ::Logger.new(output)
+        logger.progname = 'Selenium'
+        logger.level = ($DEBUG ? DEBUG : WARN)
+        logger.formatter = proc do |severity, time, progname, msg|
+          "#{time.strftime('%F %T')} #{severity} #{progname} #{msg}\n"
+        end
+
+        logger
+      end
     end # Logger
   end # WebDriver
 end # Selenium

data/lib/selenium/webdriver/common/port_prober.rb

@@ -47,7 +47,7 @@ module Selenium
           begin
             TCPServer.new(host, port).close
           rescue *IGNORED_ERRORS => ex
-            WebDriver.logger.info("port prober could not bind to #{host}:#{port} (#{ex.message})")
+            WebDriver.logger.debug("port prober could not bind to #{host}:#{port} (#{ex.message})")
             # ignored - some machines appear unable to bind to some of their interfaces
           end
         end

data/lib/selenium/webdriver/common/socket_lock.rb

@@ -65,7 +65,7 @@ module Selenium
 
         true
       rescue SocketError, Errno::EADDRINUSE, Errno::EBADF => ex
-        WebDriver.logger.info("#{self}: #{ex.message}")
+        WebDriver.logger.debug("#{self}: #{ex.message}")
         false
       end
 

data/lib/selenium/webdriver/common/socket_poller.rb

@@ -90,7 +90,7 @@ module Selenium
           true
         rescue *NOT_CONNECTED_ERRORS
           sock.close if sock
-          WebDriver.logger.info("polling for socket on #{[@host, @port].inspect}")
+          WebDriver.logger.debug("polling for socket on #{[@host, @port].inspect}")
           false
         end
       end

data/lib/selenium/webdriver/edge.rb

@@ -26,23 +26,11 @@ module Selenium
   module WebDriver
     module Edge
       def self.driver_path=(path)
-        warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-          [DEPRECATION] `driver_path=` is deprecated. Pass the driver path as an option instead.
-          e.g. Selenium::WebDriver.for :edge, driver_path: '/path'
-        DEPRECATE
-
         Platform.assert_executable path
         @driver_path = path
       end
 
       def self.driver_path(warning = true)
-        if warning
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-            [DEPRECATION] `driver_path` is deprecated. Pass the driver path as an option instead.
-            e.g. Selenium::WebDriver.for :edge, driver_path: '/path'
-          DEPRECATE
-        end
-
         @driver_path ||= nil
       end
     end # Edge

data/lib/selenium/webdriver/edge/bridge.rb

@@ -30,7 +30,7 @@ module Selenium
           service_args = opts.delete(:service_args) || {}
 
           unless opts.key?(:url)
-            driver_path = opts.delete(:driver_path) || Edge.driver_path(false)
+            driver_path = opts.delete(:driver_path) || Edge.driver_path
             @service = Service.new(driver_path, port, *extract_service_args(service_args))
             @service.host = 'localhost' if @service.host == '127.0.0.1'
             @service.start

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

@@ -38,6 +38,8 @@ module Selenium
         def start_process
           server_command = [@executable_path, "--port=#{@port}", *@extra_args]
           @process = ChildProcess.build(*server_command)
+          WebDriver.logger.debug("Executing Process #{server_command}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.start
         end

data/lib/selenium/webdriver/firefox.rb

@@ -42,23 +42,11 @@ module Selenium
       DEFAULT_LOAD_NO_FOCUS_LIB = false
 
       def self.driver_path=(path)
-        warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-          [DEPRECATION] `driver_path=` is deprecated. Pass the driver path as an option instead.
-          e.g. Selenium::WebDriver.for :firefox, driver_path: '/path'
-        DEPRECATE
-
         Platform.assert_executable path
         @driver_path = path
       end
 
-      def self.driver_path(warning = true)
-        if warning
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-            [DEPRECATION] `driver_path` is deprecated. Pass the driver path as an option instead.
-            e.g. Selenium::WebDriver.for :firefox, driver_path: '/path'
-          DEPRECATE
-        end
-
+      def self.driver_path
         @driver_path ||= nil
       end
 

data/lib/selenium/webdriver/firefox/binary.rb

@@ -77,6 +77,8 @@ module Selenium
         def execute(*extra_args)
           args = [self.class.path, '-no-remote'] + extra_args
           @process = ChildProcess.build(*args)
+          WebDriver.logger.debug("Executing Process #{args}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.start
         end

data/lib/selenium/webdriver/firefox/bridge.rb

@@ -23,21 +23,23 @@ module Selenium
       # @api private
       class Bridge < Remote::Bridge
         def initialize(opts = {})
-          port = opts.delete(:port) || DEFAULT_PORT
-          profile = opts.delete(:profile)
+          opts[:desired_capabilities] ||= Remote::Capabilities.firefox
+          opts[:desired_capabilities].proxy = opts.delete(:proxy) if opts.key?(:proxy)
 
-          @launcher = create_launcher(port, profile)
-          @launcher.launch
-          opts[:url] = @launcher.url
+          unless opts.key?(:url)
+            port = opts.delete(:port) || DEFAULT_PORT
+            profile = opts.delete(:profile)
 
-          caps = opts[:desired_capabilities] ||= Remote::Capabilities.firefox
-          caps.proxy = opts.delete(:proxy) if opts.key?(:proxy)
-          Binary.path = caps[:firefox_binary] if caps[:firefox_binary]
+            Binary.path = opts[:desired_capabilities][:firefox_binary] if opts[:desired_capabilities][:firefox_binary]
+            @launcher = create_launcher(port, profile)
+            @launcher.launch
+            opts[:url] = @launcher.url
+          end
 
           begin
             super(opts)
           rescue
-            @launcher.quit
+            @launcher.quit if @launcher
             raise
           end
         end
@@ -47,9 +49,7 @@ module Selenium
         end
 
         def driver_extensions
-          [
-            DriverExtensions::TakesScreenshot
-          ]
+          [DriverExtensions::TakesScreenshot]
         end
 
         def quit

data/lib/selenium/webdriver/firefox/profile.rb

@@ -214,7 +214,7 @@ module Selenium
           destination = File.join(directory, 'extensions')
 
           @extensions.each do |name, extension|
-            WebDriver.logger.info({extenstion: name}.inspect)
+            WebDriver.logger.debug({extenstion: name}.inspect)
             extension.write_to(destination)
           end
         end

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

@@ -42,6 +42,8 @@ module Selenium
         def start_process
           server_command = [@executable_path, "--binary=#{Firefox::Binary.path}", "--port=#{@port}", *@extra_args]
           @process = ChildProcess.build(*server_command)
+          WebDriver.logger.debug("Executing Process #{server_command}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.start
         end

data/lib/selenium/webdriver/firefox/w3c_bridge.rb

@@ -23,14 +23,17 @@ module Selenium
       # @api private
       class W3CBridge < Remote::W3CBridge
         def initialize(opts = {})
-          port = opts.delete(:port) || Service::DEFAULT_PORT
           opts[:desired_capabilities] = create_capabilities(opts)
-          service_args = opts.delete(:service_args) || {}
 
-          driver_path = opts.delete(:driver_path) || Firefox.driver_path(false)
-          @service = Service.new(driver_path, port, *extract_service_args(service_args))
-          @service.start
-          opts[:url] = @service.uri
+          unless opts.key?(:url)
+            port = opts.delete(:port) || Service::DEFAULT_PORT
+            service_args = opts.delete(:service_args) || {}
+
+            driver_path = opts.delete(:driver_path) || Firefox.driver_path
+            @service = Service.new(driver_path, port, *extract_service_args(service_args))
+            @service.start
+            opts[:url] = @service.uri
+          end
 
           super(opts)
         end
@@ -40,10 +43,8 @@ module Selenium
         end
 
         def driver_extensions
-          [
-            DriverExtensions::TakesScreenshot,
-            DriverExtensions::HasWebStorage
-          ]
+          [DriverExtensions::TakesScreenshot,
+            DriverExtensions::HasWebStorage]
         end
 
         def quit

data/lib/selenium/webdriver/ie.rb

@@ -24,23 +24,11 @@ module Selenium
   module WebDriver
     module IE
       def self.driver_path=(path)
-        warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-          [DEPRECATION] `driver_path=` is deprecated. Pass the driver path as an option instead.
-          e.g. Selenium::WebDriver.for :ie, driver_path: '/path'
-        DEPRECATE
-
         Platform.assert_executable path
         @driver_path = path
       end
 
-      def self.driver_path(warning = true)
-        if warning
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-            [DEPRECATION] `driver_path` is deprecated. Pass the driver path as an option instead.
-            e.g. Selenium::WebDriver.for :ie, driver_path: '/path'
-          DEPRECATE
-        end
-
+      def self.driver_path
         @driver_path ||= nil
       end
     end # IE

data/lib/selenium/webdriver/ie/bridge.rb

@@ -26,18 +26,22 @@ module Selenium
 
       class Bridge < Remote::Bridge
         def initialize(opts = {})
-          port = opts.delete(:port) || Service::DEFAULT_PORT
-          service_args = opts.delete(:service_args) || {}
-          service_args = match_legacy(opts, service_args)
-          driver_path = opts.delete(:driver_path) || IE.driver_path(false)
+          opts[:desired_capabilities] ||= Remote::Capabilities.internet_explorer
 
-          @service = Service.new(driver_path, port, *extract_service_args(service_args))
-          @service.start
-          opts[:url] = @service.uri
+          unless opts.key?(:url)
+            port = opts.delete(:port) || Service::DEFAULT_PORT
+            service_args = opts.delete(:service_args) || {}
+            service_args = match_legacy(opts, service_args)
+            driver_path = opts.delete(:driver_path) || IE.driver_path
+            @service = Service.new(driver_path, port, *extract_service_args(service_args))
+            @service.start
+            opts[:url] = @service.uri
+          end
 
-          caps = opts[:desired_capabilities] ||= Remote::Capabilities.internet_explorer
-          caps[:ignore_protected_mode_settings] = true if opts.delete(:introduce_flakiness_by_ignoring_security_domains)
-          caps[:native_events] = opts.delete(:native_events) != false
+          if opts.delete(:introduce_flakiness_by_ignoring_security_domains)
+            opts[:desired_capabilities][:ignore_protected_mode_settings] = true
+          end
+          opts[:desired_capabilities][:native_events] = opts.delete(:native_events) != false
 
           super(opts)
         end
@@ -70,6 +74,9 @@ module Selenium
           service_args << "--log-level=#{args.delete(:log_level).to_s.upcase}" if args.key?(:log_level)
           service_args << "--log-file=#{args.delete(:log_file)}" if args.key?(:log_file)
           service_args << "--implementation=#{args.delete(:implementation).to_s.upcase}" if args.key?(:implementation)
+          service_args << "--host=#{args.delete(:host)}" if args.key?(:host)
+          service_args << "--extract_path=#{args.delete(:extract_path)}" if args.key?(:extract_path)
+          service_args << "--silent" if args[:silent] == true
           service_args
         end
       end # Bridge

data/lib/selenium/webdriver/ie/service.rb

@@ -38,6 +38,8 @@ module Selenium
         def start_process
           server_command = [@executable_path, "--port=#{@port}", *@extra_args]
           @process = ChildProcess.new(*server_command)
+          WebDriver.logger.debug("Executing Process #{server_command}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.start
         end

data/lib/selenium/webdriver/phantomjs.rb

@@ -25,25 +25,17 @@ require 'selenium/webdriver/phantomjs/bridge'
 module Selenium
   module WebDriver
     module PhantomJS
-      def self.path=(path)
-        warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-          [DEPRECATION] `path=` is deprecated. Pass the driver path as an option instead.
-          e.g. Selenium::WebDriver.for :phantomjs, driver_path: '/path'
-        DEPRECATE
-
-        Platform.assert_executable path
-        @path = path
-      end
-
-      def self.path(warning = true)
-        if warning
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-            [DEPRECATION] `path` is deprecated. Pass the driver path as an option instead.
-            e.g. Selenium::WebDriver.for :phantomjs, driver_path: '/path'
-          DEPRECATE
+      class << self
+        def path=(path)
+          Platform.assert_executable path
+          @path = path
         end
+        alias_method :driver_path=, :path=
 
-        @path ||= nil
+        def path
+          @path ||= nil
+        end
+        alias_method :driver_path, :path
       end
     end # PhantomJS
   end # WebDriver

data/lib/selenium/webdriver/phantomjs/bridge.rb

@@ -26,11 +26,11 @@ module Selenium
 
       class Bridge < Remote::Bridge
         def initialize(opts = {})
-          port = opts.delete(:port) || Service::DEFAULT_PORT
           opts[:desired_capabilities] ||= Remote::Capabilities.phantomjs
 
           unless opts.key?(:url)
-            driver_path = opts.delete(:driver_path) || PhantomJS.path(false)
+            port = opts.delete(:port) || Service::DEFAULT_PORT
+            driver_path = opts.delete(:driver_path) || PhantomJS.driver_path
             args = opts.delete(:args) || opts[:desired_capabilities]['phantomjs.cli.args']
             @service = Service.new(driver_path, port, *args)
             @service.start

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

@@ -34,6 +34,8 @@ module Selenium
         def start_process
           server_command = [@executable_path, "--webdriver=#{@port}", *@extra_args]
           @process = ChildProcess.build(*server_command.compact)
+          WebDriver.logger.debug("Executing Process #{server_command}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.start
         end

data/lib/selenium/webdriver/remote/bridge.rb

@@ -39,7 +39,13 @@ module Selenium
         def initialize(opts = {})
           opts = opts.dup
 
+          if opts.key?(:port)
+            WebDriver.logger.warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
+            [DEPRECATION] `:port` is deprecated. Use `:url` with full path
+            DEPRECATE
+          end
           port = opts.delete(:port) || 4444
+
           http_client = opts.delete(:http_client) { Http::Default.new }
           desired_capabilities = opts.delete(:desired_capabilities) { Capabilities.firefox }
           url = opts.delete(:url) { "http://#{Platform.localhost}:#{port}/wd/hub" }
@@ -379,7 +385,7 @@ module Selenium
         end
 
         def mouse
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
+          WebDriver.logger.warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
             [DEPRECATION] `Driver#mouse` is deprecated with w3c implementation. Instead use 
             driver.action.<command>.perform
           DEPRECATE
@@ -387,7 +393,7 @@ module Selenium
         end
 
         def keyboard
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
+          WebDriver.logger.warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
             [DEPRECATION] `Driver#keyboard` is deprecated with w3c implementation. Instead use 
             driver.action.<command>.perform
           DEPRECATE

data/lib/selenium/webdriver/remote/http/common.rb

@@ -52,7 +52,7 @@ module Selenium
               headers['Content-Length'] = payload.bytesize.to_s if [:post, :put].include?(verb)
 
               WebDriver.logger.info("   >>> #{url} | #{payload}")
-              WebDriver.logger.info("     > #{headers.inspect}")
+              WebDriver.logger.debug("     > #{headers.inspect}")
             elsif verb == :post
               payload = '{}'
               headers['Content-Length'] = '2'

data/lib/selenium/webdriver/remote/http/default.rb

@@ -45,7 +45,7 @@ module Selenium
           # @param [Numeric] value - Timeout in seconds to apply to both open timeout and read timeouts.
           # @deprecated Please set the specific desired timeout {#read_timeout} or {#open_timeout} directly.
           def timeout=(value)
-            Kernel.warn 'Selenium::WebDriver::Remote::Http::Default#timeout= is deprecated. Use #read_timeout= or #open_timeout= instead'
+            WebDriver.logger.warn 'Selenium::WebDriver::Remote::Http::Default#timeout= is deprecated. Use #read_timeout= or #open_timeout= instead'
             self.open_timeout = value
             self.read_timeout = value
           end

data/lib/selenium/webdriver/remote/http/persistent.rb

@@ -45,7 +45,7 @@ module Selenium
             if Net::HTTP::Persistent::VERSION >= '3'
               Net::HTTP::Persistent.new name: 'webdriver', proxy: proxy
             else
-              warn 'Support for this version of net-http-persistent is deprecated. Please upgrade.'
+              WebDriver.logger.warn 'Support for this version of net-http-persistent is deprecated. Please upgrade.'
               Net::HTTP::Persistent.new 'webdriver', proxy
             end
           end

data/lib/selenium/webdriver/remote/w3c_bridge.rb

@@ -48,7 +48,13 @@ module Selenium
 
           opts = opts.dup
 
+          if opts.key?(:port)
+            WebDriver.logger.warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
+            [DEPRECATION] `:port` is deprecated. Use full url desired.
+            DEPRECATE
+          end
           port = opts.delete(:port) || 4444
+
           http_client = opts.delete(:http_client) { Http::Default.new }
           desired_capabilities = opts.delete(:desired_capabilities) { W3CCapabilities.firefox }
           url = opts.delete(:url) { "http://#{Platform.localhost}:#{port}/wd/hub" }

data/lib/selenium/webdriver/safari.rb

@@ -24,6 +24,14 @@ module Selenium
   module WebDriver
     module Safari
       class << self
+        def technology_preview
+          "/Applications/Safari\ Technology\ Preview.app/Contents/MacOS/safaridriver"
+        end
+
+        def technology_preview!
+          self.driver_path = technology_preview
+        end
+
         def path=(path)
           Platform.assert_executable(path)
           @path = path
@@ -37,23 +45,11 @@ module Selenium
         end
 
         def driver_path=(path)
-          warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-            [DEPRECATION] `driver_path=` is deprecated. Pass the driver path as an option instead.
-            e.g. Selenium::WebDriver.for :safari, driver_path: '/path'
-          DEPRECATE
-
           Platform.assert_executable path
           @driver_path = path
         end
 
-        def driver_path(warning = true)
-          if warning
-            warn <<-DEPRECATE.gsub(/\n +| {2,}/, ' ').freeze
-              [DEPRECATION] `driver_path` is deprecated. Pass the driver path as an option instead.
-              e.g. Selenium::WebDriver.for :safari, driver_path: '/path'
-            DEPRECATE
-          end
-
+        def driver_path
           @driver_path ||= nil
         end
       end

data/lib/selenium/webdriver/safari/bridge.rb

@@ -24,13 +24,16 @@ module Selenium
       class Bridge < Remote::Bridge
         def initialize(opts = {})
           opts[:desired_capabilities] ||= Remote::Capabilities.safari
-          port = opts.delete(:port) || Service::DEFAULT_PORT
-          service_args = opts.delete(:service_args) || {}
 
-          driver_path = opts.delete(:driver_path) || Safari.driver_path(false)
-          @service = Service.new(driver_path, port, *extract_service_args(service_args))
-          @service.start
-          opts[:url] = @service.uri
+          unless opts.key?(:url)
+            port = opts.delete(:port) || Service::DEFAULT_PORT
+            service_args = opts.delete(:service_args) || {}
+
+            driver_path = opts.delete(:driver_path) || Safari.driver_path
+            @service = Service.new(driver_path, port, *extract_service_args(service_args))
+            @service.start
+            opts[:url] = @service.uri
+          end
 
           super(opts)
         end

data/lib/selenium/webdriver/safari/service.rb

@@ -48,6 +48,8 @@ module Selenium
         def start_process
           server_command = [@executable_path, "--port=#{@port}", *@extra_args]
           @process = ChildProcess.build(*server_command)
+          WebDriver.logger.debug("Executing Process #{server_command}")
+
           @process.io.stdout = @process.io.stderr = WebDriver.logger.io
           @process.start
         end

data/selenium-webdriver.gemspec

@@ -5,7 +5,7 @@ raise "cwd must be #{root} when reading gemspec" if root != Dir.pwd
 
 Gem::Specification.new do |s|
   s.name = 'selenium-webdriver'
-  s.version = '3.2.1'
+  s.version = '3.2.2'
 
   s.authors = ['Alex Rodionov', 'Titus Fortner']
   s.email = ['p0deje@gmail.com', 'titusfortner@gmail.com']

metadata

@@ -2,7 +2,7 @@
 name: selenium-webdriver
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 3.2.1
+  version: 3.2.2
 platform: ruby
 authors: 
   - Alex Rodionov
@@ -11,7 +11,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2017-02-24 00:00:00 -06:00
+date: 2017-03-01 00:00:00 -06:00
 default_executable: 
 dependencies: 
   - !ruby/object:Gem::Dependency