[ES6 modules] Fix context leak. ModuleScript should use TraceWrapperV8Reference to hold onto v8::Mo…

third_party/WebKit/Source/bindings/core/v8/ScriptModule.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 "bindings/core/v8/ScriptModule.h"
 
 #include "bindings/core/v8/V8BindingForCore.h"
 #include "core/dom/Modulator.h"
 #include "core/dom/ScriptModuleResolver.h"
 
@@ skipping 50 matching lines @@
   return ScriptModule(isolate, module);
 }
 
 ScriptValue ScriptModule::Instantiate(ScriptState* script_state) {
   v8::Isolate* isolate = script_state->GetIsolate();
   v8::TryCatch try_catch(isolate);
   try_catch.SetVerbose(true);
 
   DCHECK(!IsNull());
   v8::Local<v8::Context> context = script_state->GetContext();
-  bool success = module_->NewLocal(script_state->GetIsolate())
+  bool success = NewLocal(script_state->GetIsolate())
                      ->Instantiate(context, &ResolveModuleCallback);
   if (!success) {
     DCHECK(try_catch.HasCaught());
     return ScriptValue(script_state, try_catch.Exception());
   }
   DCHECK(!try_catch.HasCaught());
   return ScriptValue();
 }
 
 void ScriptModule::Evaluate(ScriptState* script_state) const {
@@ skipping 67 matching lines @@
   if (resolved.IsNull()) {
     DCHECK(exception_state.HadException());
     return v8::MaybeLocal<v8::Module>();
   }
 
   DCHECK(!exception_state.HadException());
   return v8::MaybeLocal<v8::Module>(resolved.module_->NewLocal(isolate));
 }
 
 }  // namespace blink