Changeset 60617 for trunk/src/wp-includes/html-api/class-wp-html-tag-processor.php

Timestamp:

08/07/2025 08:31:07 AM (13 months ago)

Author:

jonsurrell

Message:

HTML API: Reduce length checks in skip_script_data.

Apply an optimization to remove several repeated string length checks in WP_HTML_Tag_Processor::skip_script_data().

Developed in ​https://github.com/WordPress/wordpress-develop/pull/9230.

Props jonsurrell, dmsnell.
See #63738.

File:

• trunk/src/wp-includes/html-api/class-wp-html-tag-processor.php (modified) (7 diffs)

trunk/src/wp-includes/html-api/class-wp-html-tag-processor.php

@@ -1498 +1498 @@
 
                         /*
+                         * Optimization: Terminating a complete script element requires at least eight
+                         * additional bytes in the document. Some checks below may cause local escaped
+                         * state transitions when processing shorter strings, but those transitions are
+                         * irrelevant if the script tag is incomplete and the function must return false.
+                         *
+                         * This may need updating if those transitions become significant or exported from
+                         * this function in some way, such as when building safe methods to embed JavaScript
+                         * or data inside a SCRIPT element.
+                         *
+                         *     $at may be here.
+                         *        ↓
+                         *     ...</script>
+                         *         ╰──┬───╯
+                         *     $at + 8 additional bytes are required for a non-false return value.
+                         *
+                         * This single check eliminates the need to check lengths for the shorter spans:
+                         *
+                         *           $at may be here.
+                         *                  ↓
+                         *     <script><!-- --></script>
+                         *                   ├╯
+                         *             $at + 2 additional characters does not require a length check.
+                         *
+                         * The transition from "escaped" to "unescaped" is not relevant if the document ends:
+                         *
+                         *           $at may be here.
+                         *                  ↓
+                         *     <script><!-- -->[[END-OF-DOCUMENT]]
+                         *                   ╰──┬───╯
+                         *             $at + 8 additional bytes is not satisfied, return false.
+                         */
+                        if ( $at + 8 >= $doc_length ) {
+                                return false;
+                        }
+
+                        /*
                          * For all script states a "-->"  transitions
                          * back into the normal unescaped script mode,
@@ -1503 +1539 @@
                          */
                         if (
-                                $at + 2 < $doc_length &&
                                 '-' === $html[ $at ] &&
                                 '-' === $html[ $at + 1 ] &&
@@ -1511 +1546 @@
                                 $state = 'unescaped';
                                 continue;
-                        }
-
-                        if ( $at + 1 >= $doc_length ) {
-                                return false;
                         }
 
@@ -1538 +1569 @@
                          */
                         if (
-                                $at + 2 < $doc_length &&
                                 '!' === $html[ $at ] &&
                                 '-' === $html[ $at + 1 ] &&
@@ -1562 +1592 @@
                          */
                         if ( ! (
-                                $at + 6 < $doc_length &&
                                 ( 's' === $html[ $at ] || 'S' === $html[ $at ] ) &&
                                 ( 'c' === $html[ $at + 1 ] || 'C' === $html[ $at + 1 ] ) &&
@@ -1580 +1609 @@
                          * "<script" is found within the text.
                          */
-                        if ( $at + 6 >= $doc_length ) {
-                                continue;
-                        }
                         $at += 6;
                         $c   = $html[ $at ];
@@ -1612 +1638 @@
 
                                 if ( $this->bytes_already_parsed >= $doc_length ) {
-                                        $this->parser_state = self::STATE_INCOMPLETE_INPUT;
-
                                         return false;
                                 }