[NoStatePrefetch] Kill renderer after preload scanning

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

Index: third_party/WebKit/Source/core/html/parser/HTMLDocumentParser.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLDocumentParser.cpp b/third_party/WebKit/Source/core/html/parser/HTMLDocumentParser.cpp
index b06f1a3626107c7e89c5a6eb89320b9066e73ab7..b1f7bb3aaf250f81221291ccd42798d196c79ffe 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLDocumentParser.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLDocumentParser.cpp
@@ -923,6 +923,9 @@ void HTMLDocumentParser::end() {
   ASSERT(!isDetached());
   ASSERT(!isScheduledForResume());
 
+  if (document()->isPrefetchOnly())

[Charlie Harrison, 2016/10/17 13:25:50]

This might make a little more sense in Document::finishedParsing rather than
here. That's generally where document-y things hook parse finish.

[pasko, 2016/10/17 14:25:11]

OK!

Any recommendation on what part of Document::finishedParsing() is more suitable?
I see parser wrap up closer to the end, can put these new checks at the very
end, would it be the most readable place? Would the Document.this still be alive
at that point?

I'll try that after I rebase (which is a non-trivial rebase, mattcary@ added
tests that are relying on old behavior).

[Charlie Harrison, 2016/10/17 14:49:16]

Adding the call to the very end SGTM. Yeah the Document should still be alive by
the end of that method.

[pasko, 2016/10/25 14:49:40]

Done.

+    document()->onPrefetchFinished();
+
   if (m_haveBackgroundParser)
     stopBackgroundParser();