Changeset 58877

Timestamp:

08/10/2024 04:58:16 AM (5 weeks ago)

Author:

dmsnell

Message:

HTML API: Remove completed TODO comments.

This patch removes TODO comments indicating the need to verify certain behaviors and algorithms. Those verifications have taken place and the comments are no longer needed.

Developed in ​https://github.com/wordpress/wordpress-develop/pull/7174
Discussed in https://core.trac.wordpress.org/ticket/61646

Follow-up to [58867], [58870].

Props jonsurrell.
See #64646.

File:

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

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

@@ -776 +776 @@
      * closing as soon as the processor advances to the next token.
      *
-     * @todo Review the self-closing logic when no node is present, ensure it
-     *       matches the expectations in `step()`.
-     *
      * @since 6.6.0
      *
@@ -3318 +3315 @@
             case '-TFOOT':
             case '-THEAD':
-                /*
-                 * @todo This needs to check if the element in scope is an HTML element, meaning that
-                 *       when SVG and MathML support is added, this needs to differentiate between an
-                 *       HTML element of the given name, such as `<center>`, and a foreign element of
-                 *       the same given name.
-                 */
                 if ( ! $this->state->stack_of_open_elements->has_element_in_table_scope( $tag_name ) ) {
                     // Parse error: ignore the token.
@@ -3454 +3445 @@
             case '-TFOOT':
             case '-THEAD':
-                /*
-                 * @todo This needs to check if the element in scope is an HTML element, meaning that
-                 *       when SVG and MathML support is added, this needs to differentiate between an
-                 *       HTML element of the given name, such as `<center>`, and a foreign element of
-                 *       the same given name.
-                 */
                 if ( ! $this->state->stack_of_open_elements->has_element_in_table_scope( $tag_name ) ) {
                     // Parse error: ignore the token.
@@ -3522 +3507 @@
             case '-TD':
             case '-TH':
-                /*
-                 * @todo This needs to check if the element in scope is an HTML element, meaning that
-                 *       when SVG and MathML support is added, this needs to differentiate between an
-                 *       HTML element of the given name, such as `<center>`, and a foreign element of
-                 *       the same given name.
-                 */
                 if ( ! $this->state->stack_of_open_elements->has_element_in_table_scope( $tag_name ) ) {
                     // Parse error: ignore the token.
@@ -3591 +3570 @@
             case '-THEAD':
             case '-TR':
-                /*
-                 * @todo This needs to check if the element in scope is an HTML element, meaning that
-                 *       when SVG and MathML support is added, this needs to differentiate between an
-                 *       HTML element of the given name, such as `<center>`, and a foreign element of
-                 *       the same given name.
-                 */
                 if ( ! $this->state->stack_of_open_elements->has_element_in_table_scope( $tag_name ) ) {
                     // Parse error: ignore the token.