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

third_party/WebKit/Source/core/loader/modulescript/ModuleScriptLoaderTest.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/loader/modulescript/ModuleScriptLoader.h"
 
 #include "bindings/core/v8/V8BindingForCore.h"
 #include "core/dom/Document.h"
 #include "core/dom/Modulator.h"
 #include "core/dom/ModuleScript.h"
@@ skipping 46 matching lines @@
                                   RefPtr<SecurityOrigin> security_origin)
       : script_state_(std::move(script_state)),
         security_origin_(std::move(security_origin)) {}
 
   ~ModuleScriptLoaderTestModulator() override {}
 
   SecurityOrigin* GetSecurityOrigin() override {
     return security_origin_.Get();
   }
 
+  ScriptState* GetScriptState() override { return script_state_.Get(); }
+
   ScriptModule CompileModule(const String& script,
                              const String& url_str,
                              AccessControlStatus access_control_status,
                              const TextPosition& position) override {
     ScriptState::Scope scope(script_state_.Get());
     return ScriptModule::Compile(script_state_->GetIsolate(),
                                  "export default 'foo';", "",
                                  access_control_status);
   }
 
@@ skipping 85 matching lines @@
 
   EXPECT_FALSE(client->WasNotifyFinished())
       << "ModuleScriptLoader unexpectedly finished synchronously.";
   platform_->GetURLLoaderMockFactory()->ServeAsynchronousRequests();
 
   EXPECT_TRUE(client->WasNotifyFinished());
   EXPECT_FALSE(client->GetModuleScript());
 }
 
 }  // namespace blink