From 29348dfc9a9fbddb4adcef4b8353ef695b8639ce Mon Sep 17 00:00:00 2001 From: Dennis Snell Date: Sat, 10 Aug 2024 04:58:16 +0000 Subject: [PATCH] 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. git-svn-id: https://develop.svn.wordpress.org/trunk@58877 602fd350-edb4-49c9-b593-d223f7449a82 --- .../html-api/class-wp-html-processor.php | 27 ------------------- 1 file changed, 27 deletions(-) diff --git a/src/wp-includes/html-api/class-wp-html-processor.php b/src/wp-includes/html-api/class-wp-html-processor.php index 2bb6302c99..6f5da5477f 100644 --- a/src/wp-includes/html-api/class-wp-html-processor.php +++ b/src/wp-includes/html-api/class-wp-html-processor.php @@ -775,9 +775,6 @@ class WP_HTML_Processor extends WP_HTML_Tag_Processor { * foreign content will also act just like a void tag, immediately * 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 * * @param WP_HTML_Token|null $node Optional. Node to examine, if provided. @@ -3317,12 +3314,6 @@ class WP_HTML_Processor extends WP_HTML_Tag_Processor { case '-TBODY': 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 `
`, 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. return $this->step(); @@ -3453,12 +3444,6 @@ class WP_HTML_Processor extends WP_HTML_Tag_Processor { case '-TBODY': 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 `
`, 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. return $this->step(); @@ -3521,12 +3506,6 @@ class WP_HTML_Processor extends WP_HTML_Tag_Processor { */ 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 `
`, 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. return $this->step(); @@ -3590,12 +3569,6 @@ class WP_HTML_Processor extends WP_HTML_Tag_Processor { case '-TFOOT': 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 `
`, 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. return $this->step();