Backed out changeset 0da564b9ec2e (bug 1681046) for causing dt failures in browser_webconsole_worklet_error.js

CLOSED TREE

js/public/CompileOptions.h

@@ -131,7 +131,7 @@ class JS_PUBLIC_API TransitiveCompileOptions {
   bool nonSyntacticScope = false;
   bool privateClassFields = false;
   bool privateClassMethods = false;
-  bool topLevelAwait = true;
+  bool topLevelAwait = false;
 
   // True if transcoding to XDR should use Stencil instead of JSScripts.
   bool useStencilXDR = false;

js/public/ContextOptions.h

@@ -49,7 +49,7 @@ class JS_PUBLIC_API ContextOptions {
         fuzzing_(false),
         privateClassFields_(false),
         privateClassMethods_(false),
-        topLevelAwait_(true) {
+        topLevelAwait_(false) {
   }
 
   bool asmJS() const { return asmJS_; }

js/src/jsapi-tests/testStencil.cpp

@@ -94,9 +94,8 @@ bool basic_test(const CharT* chars) {
   JS::RootedValue rval(cx);
   CHECK(JS::ModuleInstantiate(cx, moduleObject));
   CHECK(JS::ModuleEvaluate(cx, moduleObject, &rval));
-  CHECK(!rval.isUndefined());
+  CHECK(rval.isUndefined());
 
-  js::RunJobs(cx);
   CHECK(JS_GetProperty(cx, global, "x", &rval));
   CHECK(rval.isNumber() && rval.toNumber() == 42);
 
@@ -328,9 +327,8 @@ BEGIN_TEST(testStencil_OffThreadModule) {
   JS::RootedValue rval(cx);
   CHECK(JS::ModuleInstantiate(cx, moduleObject));
   CHECK(JS::ModuleEvaluate(cx, moduleObject, &rval));
-  CHECK(!rval.isUndefined());
+  CHECK(rval.isUndefined());
 
-  js::RunJobs(cx);
   CHECK(JS_GetProperty(cx, global, "x", &rval));
   CHECK(rval.isNumber() && rval.toNumber() == 42);
 

js/src/shell/js.cpp

@@ -631,7 +631,7 @@ bool shell::enablePropertyErrorMessageFix = false;
 bool shell::enableIteratorHelpers = false;
 bool shell::enablePrivateClassFields = false;
 bool shell::enablePrivateClassMethods = false;
-bool shell::enableTopLevelAwait = true;
+bool shell::enableTopLevelAwait = false;
 bool shell::useOffThreadParseGlobal = true;
 #ifdef JS_GC_ZEAL
 uint32_t shell::gZealBits = 0;

js/xpconnect/src/XPCJSContext.cpp

@@ -1000,9 +1000,6 @@ static void ReloadPrefsCallback(const char* pref, void* aXpccx) {
   sWeakRefsExposeCleanupSome = Preferences::GetBool(
       JS_OPTIONS_DOT_STR "experimental.weakrefs.expose_cleanupSome");
 
-  bool topLevelAwaitEnabled =
-      Preferences::GetBool(JS_OPTIONS_DOT_STR "experimental.top_level_await");
-
   // Require private fields disabled outside of nightly.
   bool privateFieldsEnabled = false;
   bool privateMethodsEnabled = false;
@@ -1015,6 +1012,13 @@ static void ReloadPrefsCallback(const char* pref, void* aXpccx) {
       Preferences::GetBool(JS_OPTIONS_DOT_STR "experimental.private_methods");
 #endif
 
+  // Require top level await disabled outside of nightly.
+  bool topLevelAwaitEnabled = false;
+#ifdef NIGHTLY_BUILD
+  topLevelAwaitEnabled =
+      Preferences::GetBool(JS_OPTIONS_DOT_STR "experimental.top_level_await");
+#endif
+
 #ifdef JS_GC_ZEAL
   int32_t zeal = Preferences::GetInt(JS_OPTIONS_DOT_STR "gczeal", -1);
   int32_t zeal_frequency = Preferences::GetInt(

modules/libpref/init/StaticPrefList.yaml

@@ -5517,13 +5517,13 @@
     type: bool
     value: false
     mirror: always
-#endif  // NIGHTLY_BUILD
 
   # Experimental support for Top-level Await in JavaScript.
 -   name: javascript.options.experimental.top_level_await
     type: bool
-    value: true
+    value: false
     mirror: always
+#endif  // NIGHTLY_BUILD
 
 # The amount of time we wait between a request to GC (due to leaving a page) and doing the actual GC, in ms.
 - name: javascript.options.gc_delay