refactor inspect-based log method finder, use for firefox

Author: bollwyvl

src/SeleniumLibrary/keywords/webdrivertools/webdrivertools.py

@@ -131,6 +131,16 @@ def _remote_capabilities_resolver(self, set_capabilities, default_capabilities):
             caps["browserName"] = default_capabilities["browserName"]
         return {"desired_capabilities": caps}
 
+    def _get_log_method(service_cls, service_log_path):
+        # -- temporary fix to transition selenium to v4.13 from v4.11 and prior
+        from inspect import signature
+        sig = signature(service_cls)
+        if 'log_output' in str(sig):
+            return {'log_output': service_log_path}
+        else:
+            return {'log_path': service_log_path}
+        # --
+
     def create_chrome(
         self,
         desired_capabilities,
@@ -145,14 +155,7 @@ def create_chrome(
             return self._remote(remote_url, options=options)
         if not executable_path:
             executable_path = self._get_executable_path(webdriver.chrome.service.Service)
-        # -- temporary fix to transition selenium to v4.13 from v4.11 and prior
-        from inspect import signature
-        sig = signature(ChromeService)
-        if 'log_output' in str(sig):
-            log_method = {'log_output': service_log_path}
-        else:
-            log_method = {'log_path': service_log_path}
-        # --
+        log_method = self._get_log_method(ChromeService, service_log_path)
         service = ChromeService(executable_path=executable_path, **log_method)
         return webdriver.Chrome(
             options=options,
@@ -203,12 +206,10 @@ def create_firefox(
 
         if remote_url:
             return self._remote(remote_url, options)
-        service_log_path = (
-            service_log_path if service_log_path else self._geckodriver_log
-        )
         if not executable_path:
             executable_path = self._get_executable_path(webdriver.firefox.service.Service)
-        service = FirefoxService(executable_path=executable_path, log_path=service_log_path)
+        log_method = self._get_log_method(FirefoxService, service_log_path or self._geckodriver_log)
+        service = FirefoxService(executable_path=executable_path, **log_method)
         return webdriver.Firefox(
             options=options,
             service=service,