Stream scripts to V8 as they load - Blink side.

Source/core/Init.cpp

Index: Source/core/Init.cpp
diff --git a/Source/core/Init.cpp b/Source/core/Init.cpp
index af7c9e1506c20245a48856c202a86bbbb13ee7d2..135f8aff94065359bad5316224f14d48efcb3c6b 100644
--- a/Source/core/Init.cpp
+++ b/Source/core/Init.cpp
@@ -31,6 +31,7 @@
 #include "config.h"
 #include "Init.h"
 
+#include "bindings/core/v8/V8ScriptStreamerThread.h"
 #include "core/EventNames.h"
 #include "core/EventTargetNames.h"
 #include "core/EventTypeNames.h"
@@ -117,14 +118,18 @@ void CoreInitializer::init()
 
     StringImpl::freezeStaticStrings();
 
-    // Creates HTMLParserThread::shared, but does not start the thread.
+    // Creates HTMLParserThread::shared and V8ScriptStreamerThread::shared, but
+    // does not start the threads.
     HTMLParserThread::init();
+    V8ScriptStreamerThread::init();
 }
 
 void CoreInitializer::shutdown()
 {
-    // Make sure we stop the HTMLParserThread before Platform::current() is cleared.
+    // Make sure we stop the HTMLParserThread and V8ScriptStreamerThread before
+    // Platform::current() is cleared.
     HTMLParserThread::shutdown();
+    V8ScriptStreamerThread::shutdown();

[haraken, 2014/08/17 16:05:28]

We want to shutdown threads in the reverse order of init() calls. So let's call
V8ScriptStreamerThread::shutdown() and then HTMLParserThread::shutdown().

[marja, 2014/08/20 11:45:56]

Done. I guess it'd only make a difference if the threads post tasks to each
other, and even then, that can only go one way... it's not like both can post
tasks to each other. Or is there some more realistic use case where it would
matter?

(I was also wondering if we need to shut down the thread before destroying the
Isolate in shutdown() in WebKit.cpp, but I guess not, since the background
parser thread doesn't access the isolate.)

[haraken, 2014/08/20 15:00:19]

I don't know any specific cases where the order matters, but the current
shutdown logic is carefully written in the exactly reverse order of init()
calls. So let's just follow the rule here.

 
     Partitions::shutdown();
 }