Notify the HTMLDocumentParser on document element available

third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp

Index: third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp b/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
index 48def60177b066daa1ab382406affcc826fc7675..7b7592bb68e42a34f527be344a69796668d1ea16 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
@@ -30,6 +30,7 @@
 #include "core/HTMLNames.h"
 #include "core/dom/Comment.h"
 #include "core/dom/DocumentFragment.h"
+#include "core/dom/DocumentParser.h"
 #include "core/dom/DocumentType.h"
 #include "core/dom/Element.h"
 #include "core/dom/ScriptLoader.h"
@@ -382,6 +383,7 @@ HTMLFormElement* HTMLConstructionSite::takeForm()
 void HTMLConstructionSite::dispatchDocumentElementAvailableIfNeeded()
 {
     ASSERT(m_document);
+    m_document->parser()->documentElementAvailable();

[kouhei (in TOK), 2016/06/08 05:09:11]

I'm not a fan of cyclic-dependency introduced here.
HTMLDocumentParser owns HTMLConstructionSite via HTMLTreeBuilder.

Can we remove dispatchDocumentElementAvailableIfNeeded() and move the logic
including this to HTMLHtmlElement::insertedByParser?

[Charlie Harrison, 2016/06/08 22:25:50]

Yeah sure. I agree this is a non ideal architecture. One thing to note is that
|insertedByParser| is called in more places than this method. I've had to make
some changes to get this to work.

     if (m_document->frame() && !m_isParsingFragment) {
         m_document->frame()->loader().dispatchDocumentElementAvailable();
         m_document->frame()->loader().runScriptsAtDocumentElementAvailable();