Remove outdated comments

stevecheckoway · stevecheckoway · commit 97274452cc66 · 2021-10-13T16:57:59.000-04:00
The first comment is incorrect. The fragment case was changed to match the nonfragment case in whatwg/html#6399. This introduced a bug that was noted by the second comment indicating that the parser diverged from the standard to fix the bug. The standard was subsequently fixed in whatwg/html#6455, obviating the comment.
diff --git a/gumbo-parser/src/parser.c b/gumbo-parser/src/parser.c
@@ -4423,11 +4423,6 @@ static void handle_in_foreign_content(GumboParser* parser, GumboToken* token) {
     /* Parse error */
     parser_add_parse_error(parser, token);
 
-    /*
-     * Fragment case: If the parser was originally created for the HTML
-     * fragment parsing algorithm, then act as described in the "any other
-     * start tag" entry below.
-     */
     while (
       !(
         is_mathml_integration_point(get_current_node(parser))
@@ -4437,12 +4432,6 @@ static void handle_in_foreign_content(GumboParser* parser, GumboToken* token) {
     ) {
       pop_current_node(parser);
     }
-    // XXX: The spec currently says to handle this using the in body insertion
-    // mode rules. That seems wrong. See
-    // <https://github.com/whatwg/html/issues/6808>. Instead, use the current
-    // insertion mode which seems like it works.
-    //
-    // handle_in_body(parser, token);
     handle_html_content(parser, token);
     return;
   }
