jdoc fixes

Author: rbri

src/main/java/org/htmlunit/BrowserVersionFeatures.java

@@ -106,7 +106,7 @@ public enum BrowserVersionFeatures {
     @BrowserFeature({CHROME, EDGE})
     EVENT_TYPE_WHEELEVENT,
 
-    /** Form elements are able to refer to the for by using the from attribute. */
+    /** Form elements are able to refer to the for by using the form attribute. */
     @BrowserFeature({CHROME, EDGE})
     FORM_IGNORE_REL_NOREFERRER,
 
@@ -302,7 +302,7 @@ public enum BrowserVersionFeatures {
     @BrowserFeature({FF, FF_ESR})
     JS_CSSRULELIST_ENUM_ITEM_LENGTH,
 
-    /** Javascript document.evaluate creates a new result object even if provided as param. */
+    /** Javascript document.evaluate() creates a new result object even if provided as param. */
     @BrowserFeature({CHROME, EDGE})
     JS_DOCUMENT_EVALUATE_RECREATES_RESULT,
 
@@ -330,7 +330,7 @@ public enum BrowserVersionFeatures {
     @BrowserFeature({FF, FF_ESR})
     JS_FORM_DISPATCHEVENT_SUBMITS,
 
-    /** Executes the {@code onload} handler, regardless of the whether the element was already attached to the page. */
+    /** Executes the {@code onload} handler, regardless of the element was already attached to the page. */
     @BrowserFeature({FF, FF_ESR})
     JS_IFRAME_ALWAYS_EXECUTE_ONLOAD,
 
@@ -352,7 +352,7 @@ public enum BrowserVersionFeatures {
     @BrowserFeature({CHROME, EDGE})
     JS_INNER_TEXT_SVG_NL,
 
-    /** The value is ignored when the type of an week/month input is changed. */
+    /** The value is ignored when the type of the week/month input is changed. */
     @BrowserFeature({CHROME, EDGE})
     JS_INPUT_CHANGE_TYPE_DROPS_VALUE_WEEK_MONTH,
 
@@ -408,7 +408,7 @@ public enum BrowserVersionFeatures {
     @BrowserFeature({FF, FF_ESR})
     JS_PROPERTY_DESCRIPTOR_NAME,
 
-    /** script tags created from js as child of templates are processed if added to the dom. */
+    /** script tags created from js as child of templates is processed if added to the dom. */
     @BrowserFeature({CHROME, EDGE})
     JS_SCRIPT_IN_TEMPLATE_EXECUTED_ON_ATTACH,
 

src/main/java/org/htmlunit/DefaultCredentialsProvider.java

@@ -59,7 +59,7 @@ public class DefaultCredentialsProvider implements CredentialsProvider, Serializ
     private static SocksProxyAuthenticator SocksAuthenticator_;
     private final Map<AuthScopeProxy, Credentials> credentialsMap_ = new HashMap<>();
 
-    // Because this is used for the whole JVM i try to make it as less invasive as possible.
+    // Because this is used for the whole JVM I try to make it as less invasive as possible.
     // But in general this might disturb other application running on the same JVM.
     private static final class SocksProxyAuthenticator extends Authenticator {
         private CredentialsProvider credentialsProvider_;

src/main/java/org/htmlunit/HttpWebConnection.java

@@ -825,7 +825,7 @@ public static DownloadedContent downloadContent(final InputStream is, final int
     /**
      * Constructs an appropriate WebResponse.
      * May be overridden by subclasses to return a specialized WebResponse.
-     * @param responseData Data that was send back
+     * @param responseData Data that was sent back
      * @param webRequest the request used to get this response
      * @param loadTime How long the response took to be sent
      * @return the new WebResponse

src/main/java/org/htmlunit/MockWebConnection.java

@@ -485,19 +485,19 @@ public void setDefaultResponse(final String content, final int statusCode,
     }
 
     /**
-     * Returns the additional headers that were used in the in the last call
+     * Returns the additional headers that were used in the last call
      * to {@link #getResponse(WebRequest)}.
-     * @return the additional headers that were used in the in the last call
+     * @return the additional headers that were used in the last call
      *         to {@link #getResponse(WebRequest)}
      */
     public Map<String, String> getLastAdditionalHeaders() {
         return lastRequest_.getAdditionalHeaders();
     }
 
     /**
-     * Returns the {@link WebRequest} that was used in the in the last call
+     * Returns the {@link WebRequest} that was used in the last call
      * to {@link #getResponse(WebRequest)}.
-     * @return the {@link WebRequest} that was used in the in the last call
+     * @return the {@link WebRequest} that was used in the last call
      *         to {@link #getResponse(WebRequest)}
      */
     public WebRequest getLastWebRequest() {

src/main/java/org/htmlunit/attachment/AttachmentHandler.java

@@ -27,7 +27,7 @@
  * {@code Content-Disposition=attachment} headers. Normally pages are loaded inline: clicking on
  * a link, for example, loads the linked page in the current window. Attached pages are different
  * in that they are intended to be loaded outside of this flow: clicking on a link prompts the
- * user to either save the linked page, or open it outside of the current window, but does not
+ * user to either save the linked page, or open it outside the current window, but does not
  * load the page in the current window.</p>
  *
  * <p>HtmlUnit complies with the semantics described above when an <code>AttachmentHandler</code> has

src/main/java/org/htmlunit/html/AbstractDomNodeList.java

@@ -77,7 +77,7 @@ protected DomNode getDomNode() {
      * @return the nodes in this node list
      */
     private List<E> getNodes() {
-        // a bit of a hack but I like to avoid synchronization
+        // a bit of a hack, but I like to avoid synchronization
         // see https://github.com/HtmlUnit/htmlunit/issues/882
         //
         // there is a small chance that the cachedElements_ are

src/main/java/org/htmlunit/html/DomElement.java

@@ -1254,9 +1254,9 @@ public <P extends Page> P dblClick() throws IOException {
      * action listeners, etc. Note also that {@link #click(boolean, boolean, boolean)} is automatically
      * called first.
      *
-     * @param shiftKey {@code true} if SHIFT is pressed during the double-click
-     * @param ctrlKey {@code true} if CTRL is pressed during the double-click
-     * @param altKey {@code true} if ALT is pressed during the double-click
+     * @param shiftKey {@code true} if SHIFT is pressed during the double click
+     * @param ctrlKey {@code true} if CTRL is pressed during the double click
+     * @param altKey {@code true} if ALT is pressed during the double click
      * @param <P> the page type
      * @return the page that occupies this element's window after the element has been double-clicked
      * @exception IOException if an IO error occurs
@@ -1736,12 +1736,12 @@ class NamedAttrNodeMapImpl implements Map<String, DomAttr>, NamedNodeMap, Serial
 
         if (attributes instanceof OrderedFastHashMapWithLowercaseKeys) {
             // no need to rework the map at all, we are case sensitive, so
-            // we keep all attributes and we got the right map from outside too
+            // we keep all attributes, and we got the right map from outside too
             map_ = (OrderedFastHashMap) attributes;
         }
         else if (caseSensitive && attributes instanceof OrderedFastHashMap) {
             // no need to rework the map at all, we are case sensitive, so
-            // we keep all attributes and we got the right map from outside too
+            // we keep all attributes, and we got the right map from outside too
             map_ = (OrderedFastHashMap) attributes;
         }
         else {

src/main/java/org/htmlunit/html/DomNode.java

@@ -126,7 +126,7 @@ public abstract class DomNode implements Cloneable, Serializable, Node {
      */
     private HtmlUnitScriptable scriptObject_;
 
-    /** The ready state is an value that is available to a large number of elements. */
+    /** The ready state is a value that is available to a large number of elements. */
     private String readyState_;
 
     /**
@@ -879,7 +879,7 @@ public DomNode cloneNode(final boolean deep) {
      *
      * <p>Returns the JavaScript object that corresponds to this node, lazily initializing a new one if necessary.</p>
      *
-     * <p>The logic of when and where the JavaScript object is created needs a clean up: functions using
+     * <p>The logic of when and where the JavaScript object is created needs a cleanup: functions using
      * a DOM node's JavaScript object should not have to check if they should create it first.</p>
      *
      * @param <T> the object type

src/main/java/org/htmlunit/html/HtmlElement.java

@@ -437,7 +437,7 @@ public String getNodeName() {
     }
 
     /**
-     * Returns this element's tab index, if it has one. If the tab index is outside of the
+     * Returns this element's tab index, if it has one. If the tab index is outside the
      * valid range (less than <code>0</code> or greater than <code>32767</code>), this method
      * returns {@link #TAB_INDEX_OUT_OF_BOUNDS}. If this element does not have
      * a tab index, or its tab index is otherwise invalid, this method returns {@code null}.
@@ -480,7 +480,7 @@ public HtmlElement getEnclosingElement(final String tagName) {
 
     /**
      * Returns the form which contains this element, or {@code null} if this element is not inside
-     * of a form.
+     * a form.
      * @return the form which contains this element
      */
     public HtmlForm getEnclosingForm() {

src/main/java/org/htmlunit/html/HtmlForm.java

@@ -631,7 +631,7 @@ public List<HtmlElement> getElements(final Predicate<HtmlElement> filter) {
      * @param name the input name to search for
      * @param <I> the input type
      * @return the first input element which is a member of this form and has the specified name
-     * @throws ElementNotFoundException if there is not input in this form with the specified name
+     * @throws ElementNotFoundException if there is no input in this form with the specified name
      */
     @SuppressWarnings("unchecked")
     public final <I extends HtmlInput> I getInputByName(final String name) throws ElementNotFoundException {