time to remove IE (issue #735)

Author: rbri

src/main/java/org/htmlunit/html/parser/neko/HtmlUnitNekoDOMBuilder.java

@@ -103,26 +103,15 @@ final class HtmlUnitNekoDOMBuilder extends AbstractSAXParser
 
     static {
         // continue short code enumeration
-        final short isIndexShortCode = HTMLElements.UNKNOWN + 1;
-        final short commandShortCode = isIndexShortCode + 1;
-
-        // isIndex is special - we have to add it here because all browsers moving this to
-        // the body (even if it is not supported)
-        final HTMLElements.Element isIndex = new HTMLElements.Element(isIndexShortCode, "ISINDEX",
-                HTMLElements.Element.CONTAINER, HTMLElements.BODY, null);
+        final short commandShortCode = HTMLElements.UNKNOWN + 1;
 
         final HTMLElements.Element command = new HTMLElements.Element(commandShortCode, "COMMAND",
                 HTMLElements.Element.EMPTY, new short[] {HTMLElements.BODY, HTMLElements.HEAD}, null);
 
-        HTMLElements value;
-
-        value = new HTMLElements();
-        value.setElement(isIndex);
-        HTMLELEMENTS = value;
+        HTMLELEMENTS = new HTMLElements();
 
-        value = new HTMLElements();
+        final HTMLElements value = new HTMLElements();
         value.setElement(command);
-        value.setElement(isIndex);
         HTMLELEMENTS_WITH_CMD = value;
     }
 

src/test/java/org/htmlunit/general/ElementPropertiesTest.java

@@ -8706,7 +8706,7 @@ public void geolocation() throws Exception {
     }
 
     /**
-     * Test {@link org.htmlunit.javascript.host.geo.Geolocation}.
+     * Test {@link org.htmlunit.javascript.host.xml.XMLHttpRequest}.
      *
      * @throws Exception if the test fails
      */

src/test/java/org/htmlunit/general/HostTypeOfTest.java

@@ -2677,7 +2677,7 @@ public void geolocationPosition() throws Exception {
      */
     @Test
     @Alerts("function")
-    public void GeolocationPositionError() throws Exception {
+    public void geolocationPositionError() throws Exception {
         test("GeolocationPositionError");
     }
 

src/test/java/org/htmlunit/general/huge/HostParentOfFTest.java

@@ -262,4 +262,30 @@ public void _Geolocation_Geolocation() throws Exception {
         test("Geolocation", "Geolocation");
     }
 
+    /**
+     * @throws Exception if the test fails
+     */
+    @Test
+    @Alerts("true")
+    public void _GeolocationCoordinates_GeolocationCoordinates() throws Exception {
+        test("GeolocationCoordinates", "GeolocationCoordinates");
+    }
+
+    /**
+     * @throws Exception if the test fails
+     */
+    @Test
+    @Alerts("true")
+    public void _GeolocationPosition_GeolocationPosition() throws Exception {
+        test("GeolocationPosition", "GeolocationPosition");
+    }
+
+    /**
+     * @throws Exception if the test fails
+     */
+    @Test
+    @Alerts("true")
+    public void _GeolocationPositionError_GeolocationPositionError() throws Exception {
+        test("GeolocationPositionError", "GeolocationPositionError");
+    }
 }

src/test/java/org/htmlunit/general/huge/HostParentOfPTest.java

@@ -476,11 +476,7 @@ public void _Promise_Promise() throws Exception {
      * @throws Exception if the test fails
      */
     @Test
-    @Alerts(DEFAULT = "false",
-            CHROME = "true",
-            EDGE = "true",
-            FF = "true",
-            FF_ESR = "true")
+    @Alerts("true")
     public void _PromiseRejectionEvent_PromiseRejectionEvent() throws Exception {
         test("PromiseRejectionEvent", "PromiseRejectionEvent");
     }
@@ -490,11 +486,7 @@ public void _PromiseRejectionEvent_PromiseRejectionEvent() throws Exception {
      *             if the test fails
      */
     @Test
-    @Alerts(DEFAULT = "false",
-            CHROME = "true",
-            EDGE = "true",
-            FF = "true",
-            FF_ESR = "true")
+    @Alerts("true")
     public void _PushManager_PushManager() throws Exception {
         test("PushManager", "PushManager");
     }
@@ -504,11 +496,7 @@ public void _PushManager_PushManager() throws Exception {
      *             if the test fails
      */
     @Test
-    @Alerts(DEFAULT = "false",
-            CHROME = "true",
-            EDGE = "true",
-            FF = "true",
-            FF_ESR = "true")
+    @Alerts("true")
     public void _PushSubscription_PushSubscription() throws Exception {
         test("PushSubscription", "PushSubscription");
     }
@@ -517,11 +505,7 @@ public void _PushSubscription_PushSubscription() throws Exception {
      * @throws Exception if the test fails
      */
     @Test
-    @Alerts(DEFAULT = "false",
-            CHROME = "true",
-            EDGE = "true",
-            FF = "true",
-            FF_ESR = "true")
+    @Alerts("true")
     public void _PushSubscriptionOptions_PushSubscriptionOptions() throws Exception {
         test("PushSubscriptionOptions", "PushSubscriptionOptions");
     }