Move a RegisterModuleScript() call to ModuleScript::Create()

third_party/WebKit/Source/core/dom/ModuleScript.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/ModuleScript.h"
 
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/ScriptValue.h"
+#include "core/dom/ScriptModuleResolver.h"
 #include "v8/include/v8.h"
 
 namespace blink {
 
 ModuleScript* ModuleScript::Create(
     const String& source_text,
     Modulator* modulator,
     const KURL& base_url,
     const String& nonce,
     ParserDisposition parser_state,
@@ skipping 25 matching lines @@
     ParserDisposition parser_state,
     WebURLRequest::FetchCredentialsMode credentials_mode) {
   // https://html.spec.whatwg.org/#creating-a-module-script
   // Step 7. Set script's module record to result.
   // Step 8. Set script's base URL to the script base URL provided.
   // Step 9. Set script's cryptographic nonce to the cryptographic nonce
   // provided.
   // Step 10. Set script's parser state to the parser state.
   // Step 11. Set script's credentials mode to the credentials mode provided.
   // Step 12. Return script.
-  return new ModuleScript(modulator, result, base_url, nonce, parser_state,
-                          credentials_mode);
+  ModuleScript* module_script = new ModuleScript(
+      modulator, result, base_url, nonce, parser_state, credentials_mode);
+
+  // Step 5, a part of ParseModule(): Passing script as the last parameter
+  // here ensures result.[[HostDefined]] will be script.
+  modulator->GetScriptModuleResolver()->RegisterModuleScript(module_script);
+
+  return module_script;
 }
 
 ModuleScript* ModuleScript::CreateForTest(
     Modulator* modulator,
     ScriptModule record,
     const KURL& base_url,
     const String& nonce,
     ParserDisposition parser_state,
     WebURLRequest::FetchCredentialsMode credentials_mode) {
   return CreateInternal(modulator, record, base_url, nonce, parser_state,
@@ skipping 52 matching lines @@
 }
 
 String ModuleScript::InlineSourceTextForCSP() const {
   // Currently we don't support inline module scripts.
   // TODO(hiroshige): Implement this.
   NOTREACHED();
   return String();
 }
 
 }  // namespace blink