[HTML Imports] Let script of imported document running.

Source/core/html/parser/HTMLDocumentParser.cpp

Index: Source/core/html/parser/HTMLDocumentParser.cpp
diff --git a/Source/core/html/parser/HTMLDocumentParser.cpp b/Source/core/html/parser/HTMLDocumentParser.cpp
index 50a998529b7271e13c785fd747a5cba24a9d7883..2df7fcdf850afd81fa9de987a79544e40d5db5d0 100644
--- a/Source/core/html/parser/HTMLDocumentParser.cpp
+++ b/Source/core/html/parser/HTMLDocumentParser.cpp
@@ -461,7 +461,9 @@ void HTMLDocumentParser::pumpPendingSpeculations()
     while (!m_speculations.isEmpty()) {
         processParsedChunkFromBackgroundParser(m_speculations.takeFirst());
 
-        if (isWaitingForScripts() || isStopped())
+        // The order matters! If this isStopped(), isWaitingForScripts() can hit and ASSERT since
+        // m_document can be null which is used to decide the readiness.
+        if (isStopped() || isWaitingForScripts())

[abarth-chromium, 2013/07/26 17:17:34]

Can you fix the instance on line 274 as well?

The idiom we use elsewhere in this file is to make these two separate if
statements (so that it's clear the order is important).

Good catch.  Thank you.

             break;
 
         if (currentTime() - startTime > parserTimeLimit && !m_speculations.isEmpty()) {