[HTML Imports] Let script of imported document running.

Source/core/html/parser/HTMLDocumentParser.cpp

 /*
  * Copyright (C) 2010 Google, Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 443 matching lines @@
     ASSERT(!isStopped());
 
     // FIXME: Pass in current input length.
     InspectorInstrumentationCookie cookie = InspectorInstrumentation::willWriteHTML(document(), lineNumber().zeroBasedInt());
 
     double startTime = currentTime();
 
     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()) {
             m_parserScheduler->scheduleForResume();
             break;
         }
     }
 
     InspectorInstrumentation::didWriteHTML(cookie, lineNumber().zeroBasedInt());
 }
@@ skipping 465 matching lines @@
         m_parserScheduler->suspend();
 }
 
 void HTMLDocumentParser::resumeScheduledTasks()
 {
     if (m_parserScheduler)
         m_parserScheduler->resume();
 }
 
 }