BREAKING CHANGE: remove all Location-related methods

classes `Location` and `LocationContext` have been removed in Selenium 4.35.0

Author: asolntsev

src/main/java/io/appium/java_client/AppiumDriver.java

@@ -44,7 +44,6 @@
 import org.openqa.selenium.remote.RemoteWebDriver;
 import org.openqa.selenium.remote.Response;
 import org.openqa.selenium.remote.codec.w3c.W3CHttpResponseCodec;
-import org.openqa.selenium.remote.html5.RemoteLocationContext;
 import org.openqa.selenium.remote.http.HttpClient;
 import org.openqa.selenium.remote.http.HttpMethod;
 
@@ -81,8 +80,6 @@ public class AppiumDriver extends RemoteWebDriver implements
     // frequently used command parameters
     @Getter
     private final URL remoteAddress;
-    @Deprecated(forRemoval = true)
-    protected final RemoteLocationContext locationContext;
     private final ExecuteMethod executeMethod;
     private final Set<String> absentExtensionNames = new HashSet<>();
     private URI biDiUri;
@@ -100,7 +97,6 @@ public class AppiumDriver extends RemoteWebDriver implements
     public AppiumDriver(HttpCommandExecutor executor, Capabilities capabilities) {
         super(executor, capabilities);
         this.executeMethod = new AppiumExecutionMethod(this);
-        locationContext = new RemoteLocationContext(executeMethod);
         super.setErrorHandler(ERROR_HANDLER);
         this.remoteAddress = executor.getAddressOfRemoteServer();
     }
@@ -176,7 +172,6 @@ public AppiumDriver(URL remoteSessionAddress, String platformName, String automa
         executor.setResponseCodec(new W3CHttpResponseCodec());
         setCommandExecutor(executor);
         this.executeMethod = new AppiumExecutionMethod(this);
-        this.locationContext = new RemoteLocationContext(executeMethod);
         super.setErrorHandler(ERROR_HANDLER);
         this.remoteAddress = executor.getAddressOfRemoteServer();
 

src/main/java/io/appium/java_client/android/AndroidDriver.java

@@ -43,7 +43,6 @@
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.Platform;
 import org.openqa.selenium.remote.HttpCommandExecutor;
-import org.openqa.selenium.remote.html5.RemoteLocationContext;
 import org.openqa.selenium.remote.http.ClientConfig;
 import org.openqa.selenium.remote.http.HttpClient;
 
@@ -251,20 +250,6 @@ public AndroidBatteryInfo getBatteryInfo() {
         return new AndroidBatteryInfo(CommandExecutionHelper.executeScript(this, "mobile: batteryInfo"));
     }
 
-    /**
-     * Provides the location context.
-     *
-     * @return instance of {@link RemoteLocationContext}
-     * @deprecated This method, {@link org.openqa.selenium.html5.LocationContext} and {@link RemoteLocationContext}
-     *     interface are deprecated, use {@link #getLocation()} and
-     *     {@link #setLocation(io.appium.java_client.Location)} instead.
-     */
-    @Override
-    @Deprecated(forRemoval = true)
-    public RemoteLocationContext getLocationContext() {
-        return locationContext;
-    }
-
     @Override
     public synchronized StringWebSocketClient getLogcatClient() {
         if (logcatClient == null) {

src/main/java/io/appium/java_client/ios/IOSDriver.java

@@ -44,7 +44,6 @@
 import org.openqa.selenium.remote.DriverCommand;
 import org.openqa.selenium.remote.HttpCommandExecutor;
 import org.openqa.selenium.remote.Response;
-import org.openqa.selenium.remote.html5.RemoteLocationContext;
 import org.openqa.selenium.remote.http.ClientConfig;
 import org.openqa.selenium.remote.http.HttpClient;
 
@@ -279,20 +278,6 @@ class IOSAlert implements Alert {
 
     }
 
-    /**
-     * Provides the location context.
-     *
-     * @return instance of {@link RemoteLocationContext}
-     * @deprecated This method, {@link org.openqa.selenium.html5.LocationContext} and {@link RemoteLocationContext}
-     *     interface are deprecated, use {@link #getLocation()} and
-     *     {@link #setLocation(io.appium.java_client.Location)} instead.
-     */
-    @Override
-    @Deprecated(forRemoval = true)
-    public RemoteLocationContext getLocationContext() {
-        return locationContext;
-    }
-
     @Override
     public synchronized StringWebSocketClient getSyslogClient() {
         if (syslogClient == null) {

src/main/java/io/appium/java_client/remote/SupportsLocation.java

@@ -21,42 +21,17 @@
 import io.appium.java_client.MobileCommand;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.WebDriverException;
-import org.openqa.selenium.html5.Location;
-import org.openqa.selenium.html5.LocationContext;
-import org.openqa.selenium.remote.html5.RemoteLocationContext;
 
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Optional;
 
-public interface SupportsLocation extends WebDriver, ExecutesMethod, LocationContext {
-
-    /**
-     * Provides the location context.
-     *
-     * @return instance of {@link RemoteLocationContext}
-     * @deprecated This method, {@link LocationContext} and {@link RemoteLocationContext} interface are deprecated, use
-     *     {@link #getLocation()} and {@link #setLocation(io.appium.java_client.Location)} instead.
-     */
-    @Deprecated(forRemoval = true)
-    RemoteLocationContext getLocationContext();
-
-    /**
-     * Gets the current device's geolocation coordinates.
-     *
-     * @return A {@link Location} containing the location information. Returns null if the location is not available
-     * @deprecated This method and whole {@link LocationContext} interface are deprecated, use {@link #getLocation()}
-     *     instead.
-     */
-    @Deprecated(forRemoval = true)
-    default Location location() {
-        return getLocationContext().location();
-    }
+public interface SupportsLocation extends WebDriver, ExecutesMethod {
 
     /**
      * Gets the current device's geolocation coordinates.
      *
-     * @return A {@link Location} containing the location information. Throws {@link WebDriverException} if the
+     * @return A {@link io.appium.java_client.Location} containing the location information. Throws {@link WebDriverException} if the
      *     location is not available.
      */
     default io.appium.java_client.Location getLocation() {
@@ -71,19 +46,7 @@ default io.appium.java_client.Location getLocation() {
     /**
      * Sets the current device's geolocation coordinates.
      *
-     * @param location A {@link Location} containing the new location information.
-     * @deprecated This method and whole {@link LocationContext} interface are deprecated, use
-     *     {@link #setLocation(io.appium.java_client.Location)} instead.
-     */
-    @Deprecated(forRemoval = true)
-    default void setLocation(Location location) {
-        getLocationContext().setLocation(location);
-    }
-
-    /**
-     * Sets the current device's geolocation coordinates.
-     *
-     * @param location A {@link Location} containing the new location information.
+     * @param location A {@link io.appium.java_client.Location} containing the new location information.
      */
     default void setLocation(io.appium.java_client.Location location) {
         var locationParameters = new HashMap<String, Object>();