Skip to content

Commit

Permalink
HTML API: Remove completed TODO comments.
Browse files Browse the repository at this point in the history
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 WordPress#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
  • Loading branch information
dmsnell committed Aug 10, 2024
1 parent 4333979 commit 29348df
Showing 1 changed file with 0 additions and 27 deletions.
27 changes: 0 additions & 27 deletions src/wp-includes/html-api/class-wp-html-processor.php
Original file line number Diff line number Diff line change
Expand Up @@ -775,9 +775,6 @@ public function matches_breadcrumbs( $breadcrumbs ): bool {
* 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.
Expand Down Expand Up @@ -3317,12 +3314,6 @@ private function step_in_table_body(): bool {
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();
Expand Down Expand Up @@ -3453,12 +3444,6 @@ private function step_in_row(): bool {
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();
Expand Down Expand Up @@ -3521,12 +3506,6 @@ private function step_in_cell(): bool {
*/
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();
Expand Down Expand Up @@ -3590,12 +3569,6 @@ private function step_in_cell(): bool {
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();
Expand Down

0 comments on commit 29348df

Please sign in to comment.