add HTMLElements.getElementLC(String) as a slightly optimized version for code paths where we know the lower case element name

rbri · rbri · commit 692ade546e7f · 2025-09-10T16:08:34.000+02:00
diff --git a/src/main/java/org/htmlunit/cyberneko/HTMLElements.java b/src/main/java/org/htmlunit/cyberneko/HTMLElements.java
@@ -620,7 +620,7 @@ public final Element getElement(final short code) {
     /**
      * @return the element information for the specified element name.
      *
-     * @param ename The element name.
+     * @param ename the element name.
      */
     public final Element getElement(final String ename) {
         Element element = getElement(ename, NO_SUCH_ELEMENT);
@@ -636,6 +636,25 @@ public final Element getElement(final String ename) {
         return element;
     }
 
+    /**
+     * @return the element information for the specified element name.
+     *
+     * @param enameLC the element name as lower case.
+     */
+    public final Element getElementLC(final String enameLC) {
+        Element element = getElementLC(enameLC, NO_SUCH_ELEMENT);
+        if (element == NO_SUCH_ELEMENT) {
+            element = new Element(UNKNOWN,
+                                    enameLC.toUpperCase(Locale.ROOT),
+                                    NO_SUCH_ELEMENT.flags,
+                                    NO_SUCH_ELEMENT.parentCodes_,
+                                    NO_SUCH_ELEMENT.bounds,
+                                    NO_SUCH_ELEMENT.closes);
+            element.parent = NO_SUCH_ELEMENT.parent;
+        }
+        return element;
+    }
+
     /**
      * @return the element information for the specified element name.
      *
@@ -673,7 +692,7 @@ public final Element getElement(final String ename, final Element elementIfNotFo
     /**
      * @return the element information for the specified element name.
      *
-     * @param ename the element name in lower case
+     * @param enameLC the element name in lower case
      * @param elementIfNotFound the default element to return if not found.
      */
     public final Element getElementLC(final String enameLC, final Element elementIfNotFound) {
diff --git a/src/main/java/org/htmlunit/cyberneko/HTMLScanner.java b/src/main/java/org/htmlunit/cyberneko/HTMLScanner.java
@@ -2874,41 +2874,43 @@ protected String scanStartElement(final boolean[] empty) throws IOException {
             fBeginColumnNumber = beginColumnNumber;
             fBeginCharacterOffset = beginCharacterOffset;
             if (fElementDepth == -1) {
-                if (fByteStream != null && !fIgnoreSpecifiedCharset_ && "META".equalsIgnoreCase(ename)) {
-                    if (DEBUG_CHARSET) {
-                        System.out.println("+++ <META>");
-                    }
-                    final String httpEquiv = getValue(attributes_, "http-equiv");
-                    if ("content-type".equalsIgnoreCase(httpEquiv)) {
+                if (fByteStream != null) {
+                    final String enameLC = ename.toLowerCase(Locale.ROOT);
+
+                    if ( !fIgnoreSpecifiedCharset_ && "meta".equals(enameLC)) {
                         if (DEBUG_CHARSET) {
-                            System.out.println("+++ @content-type: \"" + httpEquiv + '"');
+                            System.out.println("+++ <META>");
                         }
-                        String content = getValue(attributes_, "content");
-                        if (content != null) {
-                            content = removeSpaces(content);
-                            final int index1 = content.toLowerCase(Locale.ROOT).indexOf("charset=");
-                            if (index1 != -1) {
-                                final int index2 = content.indexOf(';', index1);
-                                final String charset = index2 != -1 ? content.substring(index1 + 8, index2)
-                                        : content.substring(index1 + 8);
-                                changeEncoding(charset);
+                        final String httpEquiv = getValue(attributes_, "http-equiv");
+                        if ("content-type".equalsIgnoreCase(httpEquiv)) {
+                            if (DEBUG_CHARSET) {
+                                System.out.println("+++ @content-type: \"" + httpEquiv + '"');
+                            }
+                            String content = getValue(attributes_, "content");
+                            if (content != null) {
+                                content = removeSpaces(content);
+                                final int index1 = content.toLowerCase(Locale.ROOT).indexOf("charset=");
+                                if (index1 != -1) {
+                                    final int index2 = content.indexOf(';', index1);
+                                    final String charset = index2 != -1 ? content.substring(index1 + 8, index2)
+                                            : content.substring(index1 + 8);
+                                    changeEncoding(charset);
+                                }
                             }
                         }
-                    }
-                    else {
-                        final String metaCharset = getValue(attributes_, "charset");
-                        if (metaCharset != null) {
-                            changeEncoding(metaCharset);
+                        else {
+                            final String metaCharset = getValue(attributes_, "charset");
+                            if (metaCharset != null) {
+                                changeEncoding(metaCharset);
+                            }
                         }
                     }
-                }
-                else if (fByteStream != null) {
-                    if ("BODY".equalsIgnoreCase(ename)) {
+                    else if ("body".equals(enameLC)) {
                         fByteStream.clear();
                         fByteStream = null;
                     }
                     else {
-                        final HTMLElements.Element element = htmlConfiguration_.getHtmlElements().getElement(ename);
+                        final HTMLElements.Element element = htmlConfiguration_.getHtmlElements().getElementLC(enameLC);
                         if (element.parent != null
                                 && element.parent.length > 0
                                 && element.parent[0].code == HTMLElements.BODY) {
