time to remove IE (issue #735)

Author: rbri

src/main/java/org/htmlunit/javascript/host/html/HTMLFieldSetElement.java

@@ -14,8 +14,6 @@
  */
 package org.htmlunit.javascript.host.html;
 
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
-
 import org.htmlunit.html.DomElement;
 import org.htmlunit.html.HtmlFieldSet;
 import org.htmlunit.html.HtmlForm;
@@ -49,24 +47,6 @@ public void jsConstructor() {
         super.jsConstructor();
     }
 
-    /**
-     * Returns the value of the {@code align} property.
-     * @return the value of the {@code align} property
-     */
-    @JsxGetter(IE)
-    public String getAlign() {
-        return getAlign(false);
-    }
-
-    /**
-     * Sets the value of the {@code align} property.
-     * @param align the value of the {@code align} property
-     */
-    @JsxSetter(IE)
-    public void setAlign(final String align) {
-        setAlign(align, false);
-    }
-
     /**
      * Returns the {@code name} attribute.
      * @return the {@code name} attribute

src/main/java/org/htmlunit/javascript/host/html/HTMLFormElement.java

@@ -16,7 +16,6 @@
 
 import static org.htmlunit.BrowserVersionFeatures.JS_FORM_DISPATCHEVENT_SUBMITS;
 import static org.htmlunit.BrowserVersionFeatures.JS_FORM_USABLE_AS_FUNCTION;
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
 
 import java.io.Serializable;
 import java.net.MalformedURLException;
@@ -368,31 +367,6 @@ else if (subHtmlElement instanceof HTMLInputElement) {
         this.getHtmlForm().submit(submittable);
     }
 
-    /**
-     * Retrieves a form object or an object from an elements collection.
-     * @param index Integer or String that specifies the object or collection to retrieve.
-     *              If this parameter is an integer, it is the zero-based index of the object.
-     *              If this parameter is a string, all objects with matching name or id properties are retrieved,
-     *              and a collection is returned if more than one match is made
-     * @param subIndex Optional. Integer that specifies the zero-based index of the object to retrieve
-     *              when a collection is returned
-     * @return an object or a collection of objects if successful, or null otherwise
-     */
-    @JsxFunction(IE)
-    public Object item(final Object index, final Object subIndex) {
-        if (index instanceof Number) {
-            return getElements().item(index);
-        }
-
-        final String name = JavaScriptEngine.toString(index);
-        final Object response = getWithPreemption(name);
-        if (subIndex instanceof Number && response instanceof HTMLCollection) {
-            return ((HTMLCollection) response).item(subIndex);
-        }
-
-        return response;
-    }
-
     /**
      * Resets this form.
      */

src/main/java/org/htmlunit/javascript/host/html/HTMLFrameElement.java

@@ -14,8 +14,6 @@
  */
 package org.htmlunit.javascript.host.html;
 
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
-
 import org.htmlunit.html.BaseFrameElement;
 import org.htmlunit.html.FrameWindow;
 import org.htmlunit.html.FrameWindow.PageDenied;
@@ -121,24 +119,6 @@ private BaseFrameElement getFrame() {
         return (BaseFrameElement) getDomNodeOrDie();
     }
 
-    /**
-     * Gets the {@code border} attribute.
-     * @return the {@code border} attribute
-     */
-    @JsxGetter(IE)
-    public String getBorder() {
-        return getDomNodeOrDie().getAttributeDirect("border");
-    }
-
-    /**
-     * Sets the {@code border} attribute.
-     * @param border the {@code border} attribute
-     */
-    @JsxSetter(IE)
-    public void setBorder(final String border) {
-        getDomNodeOrDie().setAttribute("border", border);
-    }
-
     /**
      * {@inheritDoc}
      */

src/main/java/org/htmlunit/javascript/host/html/HTMLFrameSetElement.java

@@ -16,7 +16,6 @@
 
 import static org.htmlunit.javascript.configuration.SupportedBrowser.FF;
 import static org.htmlunit.javascript.configuration.SupportedBrowser.FF_ESR;
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
 
 import org.htmlunit.corejs.javascript.Function;
 import org.htmlunit.html.HtmlFrameSet;
@@ -108,24 +107,6 @@ public String getCols() {
         return htmlFrameSet.getColsAttribute();
     }
 
-    /**
-     * Gets the {@code border} attribute.
-     * @return the {@code border} attribute
-     */
-    @JsxGetter(IE)
-    public String getBorder() {
-        return getDomNodeOrDie().getAttributeDirect("border");
-    }
-
-    /**
-     * Sets the {@code border} attribute.
-     * @param border the {@code border} attribute
-     */
-    @JsxSetter(IE)
-    public void setBorder(final String border) {
-        getDomNodeOrDie().setAttribute("border", border);
-    }
-
     /**
      * Returns the {@code onbeforeunload} event handler.
      * @return the {@code onbeforeunload} event handler
@@ -449,23 +430,4 @@ public Function getOnbeforeprint() {
     public void setOnbeforeprint(final Object beforeprint) {
         setEventHandler(Event.TYPE_BEFOREPRINT, beforeprint);
     }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    @JsxGetter(IE)
-    public Function getOnresize() {
-        return super.getOnresize();
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    @Override
-    @JsxSetter(IE)
-    public void setOnresize(final Object resize) {
-        super.setOnresize(resize);
-    }
-
 }

src/main/java/org/htmlunit/javascript/host/html/HTMLHeadingElement.java

@@ -14,16 +14,12 @@
  */
 package org.htmlunit.javascript.host.html;
 
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
-
-import org.apache.commons.lang3.ArrayUtils;
 import org.htmlunit.html.HtmlHeading1;
 import org.htmlunit.html.HtmlHeading2;
 import org.htmlunit.html.HtmlHeading3;
 import org.htmlunit.html.HtmlHeading4;
 import org.htmlunit.html.HtmlHeading5;
 import org.htmlunit.html.HtmlHeading6;
-import org.htmlunit.javascript.JavaScriptEngine;
 import org.htmlunit.javascript.configuration.JsxClass;
 import org.htmlunit.javascript.configuration.JsxConstructor;
 import org.htmlunit.javascript.configuration.JsxGetter;
@@ -43,9 +39,6 @@
 @JsxClass(domClass = HtmlHeading6.class)
 public class HTMLHeadingElement extends HTMLElement {
 
-    /** Valid values for the {@link #getClear() clear} property. */
-    private static final String[] VALID_CLEAR_VALUES = {"left", "right", "all", "none"};
-
     /**
      * Creates an instance.
      */
@@ -78,29 +71,4 @@ public String getAlign() {
     public void setAlign(final String align) {
         setAlign(align, false);
     }
-
-    /**
-     * Returns the value of the {@code clear} property.
-     * @return the value of the {@code clear} property
-     */
-    @JsxGetter(IE)
-    public String getClear() {
-        final String clear = getDomNodeOrDie().getAttributeDirect("clear");
-        if (!ArrayUtils.contains(VALID_CLEAR_VALUES, clear)) {
-            return "";
-        }
-        return clear;
-    }
-
-    /**
-     * Sets the value of the {@code clear} property.
-     * @param clear the value of the {@code clear} property
-     */
-    @JsxSetter(IE)
-    public void setClear(final String clear) {
-        if (!ArrayUtils.contains(VALID_CLEAR_VALUES, clear)) {
-            throw JavaScriptEngine.reportRuntimeError("Invalid clear property value: '" + clear + "'.");
-        }
-        getDomNodeOrDie().setAttribute("clear", clear);
-    }
 }

src/main/java/org/htmlunit/javascript/host/html/HTMLIFrameElement.java

@@ -15,7 +15,6 @@
 package org.htmlunit.javascript.host.html;
 
 import static org.htmlunit.BrowserVersionFeatures.JS_IFRAME_ALWAYS_EXECUTE_ONLOAD;
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
 
 import org.htmlunit.html.BaseFrameElement;
 import org.htmlunit.html.FrameWindow;
@@ -139,24 +138,6 @@ public void setOnload(final Object eventHandler) {
         isAttachedToPageDuringOnload_ = getDomNodeOrDie().isAttachedToPage();
     }
 
-    /**
-     * Gets the {@code border} attribute.
-     * @return the {@code border} attribute
-     */
-    @JsxGetter(IE)
-    public String getBorder() {
-        return getDomNodeOrDie().getAttributeDirect("border");
-    }
-
-    /**
-     * Sets the {@code border} attribute.
-     * @param border the {@code border} attribute
-     */
-    @JsxSetter(IE)
-    public void setBorder(final String border) {
-        getDomNodeOrDie().setAttribute("border", border);
-    }
-
     /**
      * Returns the value of the {@code align} property.
      * @return the value of the {@code align} property

src/main/java/org/htmlunit/javascript/host/html/HTMLOptionsCollection.java

@@ -14,13 +14,6 @@
  */
 package org.htmlunit.javascript.host.html;
 
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
-
-import java.io.Serializable;
-import java.util.ArrayList;
-import java.util.List;
-import java.util.function.Supplier;
-
 import org.htmlunit.SgmlPage;
 import org.htmlunit.WebAssert;
 import org.htmlunit.corejs.javascript.Context;
@@ -30,7 +23,6 @@
 import org.htmlunit.corejs.javascript.Scriptable;
 import org.htmlunit.corejs.javascript.ScriptableObject;
 import org.htmlunit.corejs.javascript.Undefined;
-import org.htmlunit.html.DomNode;
 import org.htmlunit.html.ElementFactory;
 import org.htmlunit.html.HtmlOption;
 import org.htmlunit.html.HtmlSelect;
@@ -41,7 +33,6 @@
 import org.htmlunit.javascript.configuration.JsxGetter;
 import org.htmlunit.javascript.configuration.JsxSetter;
 import org.htmlunit.javascript.configuration.JsxSymbol;
-import org.htmlunit.javascript.host.dom.NodeList;
 
 /**
  * This is the array returned by the "options" property of Select.
@@ -55,7 +46,6 @@
  * @author Ronald Brill
  */
 @JsxClass
-@JsxClass(isJSObject = false, value = IE)
 public class HTMLOptionsCollection extends HtmlUnitScriptable {
 
     private HtmlSelect htmlSelect_;
@@ -317,26 +307,6 @@ public void setSelectedIndex(final int index) {
         htmlSelect_.setSelectedIndex(index);
     }
 
-    /**
-     * Returns the child nodes of the current element.
-     * @return the child nodes of the current element
-     */
-    @JsxGetter(IE)
-    public NodeList getChildNodes() {
-        final NodeList childNodes = new NodeList(htmlSelect_, false);
-        childNodes.setElementsSupplier(
-                (Supplier<List<DomNode>> & Serializable)
-                () -> {
-                    final List<DomNode> response = new ArrayList<>();
-                    for (final DomNode child : htmlSelect_.getChildren()) {
-                        response.add(child);
-                    }
-
-                    return response;
-                });
-        return childNodes;
-    }
-
     @JsxSymbol
     public ES6Iterator iterator() {
         return new NativeArrayIterator(getParentScope(), this, NativeArrayIterator.ARRAY_ITERATOR_TYPE.VALUES);

src/main/java/org/htmlunit/javascript/host/html/HTMLParagraphElement.java

@@ -14,11 +14,7 @@
  */
 package org.htmlunit.javascript.host.html;
 
-import static org.htmlunit.javascript.configuration.SupportedBrowser.IE;
-
-import org.apache.commons.lang3.ArrayUtils;
 import org.htmlunit.html.HtmlParagraph;
-import org.htmlunit.javascript.JavaScriptEngine;
 import org.htmlunit.javascript.configuration.JsxClass;
 import org.htmlunit.javascript.configuration.JsxConstructor;
 import org.htmlunit.javascript.configuration.JsxGetter;
@@ -34,9 +30,6 @@
 @JsxClass(domClass = HtmlParagraph.class)
 public class HTMLParagraphElement extends HTMLElement {
 
-    /** Valid values for the {@link #getClear() clear} property. */
-    private static final String[] VALID_CLEAR_VALUES = {"left", "right", "all", "none"};
-
     /**
      * Creates an instance.
      */
@@ -69,29 +62,4 @@ public String getAlign() {
     public void setAlign(final String align) {
         setAlign(align, false);
     }
-
-    /**
-     * Returns the value of the {@code clear} property.
-     * @return the value of the {@code clear} property
-     */
-    @JsxGetter(IE)
-    public String getClear() {
-        final String clear = getDomNodeOrDie().getAttributeDirect("clear");
-        if (!ArrayUtils.contains(VALID_CLEAR_VALUES, clear)) {
-            return "";
-        }
-        return clear;
-    }
-
-    /**
-     * Sets the value of the {@code clear} property.
-     * @param clear the value of the {@code clear} property
-     */
-    @JsxSetter(IE)
-    public void setClear(final String clear) {
-        if (!ArrayUtils.contains(VALID_CLEAR_VALUES, clear)) {
-            throw JavaScriptEngine.reportRuntimeError("Invalid clear property value: '" + clear + "'.");
-        }
-        getDomNodeOrDie().setAttribute("clear", clear);
-    }
 }