mirror_php/wordpress
1
0
Fork 0
mirror of git://develop.git.wordpress.org/ synced 2025-03-20 12:00:03 +01:00
Code Issues Actions 15 Packages Projects Releases Wiki Activity

HTML API: Remove completed TODO comments.

Browse Source 
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
This commit is contained in:
Dennis Snell 2024-08-10 04:58:16 +00:00
parent 433397998a
commit 29348dfc9a
1 changed files with 0 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/wp-includes/html-api/class-wp-html-processor.php
View File

@ -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 `<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.
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 `<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.
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 `<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.
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 `<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.
return $this->step();