[ES6 modules] Introduce ModuleTreeLinker

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"
+
 namespace blink {
 
-void ModuleScript::SetInstantiationError(v8::Isolate* isolate,
-                                         v8::Local<v8::Value> error) {
+void ModuleScript::SetInstantiationErrorAndClearRecord(ScriptValue error) {
+  // Implements Step 7.1 of:
+  // https://html.spec.whatwg.org/multipage/webappapis.html#internal-module-script-graph-fetching-procedure
+
+  // "set script's instantiation state to "errored", ..."
   DCHECK_EQ(instantiation_state_, ModuleInstantiationState::kUninstantiated);
   instantiation_state_ = ModuleInstantiationState::kErrored;
 
+  // "its instantiation error to instantiationStatus.[[Value]], and ..."
   DCHECK(!error.IsEmpty());
-  instantiation_error_.Set(isolate, error);
+  {
+    ScriptState::Scope scope(error.GetScriptState());

[yhirano, 2017/04/18 04:17:48]

Is this ScriptState::Scope really needed? Is just HandleScope enough?

[kouhei (in TOK), 2017/04/18 06:20:09]

Done.

+    instantiation_error_.Set(error.GetIsolate(), error.V8Value());
+  }
+
+  // "its module record to null."
+  record_ = ScriptModule();
 }
 
 void ModuleScript::SetInstantiationSuccess() {
+  // Implements Step 7.2 of:
+  // https://html.spec.whatwg.org/multipage/webappapis.html#internal-module-script-graph-fetching-procedure
+
+  // "set script's instantiation state to "instantiated"."
   DCHECK_EQ(instantiation_state_, ModuleInstantiationState::kUninstantiated);
   instantiation_state_ = ModuleInstantiationState::kInstantiated;
 }
 
 DEFINE_TRACE(ModuleScript) {
   visitor->Trace(settings_object_);
   Script::Trace(visitor);
 }
 DEFINE_TRACE_WRAPPERS(ModuleScript) {
   visitor->TraceWrappers(instantiation_error_);
@@ skipping 16 matching lines @@
 }
 
 String ModuleScript::InlineSourceTextForCSP() const {
   // Currently we don't support inline module scripts.
   // TODO(hiroshige): Implement this.
   NOTREACHED();
   return String();
 }
 
 }  // namespace blink