checkstyle fixes

Author: rbri

checkstyle.xml

@@ -134,7 +134,7 @@
             <property name="authorFormat" value="\S \S"/>
         </module>
         <module name="JavadocVariable">
-            <property name="scope" value="protected"/>
+            <property name="accessModifiers" value="protected"/>
         </module>
         <module name="JavadocStyle">
             <property name="scope" value="protected"/>

src/main/java/org/htmlunit/WebResponse.java

@@ -240,7 +240,7 @@ public Charset getContentCharset() {
      * @return {@code true} if the charset of the previous call to {@link #getContentCharset()} was
      *         "tentative".
      * @see <a href="https://html.spec.whatwg.org/multipage/parsing.html#concept-encoding-confidence">
-     * https://html.spec.whatwg.org/multipage/parsing.html#concept-encoding-confidence</a>
+     *     https://html.spec.whatwg.org/multipage/parsing.html#concept-encoding-confidence</a>
      */
     public boolean wasContentCharsetTentative() {
         return wasContentCharsetTentative_;

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

@@ -26,7 +26,7 @@
  * therefore we have this impl as backend.
  *
  * @see <a href="http://www.w3.org/TR/DOM-Level-2-Traversal-Range/traversal.html">
- * DOM-Level-2-Traversal-Range</a>
+ *     DOM-Level-2-Traversal-Range</a>
  * @author <a href="mailto:mike@10gen.com">Mike Dirolf</a>
  * @author Frank Danek
  * @author Ahmed Ashour

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

@@ -224,7 +224,7 @@ public final void setLabelAttribute(final String newLabel) {
      * <a href="http://www.w3.org/TR/html401/">HTML 4.01</a>
      * documentation for details on the use of this attribute.
      * @see <a href="http://www.w3.org/TR/1999/REC-html401-19991224/interact/forms.html#adef-value-OPTION">
-     * initial value if value attribute is not set</a>
+     *     initial value if value attribute is not set</a>
      * @return the value of the attribute {@code value}
      */
     public final String getValueAttribute() {

src/main/java/org/htmlunit/javascript/JavaScriptEngine.java

@@ -103,7 +103,7 @@
  * @author Sven Strickroth
  *
  * @see <a href="http://groups-beta.google.com/group/netscape.public.mozilla.jseng/browse_thread/thread/b4edac57329cf49f/069e9307ec89111f">
- * Rhino and Java Browser</a>
+ *     Rhino and Java Browser</a>
  */
 public class JavaScriptEngine implements AbstractJavaScriptEngine<Script> {
 

src/main/java/org/htmlunit/javascript/host/Notification.java

@@ -29,7 +29,7 @@
  * A Notification.
  *
  * @see <a href="https://developer.mozilla.org/en/docs/Web/API/notification">
- * MDN - Notification</a>
+ *     MDN - Notification</a>
  * @author Marc Guillemot
  * @author Ronald Brill
  * @author Ahmed Ashour

src/main/java/org/htmlunit/javascript/host/Screen.java

@@ -38,7 +38,7 @@
  * @author cd alexndr
  *
  * @see <a href="http://msdn.microsoft.com/en-us/library/ms535868.aspx">
- * MSDN documentation</a>
+ *     MSDN documentation</a>
  * @see <a href="http://www.mozilla.org/docs/dom/domref/dom_window_ref.html">Mozilla documentation</a>
  */
 @JsxClass

src/main/java/org/htmlunit/javascript/host/Window.java

@@ -484,7 +484,7 @@ private URL makeUrlForOpenWindow(final String urlString) {
      * and does not contain an other page than the one that originated the setTimeout.
      *
      * @see <a href="https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/setTimeout">
-     * MDN web docs</a>
+     *     MDN web docs</a>
      *
      * @param context the JavaScript context
      * @param scope the scope
@@ -503,7 +503,7 @@ public static Object setTimeout(final Context context, final Scriptable scope,
      * Sets a chunk of JavaScript to be invoked each time a specified number of milliseconds has elapsed.
      *
      * @see <a href="https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/setInterval">
-     * MDN web docs</a>
+     *     MDN web docs</a>
      * @param context the JavaScript context
      * @param scope the scope
      * @param thisObj the scriptable
@@ -1676,7 +1676,7 @@ public void setOuterHeight(final int height) {
      * Prints the current page. The current implementation uses the {@link PrintHandler}
      * defined for the {@link WebClient} to process the window.
      * @see <a href="http://www.mozilla.org/docs/dom/domref/dom_window_ref85.html">
-     * Mozilla documentation</a>
+     *     Mozilla documentation</a>
      * @see <a href="http://msdn.microsoft.com/en-us/library/ms536672.aspx">MSDN documentation</a>
      */
     @JsxFunction

src/main/java/org/htmlunit/javascript/host/WindowOrWorkerGlobalScopeMixin.java

@@ -106,7 +106,7 @@ public static String btoa(final String stringToEncode, final HtmlUnitScriptable
      * and does not contain an other page than the one that originated the setTimeout.
      *
      * @see <a href="https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/setTimeout">
-     * MDN web docs</a>
+     *     MDN web docs</a>
      *
      * @param context the JavaScript context
      * @param thisObj the scriptable
@@ -131,7 +131,7 @@ public static Object setTimeout(final Context context, final Scriptable thisObj,
      * Sets a chunk of JavaScript to be invoked each time a specified number of milliseconds has elapsed.
      *
      * @see <a href="https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/setInterval">
-     * MDN web docs</a>
+     *     MDN web docs</a>
      * @param context the JavaScript context
      * @param thisObj the scriptable
      * @param args the arguments passed into the method

src/main/java/org/htmlunit/javascript/host/dom/DOMException.java

@@ -31,7 +31,7 @@
  * Exception for DOM manipulations.
  *
  * @see <a href="http://www.w3.org/TR/2000/REC-DOM-Level-2-Core-20001113/core.html#ID-17189187">
- * DOM-Level-2-Core</a>
+ *     DOM-Level-2-Core</a>
  * @author Marc Guillemot
  * @author Frank Danek
  * @author Ahmed Ashour