Support Inline module script

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

Index: third_party/WebKit/Source/core/dom/ModulatorImpl.cpp
diff --git a/third_party/WebKit/Source/core/dom/ModulatorImpl.cpp b/third_party/WebKit/Source/core/dom/ModulatorImpl.cpp
index 50a86e9c9488451f5af942c78423faeffbb567a4..e593e3b2d145a3afc89839236b45b9e7b2b79bb3 100644
--- a/third_party/WebKit/Source/core/dom/ModulatorImpl.cpp
+++ b/third_party/WebKit/Source/core/dom/ModulatorImpl.cpp
@@ -84,6 +84,13 @@ void ModulatorImpl::FetchTreeInternal(const ModuleScriptFetchRequest& request,
   tree_linker_registry_->Fetch(request, ancestor_list, level, this, client);
 }
 
+void ModulatorImpl::FetchDescendantsForInlineScript(ModuleScript* module_script,
+                                                    ModuleTreeClient* client) {
+  map_->SetInlineModuleScript(module_script);

[hiroshige, 2017/04/26 04:48:13]

SetInlineModuleScript() is currently needed to put |module_script| (== the
inline module script) to the module map because
ScriptModuleResolverImpl::Resolve() tries to look up |module_script| itself when
we try to instantiate |module_script|'s record.

Probably we shouldn't look up |module_script| in the module map during
instantiation.
(Because it's not clear what is the key for lookup if we continue to look up;
Currently I used the base URL, but of course this causes the problem because
there can be multiple different inline scripts with the same base URL.)

What does the spec say about this?

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

Fixed.

+  tree_linker_registry_->FetchDescendantsForInlineScript(module_script, this,
+                                                         client);
+}
+
 void ModulatorImpl::FetchSingle(const ModuleScriptFetchRequest& request,
                                 ModuleGraphLevel level,
                                 SingleModuleClient* client) {