refactor[py]: Add type hints to webdriver.py and webelement.py

- Added explicit type annotations to selenium.webdriver.remote.webdriver.py and selenium.webdriver.remote.webelement.py
- Improved code clarity and static type checking
- Ensured compatibility with modern type checkers like Pyright and Mypy

Author: duxtec

py/selenium/webdriver/common/options.py

@@ -19,6 +19,7 @@
 from abc import ABCMeta
 from abc import abstractmethod
 from enum import Enum
+from typing import Any
 from typing import Optional
 
 from selenium.common.exceptions import InvalidArgumentException
@@ -42,7 +43,7 @@ class PageLoadStrategy(str, Enum):
 
 
 class _BaseOptionsDescriptor:
-    def __init__(self, name):
+    def __init__(self, name: str):
         self.name = name
 
     def __get__(self, obj, cls):
@@ -54,7 +55,12 @@ def __get__(self, obj, cls):
             # Return socket url or None if not created yet
             value = obj._caps.get(self.name)
             return None if not isinstance(value, str) else value
-        if self.name in ("acceptInsecureCerts", "strictFileInteractability", "setWindowRect", "se:downloadsEnabled"):
+        if self.name in (
+            "acceptInsecureCerts",
+            "strictFileInteractability",
+            "setWindowRect",
+            "se:downloadsEnabled",
+        ):
             return obj._caps.get(self.name, False)
         return obj._caps.get(self.name)
 
@@ -82,7 +88,9 @@ def __set__(self, obj, value):
         if value in ("normal", "eager", "none"):
             obj.set_capability(self.name, value)
         else:
-            raise ValueError("Strategy can only be one of the following: normal, eager, none")
+            raise ValueError(
+                "Strategy can only be one of the following: normal, eager, none"
+            )
 
 
 class _UnHandledPromptBehaviorDescriptor:
@@ -102,7 +110,13 @@ def __get__(self, obj, cls):
         return obj._caps.get(self.name)
 
     def __set__(self, obj, value):
-        if value in ("dismiss", "accept", "dismiss and notify", "accept and notify", "ignore"):
+        if value in (
+            "dismiss",
+            "accept",
+            "dismiss and notify",
+            "accept and notify",
+            "ignore",
+        ):
             obj.set_capability(self.name, value)
         else:
             raise ValueError(
@@ -130,7 +144,9 @@ def __set__(self, obj, value):
         if all(x in ("implicit", "pageLoad", "script") for x in value.keys()):
             obj.set_capability(self.name, value)
         else:
-            raise ValueError("Timeout keys can only be one of the following: implicit, pageLoad, script")
+            raise ValueError(
+                "Timeout keys can only be one of the following: implicit, pageLoad, script"
+            )
 
 
 class _ProxyDescriptor:
@@ -144,7 +160,9 @@ def __get__(self, obj, cls):
 
     def __set__(self, obj, value):
         if not isinstance(value, Proxy):
-            raise InvalidArgumentException("Only Proxy objects can be passed in.")
+            raise InvalidArgumentException(
+                "Only Proxy objects can be passed in."
+            )
         obj._proxy = value
         obj._caps[self.name] = value.to_capabilities()
 
@@ -218,7 +236,9 @@ class BaseOptions(metaclass=ABCMeta):
         - `None`
     """
 
-    strict_file_interactability = _BaseOptionsDescriptor("strictFileInteractability")
+    strict_file_interactability = _BaseOptionsDescriptor(
+        "strictFileInteractability"
+    )
     """Gets and Sets whether session is about file interactability.
 
     Usage
@@ -328,7 +348,9 @@ class BaseOptions(metaclass=ABCMeta):
         - `None`
     """
 
-    unhandled_prompt_behavior = _UnHandledPromptBehaviorDescriptor("unhandledPromptBehavior")
+    unhandled_prompt_behavior = _UnHandledPromptBehaviorDescriptor(
+        "unhandledPromptBehavior"
+    )
     """:Gets and Sets unhandled prompt behavior, the default is "dismiss and
     notify".
 
@@ -452,7 +474,7 @@ def __init__(self) -> None:
     def capabilities(self):
         return self._caps
 
-    def set_capability(self, name, value) -> None:
+    def set_capability(self, name: Any, value: Any) -> None:
         """Sets a capability."""
         self._caps[name] = value
 
@@ -476,12 +498,12 @@ def enable_mobile(
             self.mobile_options["androidDeviceSerial"] = device_serial
 
     @abstractmethod
-    def to_capabilities(self):
+    def to_capabilities(self) -> dict[Any, Any]:
         """Convert options into capabilities dictionary."""
 
     @property
     @abstractmethod
-    def default_capabilities(self):
+    def default_capabilities(self) -> dict[Any, Any]:
         """Return minimal capabilities necessary as a dictionary."""
 
     def ignore_local_proxy_environment_variables(self) -> None:
@@ -497,14 +519,15 @@ class ArgOptions(BaseOptions):
 
     def __init__(self) -> None:
         super().__init__()
-        self._arguments = []
+        self._arguments: list[Any] = []
+        self._caps: dict[Any, Any]
 
     @property
     def arguments(self):
         """:Returns: A list of arguments needed for the browser."""
         return self._arguments
 
-    def add_argument(self, argument) -> None:
+    def add_argument(self, argument: Any) -> None:
         """Adds an argument to the list.
 
         :Args:
@@ -529,9 +552,9 @@ def ignore_local_proxy_environment_variables(self) -> None:
 
         super().ignore_local_proxy_environment_variables()
 
-    def to_capabilities(self):
+    def to_capabilities(self) -> dict[Any, Any]:
         return self._caps
 
     @property
-    def default_capabilities(self):
+    def default_capabilities(self) -> dict[Any, Any]:
         return {}