Bug 1311726 - Move CheckModuleDependeciesLoaded to ModuleLoadRequest; r=jonco

Differential Revision: https://phabricator.services.mozilla.com/D132612
This commit is contained in:
Yulia Startsev
2021-12-08 16:57:19 +00:00
parent d28674c471
commit 4debb9b07d
4 changed files with 22 additions and 23 deletions

View File

@@ -1291,26 +1291,6 @@ void ScriptLoader::EnsureModuleHooksInitialized() {
ModuleLoader::~ModuleLoader() { LOG(("ModuleLoader::~ModuleLoader %p", this)); }
void ScriptLoader::CheckModuleDependenciesLoaded(ModuleLoadRequest* aRequest) {
LOG(("ScriptLoadRequest (%p): Check dependencies loaded", aRequest));
RefPtr<ModuleScript> moduleScript = aRequest->mModuleScript;
if (!moduleScript || moduleScript->HasParseError()) {
return;
}
for (auto childRequest : aRequest->mImports) {
ModuleScript* childScript = childRequest->mModuleScript;
if (!childScript) {
aRequest->mModuleScript = nullptr;
LOG(("ScriptLoadRequest (%p): %p failed (load error)", aRequest,
childRequest.get()));
return;
}
}
LOG(("ScriptLoadRequest (%p): all ok", aRequest));
}
class ScriptRequestProcessor : public Runnable {
private:
RefPtr<ScriptLoader> mLoader;