All worker threads should be joined before main thread's V8 isolate is destroyed

third_party/WebKit/Source/web/WebKit.cpp

 /*
  * Copyright (C) 2009 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 63 matching lines @@
 static ModulesInitializer& modulesInitializer()
 {
     DEFINE_STATIC_LOCAL(OwnPtr<ModulesInitializer>, initializer, (adoptPtr(new ModulesInitializer)));
     return *initializer;
 }
 
 void initialize(Platform* platform)
 {
     Platform::initialize(platform);
 
+    V8Initializer::initializeMainThread();
+
     modulesInitializer().initialize();
 
-    V8Initializer::initializeMainThread();
-
     // currentThread is null if we are running on a thread without a message loop.
     if (WebThread* currentThread = platform->currentThread()) {
         DCHECK(!s_endOfTaskRunner);
         s_endOfTaskRunner = new EndOfTaskRunner;
         currentThread->addTaskObserver(s_endOfTaskRunner);
     }
 }
 
 void shutdown()
 {
     ThreadState::current()->cleanupMainThread();
 
     // currentThread() is null if we are running on a thread without a message loop.
     if (Platform::current()->currentThread()) {
         // We don't need to (cannot) remove s_endOfTaskRunner from the current
         // message loop, because the message loop is already destructed before
         // the shutdown() is called.
         delete s_endOfTaskRunner;
         s_endOfTaskRunner = nullptr;
     }
 
+    modulesInitializer().shutdown();
+
     V8Initializer::shutdownMainThread();
 
-    modulesInitializer().shutdown();
-
     Platform::shutdown();
 }
 
 v8::Isolate* mainThreadIsolate()
 {
     return V8PerIsolateData::mainThreadIsolate();
 }
 
 // TODO(tkent): The following functions to wrap LayoutTestSupport should be
 // moved to public/platform/.
@@ skipping 47 matching lines @@
     DCHECK(!reloadPages);
     Page::refreshPlugins();
 }
 
 void decommitFreeableMemory()
 {
     WTF::Partitions::decommitFreeableMemory();
 }
 
 } // namespace blink