[Extensions] Don't require() a module for calling a method

extensions/renderer/api_test_base.cc

 // Copyright 2014 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 "extensions/renderer/api_test_base.h"
 
 #include <utility>
 #include <vector>
 
 #include "base/location.h"
@@ skipping 204 matching lines @@
   bool did_run = false;
   auto callback = [](bool* did_run, const base::Closure& quit_closure,
                      const std::string& test_name,
                      const std::vector<v8::Local<v8::Value>>& result) {
     *did_run = true;
     if (result.empty() || result[0].IsEmpty() || !result[0]->IsTrue()) {
       base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, quit_closure);
       FAIL() << "Failed to run test \"" << test_name << "\"";
     }
   };
+
+  ASSERT_FALSE(
+      env()->module_system()->Require("testBody").ToLocalChecked().IsEmpty());
   env()->module_system()->CallModuleMethodSafe(
       "testBody", test_name, 0, nullptr,
       base::Bind(callback, &did_run, quit_closure, test_name));
   ASSERT_TRUE(did_run);
 }
 
 void ApiTestEnvironment::RunPromisesAgain() {
   v8::MicrotasksScope::PerformCheckpoint(env()->isolate());
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind(&ApiTestEnvironment::RunPromisesAgain,
@@ skipping 11 matching lines @@
   test_env_.reset(new ApiTestEnvironment(env()));
 }
 
 void ApiTestBase::RunTest(const std::string& file_name,
                           const std::string& test_name) {
   ExpectNoAssertionsMade();
   test_env_->RunTest(file_name, test_name);
 }
 
 }  // namespace extensions