Adding upstream version 125.0.1.upstream/125.0.1

Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>

1 files changed, 33 insertions, 75 deletions

diff --git a/dom/base/ChromeUtils.cpp b/dom/base/ChromeUtils.cpp
index be06bb083a..0df1cd3c9b 100644
--- a/dom/base/ChromeUtils.cpp
+++ b/dom/base/ChromeUtils.cpp
@@ -602,23 +602,6 @@ void ChromeUtils::Import(const GlobalObject& aGlobal,
   aRetval.set(exports);
 }
 
-static mozJSModuleLoader* GetContextualESLoader(
-    const Optional<bool>& aLoadInDevToolsLoader, JSObject* aGlobal) {
-  RefPtr devToolsModuleloader = mozJSModuleLoader::GetDevToolsLoader();
-  // We should load the module in the DevTools loader if:
-  // - ChromeUtils.importESModule's `loadInDevToolsLoader` option is true, or,
-  // - if the callsite is from a module loaded in the DevTools loader and
-  // `loadInDevToolsLoader` isn't an explicit false.
-  bool shouldUseDevToolsLoader =
-      (aLoadInDevToolsLoader.WasPassed() && aLoadInDevToolsLoader.Value()) ||
-      (devToolsModuleloader && !aLoadInDevToolsLoader.WasPassed() &&
-       devToolsModuleloader->IsLoaderGlobal(aGlobal));
-  if (shouldUseDevToolsLoader) {
-    return mozJSModuleLoader::GetOrCreateDevToolsLoader();
-  }
-  return mozJSModuleLoader::Get();
-}
-
 static mozJSModuleLoader* GetModuleLoaderForCurrentGlobal(
     JSContext* aCx, const GlobalObject& aGlobal,
     Maybe<loader::NonSharedGlobalSyncModuleLoaderScope>&
@@ -629,7 +612,7 @@ static mozJSModuleLoader* GetModuleLoaderForCurrentGlobal(
     return mozJSModuleLoader::Get();
   }
   if (mozJSModuleLoader::IsDevToolsLoaderGlobal(global)) {
-    return mozJSModuleLoader::GetOrCreateDevToolsLoader();
+    return mozJSModuleLoader::GetOrCreateDevToolsLoader(aCx);
   }
 
   if (loader::NonSharedGlobalSyncModuleLoaderScope::IsActive()) {
@@ -681,7 +664,7 @@ static mozJSModuleLoader* GetModuleLoaderForOptions(
     Maybe<loader::NonSharedGlobalSyncModuleLoaderScope>&
         aMaybeSyncLoaderScope) {
   if (!aOptions.mGlobal.WasPassed()) {
-    return GetContextualESLoader(aOptions.mLoadInDevToolsLoader, aGlobal.Get());
+    return mozJSModuleLoader::Get();
   }
 
   switch (aOptions.mGlobal.Value()) {
@@ -689,7 +672,7 @@ static mozJSModuleLoader* GetModuleLoaderForOptions(
       return mozJSModuleLoader::Get();
 
     case ImportESModuleTargetGlobal::Devtools:
-      return mozJSModuleLoader::GetOrCreateDevToolsLoader();
+      return mozJSModuleLoader::GetOrCreateDevToolsLoader(aCx);
 
     case ImportESModuleTargetGlobal::Contextual: {
       if (!NS_IsMainThread()) {
@@ -700,7 +683,7 @@ static mozJSModuleLoader* GetModuleLoaderForOptions(
       RefPtr devToolsModuleloader = mozJSModuleLoader::GetDevToolsLoader();
       if (devToolsModuleloader &&
           devToolsModuleloader->IsLoaderGlobal(aGlobal.Get())) {
-        return mozJSModuleLoader::GetOrCreateDevToolsLoader();
+        return mozJSModuleLoader::GetOrCreateDevToolsLoader(aCx);
       }
       return mozJSModuleLoader::Get();
     }
@@ -715,7 +698,8 @@ static mozJSModuleLoader* GetModuleLoaderForOptions(
 }
 
 static bool ValidateImportOptions(
-    JSContext* aCx, const ImportESModuleOptionsDictionary& aOptions) {
+    JSContext* aCx, const GlobalObject& aGlobal,
+    const ImportESModuleOptionsDictionary& aOptions) {
   if (!NS_IsMainThread() &&
       (!aOptions.mGlobal.WasPassed() ||
        (aOptions.mGlobal.Value() != ImportESModuleTargetGlobal::Current &&
@@ -727,12 +711,17 @@ static bool ValidateImportOptions(
     return false;
   }
 
-  if (aOptions.mGlobal.WasPassed() &&
-      aOptions.mLoadInDevToolsLoader.WasPassed()) {
-    JS_ReportErrorASCII(aCx,
-                        "global option and loadInDevToolsLoader option "
-                        "cannot be used at the same time");
-    return false;
+  if (NS_IsMainThread()) {
+    nsCOMPtr<nsIGlobalObject> global =
+        do_QueryInterface(aGlobal.GetAsSupports());
+
+    if (mozJSModuleLoader::IsDevToolsLoaderGlobal(global) &&
+        !aOptions.mGlobal.WasPassed()) {
+      JS_ReportErrorASCII(aCx,
+                          "ChromeUtils.importESModule: global option is "
+                          "required in DevTools distinct global");
+      return false;
+    }
   }
 
   return true;
@@ -745,7 +734,7 @@ void ChromeUtils::ImportESModule(
     JS::MutableHandle<JSObject*> aRetval, ErrorResult& aRv) {
   JSContext* cx = aGlobal.Context();
 
-  if (!ValidateImportOptions(cx, aOptions)) {
+  if (!ValidateImportOptions(cx, aGlobal, aOptions)) {
     aRv.Throw(NS_ERROR_FAILURE);
     return;
   }
@@ -789,21 +778,11 @@ void ChromeUtils::ImportESModule(
 class EncodedOptions {
  public:
   explicit EncodedOptions(const ImportESModuleOptionsDictionary& aOptions) {
-    uint32_t globalFlag = 0;
     if (aOptions.mGlobal.WasPassed()) {
-      globalFlag = uint32_t(aOptions.mGlobal.Value()) + 1;
-    }
-
-    uint32_t devtoolsFlag = 0;
-    if (aOptions.mLoadInDevToolsLoader.WasPassed()) {
-      if (aOptions.mLoadInDevToolsLoader.Value()) {
-        devtoolsFlag = DevToolsFlag_True;
-      } else {
-        devtoolsFlag = DevToolsFlag_False;
-      }
+      mValue = uint32_t(aOptions.mGlobal.Value()) + 1;
+    } else {
+      mValue = 0;
     }
-
-    mValue = globalFlag | devtoolsFlag;
   }
 
   explicit EncodedOptions(uint32_t aValue) : mValue(aValue) {}
@@ -811,37 +790,14 @@ class EncodedOptions {
   int32_t toInt32() const { return int32_t(mValue); }
 
   void DecodeInto(ImportESModuleOptionsDictionary& aOptions) {
-    uint32_t globalFlag = mValue & GlobalFlag_Mask;
-    if (globalFlag == 0) {
+    if (mValue == 0) {
       aOptions.mGlobal.Reset();
     } else {
-      aOptions.mGlobal.Construct(ImportESModuleTargetGlobal(globalFlag - 1));
-    }
-
-    uint32_t devtoolsFlag = mValue & DevToolsFlag_Mask;
-    switch (devtoolsFlag) {
-      case DevToolsFlag_NotPassed:
-        aOptions.mLoadInDevToolsLoader.Reset();
-        break;
-      case DevToolsFlag_False:
-        aOptions.mLoadInDevToolsLoader.Construct(false);
-        break;
-      case DevToolsFlag_True:
-        aOptions.mLoadInDevToolsLoader.Construct(true);
-        break;
-      default:
-        MOZ_CRASH("Unknown DevToolsFlag");
+      aOptions.mGlobal.Construct(ImportESModuleTargetGlobal(mValue - 1));
     }
   }
 
  private:
-  static constexpr uint32_t GlobalFlag_Mask = 0xF;
-
-  static constexpr uint32_t DevToolsFlag_NotPassed = 0x00;
-  static constexpr uint32_t DevToolsFlag_False = 0x10;
-  static constexpr uint32_t DevToolsFlag_True = 0x20;
-  static constexpr uint32_t DevToolsFlag_Mask = 0x0F0;
-
   uint32_t mValue = 0;
 };
 
@@ -1017,10 +973,6 @@ static bool ModuleGetterImpl(JSContext* aCx, unsigned aArgc, JS::Value* aVp,
     ImportESModuleOptionsDictionary options;
     encodedOptions.DecodeInto(options);
 
-    if (!ValidateImportOptions(aCx, options)) {
-      return false;
-    }
-
     GlobalObject global(aCx, callee);
 
     Maybe<loader::NonSharedGlobalSyncModuleLoaderScope> maybeSyncLoaderScope;
@@ -1198,6 +1150,11 @@ void ChromeUtils::DefineESModuleGetters(
     return;
   }
 
+  if (!ValidateImportOptions(cx, global, aOptions)) {
+    aRv.Throw(NS_ERROR_FAILURE);
+    return;
+  }
+
   EncodedOptions encodedOptions(aOptions);
 
   JS::Rooted<JS::PropertyKey> prop(cx);
@@ -1411,7 +1368,8 @@ void ChromeUtils::ClearStyleSheetCache(GlobalObject&) {
 static WebIDLProcType ProcTypeToWebIDL(mozilla::ProcType aType) {
   // Max is the value of the last enum, not the length, so add one.
   static_assert(
-      WebIDLProcTypeValues::Count == static_cast<size_t>(ProcType::Max) + 1,
+      static_cast<size_t>(MaxContiguousEnumValue<WebIDLProcType>::value) ==
+          static_cast<size_t>(ProcType::Max),
       "In order for this static cast to be okay, "
       "WebIDLProcType must match ProcType exactly");
 
@@ -2118,9 +2076,9 @@ unsigned ChromeUtils::AliveUtilityProcesses(const GlobalObject&) {
 void ChromeUtils::GetAllPossibleUtilityActorNames(GlobalObject& aGlobal,
                                                   nsTArray<nsCString>& aNames) {
   aNames.Clear();
-  for (size_t i = 0; i < WebIDLUtilityActorNameValues::Count; ++i) {
-    auto idlName = static_cast<UtilityActorName>(i);
-    aNames.AppendElement(WebIDLUtilityActorNameValues::GetString(idlName));
+  for (UtilityActorName idlName :
+       MakeWebIDLEnumeratedRange<WebIDLUtilityActorName>()) {
+    aNames.AppendElement(GetEnumString(idlName));
   }
 }