Support Inline module script

third_party/WebKit/Source/core/dom/ModulatorImpl.cpp

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/dom/ModulatorImpl.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/dom/ModuleMap.h"
 #include "core/dom/ModuleScript.h"
@@ skipping 66 matching lines @@
                                       const AncestorList& ancestor_list,
                                       ModuleGraphLevel level,
                                       ModuleTreeClient* client) {
   // We ensure module-related code is not executed without the flag.
   // https://crbug.com/715376
   CHECK(RuntimeEnabledFeatures::moduleScriptsEnabled());
 
   tree_linker_registry_->Fetch(request, ancestor_list, level, this, client);
 }
 
+void ModulatorImpl::FetchDescendantsForInlineScript(ModuleScript* module_script,
+                                                    ModuleTreeClient* client) {

[kouhei (in TOK), 2017/04/27 03:47:25]

We need to call resolver->RegisterModuleScript() somewhere before the call.
May be we can add the above call in ModuleScript::Create?
as now spec
https://html.spec.whatwg.org/multipage/webappapis.html#creating-a-module-script
Step 5 says to set it there.

[hiroshige, 2017/04/27 21:57:20]

Done in https://codereview.chromium.org/2845743003.

+  // FOXME: Remove this line and revert the changes in ModuleMap before commit.
+  // map_->SetInlineModuleScript(module_script);
+
+  tree_linker_registry_->FetchDescendantsForInlineScript(module_script, this,
+                                                         client);
+}
+
 void ModulatorImpl::FetchSingle(const ModuleScriptFetchRequest& request,
                                 ModuleGraphLevel level,
                                 SingleModuleClient* client) {
   // We ensure module-related code is not executed without the flag.
   // https://crbug.com/715376
   CHECK(RuntimeEnabledFeatures::moduleScriptsEnabled());
 
   map_->FetchSingleModuleScript(request, level, client);
 }
 
@@ skipping 106 matching lines @@
   visitor->Trace(loader_registry_);
   visitor->Trace(tree_linker_registry_);
   visitor->Trace(script_module_resolver_);
 }
 
 DEFINE_TRACE_WRAPPERS(ModulatorImpl) {
   visitor->TraceWrappers(map_);
 }
 
 }  // namespace blink