HTMLParser should use current value of scriptEnabled flag

Source/core/loader/DocumentLoader.cpp

Index: Source/core/loader/DocumentLoader.cpp
diff --git a/Source/core/loader/DocumentLoader.cpp b/Source/core/loader/DocumentLoader.cpp
index 34d665b7cd49036a4a6d50078c79239fb5382c47..8018652a9f59b039f194124e8aba19b95d2455e6 100644
--- a/Source/core/loader/DocumentLoader.cpp
+++ b/Source/core/loader/DocumentLoader.cpp
@@ -501,6 +501,8 @@ void DocumentLoader::ensureWriter(const AtomicString& mimeType, const KURL& over
     if (!overridingURL.isEmpty())
         m_frame->document()->setBaseURLOverride(overridingURL);
 
+    m_frame->document()->initScriptEnabled();

[abarth-chromium, 2014/06/26 19:53:37]

Can we move this call into Document::Document, right after we call
initSecurityContext ?

[mmal, 2014/06/27 19:39:16]

I'm afraid we can't. Causes crash on WebSecurityOrigin::toString().
ContentSettingsObserver uses WebDocument, doesn't it require (at least) Document
object to be fully created?

+
     // Call receivedFirstData() exactly once per load.
     frameLoader()->receivedFirstData();
     m_frame->document()->maybeHandleHttpRefresh(m_response.httpHeaderField("Refresh"), Document::HttpRefreshFromHeader);