[ES6 modules] ModuleMap::GetFetchedModuleScript to return nullptr when entry not found / "fetching"

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

Index: third_party/WebKit/Source/core/dom/ModuleMap.cpp
diff --git a/third_party/WebKit/Source/core/dom/ModuleMap.cpp b/third_party/WebKit/Source/core/dom/ModuleMap.cpp
index ea685c364e41b72adcecbe5c4afa28caeeee8a35..fb97a7fb769db6996a0002d4df4889d5aadddebc 100644
--- a/third_party/WebKit/Source/core/dom/ModuleMap.cpp
+++ b/third_party/WebKit/Source/core/dom/ModuleMap.cpp
@@ -96,7 +96,6 @@ void ModuleMap::Entry::NotifyNewSingleModuleFinished(
 }
 
 ModuleScript* ModuleMap::Entry::GetModuleScript() const {
-  DCHECK(!is_fetching_);
   return module_script_.Get();
 }
 
@@ -146,7 +145,6 @@ void ModuleMap::FetchSingleModuleScript(const ModuleScriptFetchRequest& request,
 
 ModuleScript* ModuleMap::GetFetchedModuleScript(const KURL& url) const {
   MapImpl::const_iterator it = map_.find(url);
-  CHECK_NE(it, map_.end());
   return it->value->GetModuleScript();

[neis, 2017/05/09 08:56:37]

This return statement makes no sense if the url wasn't found.

[kouhei (in TOK), 2017/05/12 20:48:46]

Done.

 }