service: move process termination to finally block

If a type error or similar occurs during send_remote_shutdown_command,
one should always terminate the service process, even if an exception
is thrown.
Thus moving self._terminate_process() into a finally block.

Signed-off-by: Sandro Pischinger <[email protected]>

Author: PSandro

py/selenium/webdriver/common/service.py

@@ -157,7 +157,8 @@ def stop(self) -> None:
                 self.send_remote_shutdown_command()
             except TypeError:
                 pass
-            self._terminate_process()
+            finally:
+                self._terminate_process()
 
     def _terminate_process(self) -> None:
         """Terminate the child process.