[java] Enabling a couple of Firefox tests

And disabling a Chrome test after reporting
the bug to ChromeDriver.

Author: diemol

java/test/org/openqa/selenium/PageLoadingTest.java

@@ -17,6 +17,20 @@
 
 package org.openqa.selenium;
 
+import org.junit.Test;
+import org.openqa.selenium.remote.CapabilityType;
+import org.openqa.selenium.support.ui.WebDriverWait;
+import org.openqa.selenium.testing.Ignore;
+import org.openqa.selenium.testing.JUnit4TestBase;
+import org.openqa.selenium.testing.NeedsFreshDriver;
+import org.openqa.selenium.testing.NoDriverAfterTest;
+import org.openqa.selenium.testing.NoDriverBeforeTest;
+import org.openqa.selenium.testing.NotYetImplemented;
+import org.openqa.selenium.testing.SwitchToTopAfterTest;
+
+import java.time.Duration;
+import java.util.Set;
+
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
@@ -35,20 +49,6 @@
 import static org.openqa.selenium.testing.drivers.Browser.IE;
 import static org.openqa.selenium.testing.drivers.Browser.SAFARI;
 
-import org.junit.Test;
-import org.openqa.selenium.remote.CapabilityType;
-import org.openqa.selenium.support.ui.WebDriverWait;
-import org.openqa.selenium.testing.Ignore;
-import org.openqa.selenium.testing.JUnit4TestBase;
-import org.openqa.selenium.testing.NeedsFreshDriver;
-import org.openqa.selenium.testing.NoDriverAfterTest;
-import org.openqa.selenium.testing.NoDriverBeforeTest;
-import org.openqa.selenium.testing.NotYetImplemented;
-import org.openqa.selenium.testing.SwitchToTopAfterTest;
-
-import java.time.Duration;
-import java.util.Set;
-
 public class PageLoadingTest extends JUnit4TestBase {
 
   private void initDriverWithLoadStrategy(String strategy) {
@@ -245,7 +245,9 @@ public void testShouldBeAbleToLoadAPageWithFramesetsAndWaitUntilAllFramesAreLoad
   @NeedsFreshDriver
   @Test
   @NotYetImplemented(value = HTMLUNIT,
-      reason = "HtmlUnit: can't execute JavaScript before a page is loaded")
+    reason = "HtmlUnit: can't execute JavaScript before a page is loaded")
+  @NotYetImplemented(value = CHROME,
+    reason = "https://bugs.chromium.org/p/chromedriver/issues/detail?id=4085")
   @Ignore(value = SAFARI, reason = "Hanging")
   public void testShouldDoNothingIfThereIsNothingToGoBackTo() {
     Set<String> currentWindowHandles = driver.getWindowHandles();

java/test/org/openqa/selenium/TypingTest.java

@@ -17,6 +17,13 @@
 
 package org.openqa.selenium;
 
+import org.junit.Test;
+import org.openqa.selenium.support.ui.ExpectedConditions;
+import org.openqa.selenium.testing.Ignore;
+import org.openqa.selenium.testing.JUnit4TestBase;
+import org.openqa.selenium.testing.NotYetImplemented;
+import org.openqa.selenium.testing.drivers.Browser;
+
 import static com.google.common.base.Joiner.on;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
@@ -27,13 +34,6 @@
 import static org.openqa.selenium.testing.drivers.Browser.IE;
 import static org.openqa.selenium.testing.drivers.Browser.SAFARI;
 
-import org.junit.Test;
-import org.openqa.selenium.support.ui.ExpectedConditions;
-import org.openqa.selenium.testing.Ignore;
-import org.openqa.selenium.testing.JUnit4TestBase;
-import org.openqa.selenium.testing.NotYetImplemented;
-import org.openqa.selenium.testing.drivers.Browser;
-
 public class TypingTest extends JUnit4TestBase {
 
   private static void checkRecordedKeySequence(WebElement element, int expectedKeyCode) {
@@ -414,7 +414,6 @@ public void testDeleteAndBackspaceKeys() {
   }
 
   @Test
-  @NotYetImplemented(value = FIREFOX)
   public void testSpecialSpaceKeys() {
     driver.get(pages.javascriptPage);
 
@@ -425,7 +424,6 @@ public void testSpecialSpaceKeys() {
   }
 
   @Test
-  @NotYetImplemented(value = FIREFOX)
   @NotYetImplemented(value = SAFARI, reason = "Enters dot instead of comma")
   public void testNumberPadKeys() {
     driver.get(pages.javascriptPage);