Deleting Java shims for /window/rect

LiangY · Apr 26, 2017 · 6c6e7ca · 6c6e7ca
1 parent db4db05
commit 6c6e7ca
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 12 deletions.
diff --git a/java/client/src/org/openqa/selenium/remote/http/W3CHttpCommandCodec.java b/java/client/src/org/openqa/selenium/remote/http/W3CHttpCommandCodec.java
@@ -232,9 +232,6 @@ public W3CHttpCommandCodec() {
       case GET_SESSION_STORAGE_SIZE:
         return toScript("return sessionStorage.length");
 
-      case GET_CURRENT_WINDOW_POSITION:
-        return toScript("return {x: window.screenX, y: window.screenY}");
-
       case IS_ELEMENT_DISPLAYED:
         return executeAtom("isDisplayed.js", asElement(parameters.get("id")));
 
@@ -264,12 +261,6 @@ public W3CHttpCommandCodec() {
           .put("value", stringToUtf8Array((String) parameters.get("text")))
           .build();
 
-      case SET_CURRENT_WINDOW_POSITION:
-        return toScript(
-          "window.screenX = arguments[0]; window.screenY = arguments[1]",
-          parameters.get("x"),
-          parameters.get("y"));
-
       case SET_TIMEOUT:
         String timeoutType = (String) parameters.get("type");
         Number duration = (Number) parameters.get("ms");

diff --git a/java/client/test/org/openqa/selenium/WindowTest.java b/java/client/test/org/openqa/selenium/WindowTest.java
@@ -24,15 +24,13 @@
 import static org.junit.Assume.assumeFalse;
 import static org.openqa.selenium.Platform.ANDROID;
 import static org.openqa.selenium.Platform.LINUX;
-import static org.openqa.selenium.testing.Driver.MARIONETTE;
 import static org.openqa.selenium.testing.Driver.PHANTOMJS;
 import static org.openqa.selenium.testing.Driver.SAFARI;
 
 import org.junit.Test;
 import org.openqa.selenium.support.ui.ExpectedCondition;
 import org.openqa.selenium.testing.Ignore;
 import org.openqa.selenium.testing.JUnit4TestBase;
-import org.openqa.selenium.testing.NotYetImplemented;
 import org.openqa.selenium.testing.SwitchToTopAfterTest;
 import org.openqa.selenium.testing.TestUtilities;
 import org.openqa.selenium.testing.drivers.SauceDriver;
@@ -109,7 +107,6 @@ public void testGetsThePositionOfTheCurrentWindow() {
       reason = "getPosition after setPosition doesn't match up exactly, " +
           "as expected - probably due to nuances in Mac OSX window manager.")
   @Ignore(PHANTOMJS)
-  @NotYetImplemented(value = MARIONETTE, reason = "/window/rect")
   public void testSetsThePositionOfTheCurrentWindow() throws InterruptedException {
     // Browser window cannot be resized or moved on ANDROID (and most mobile platforms
     // though others aren't defined in org.openqa.selenium.Platform).