From 3c2fbc825c469ef4f542e380a1e8d43775649aea Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Wed, 3 Jun 2020 22:11:54 +0200
Subject: [PATCH] LibWeb: Call children_changed() on text nodes when flushing
 characters

Now that we flush characters in a single place, we can call the Text's
children_changed() from there instead of having a goofy targeted hack
for <style> elements. :^)
---
 Libraries/LibWeb/Parser/HTMLDocumentParser.cpp | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/Libraries/LibWeb/Parser/HTMLDocumentParser.cpp b/Libraries/LibWeb/Parser/HTMLDocumentParser.cpp
index d9a18c04a3..25c66047b2 100644
--- a/Libraries/LibWeb/Parser/HTMLDocumentParser.cpp
+++ b/Libraries/LibWeb/Parser/HTMLDocumentParser.cpp
@@ -450,6 +450,7 @@ void HTMLDocumentParser::flush_character_insertions()
     if (m_character_insertion_builder.is_empty())
         return;
     m_character_insertion_node->set_data(m_character_insertion_builder.to_string());
+    m_character_insertion_node->parent()->children_changed();
     m_character_insertion_builder.clear();
 }
 
@@ -1424,13 +1425,6 @@ void HTMLDocumentParser::handle_text(HTMLToken& token)
         return;
     }
 
-    // FIXME: This is a bit hackish, we can simplify this once we don't need to support
-    //        the old parser anymore, since then we don't need to maintain its children_changed() semantics.
-    if (token.is_end_tag() && token.tag_name() == "style") {
-        current_node().children_changed();
-        // NOTE: We don't return here, keep going.
-    }
-
     if (token.is_end_tag()) {
         m_stack_of_open_elements.pop();
         m_insertion_mode = m_original_insertion_mode;