Bug 1478955 part 1 - Rename JSAutoRealm to JSAutoRealmAllowCCW. r=luke

Author: jandem

devtools/shared/heapsnapshot/tests/gtest/DevTools.h

@@ -94,7 +94,7 @@ struct DevTools : public ::testing::Test {
     if (!newGlobal)
       return nullptr;
 
-    JSAutoRealm ar(cx, newGlobal);
+    JSAutoRealmAllowCCW ar(cx, newGlobal);
 
     /* Populate the global object with the standard globals, like Object and
        Array. */

devtools/shared/heapsnapshot/tests/gtest/DoesCrossCompartmentBoundaries.cpp

@@ -18,7 +18,7 @@ DEF_TEST(DoesCrossCompartmentBoundaries, {
     ASSERT_TRUE(newGlobal);
     JS::Compartment* newCompartment = nullptr;
     {
-      JSAutoRealm ar(cx, newGlobal);
+      JSAutoRealmAllowCCW ar(cx, newGlobal);
       ASSERT_TRUE(JS::InitRealmStandardClasses(cx));
       newCompartment = js::GetContextCompartment(cx);
     }

devtools/shared/heapsnapshot/tests/gtest/DoesntCrossCompartmentBoundaries.cpp

@@ -18,7 +18,7 @@ DEF_TEST(DoesntCrossCompartmentBoundaries, {
     ASSERT_TRUE(newGlobal);
     JS::Compartment* newCompartment = nullptr;
     {
-      JSAutoRealm ar(cx, newGlobal);
+      JSAutoRealmAllowCCW ar(cx, newGlobal);
       ASSERT_TRUE(JS::InitRealmStandardClasses(cx));
       newCompartment = js::GetContextCompartment(cx);
     }

dom/base/ChromeUtils.cpp

@@ -217,7 +217,7 @@ ChromeUtils::ShallowClone(GlobalObject& aGlobal,
       return;
     }
 
-    JSAutoRealm ar(cx, obj);
+    JSAutoRealmAllowCCW ar(cx, obj);
 
     if (!JS_Enumerate(cx, obj, &ids) ||
         !values.reserve(ids.length())) {
@@ -240,7 +240,7 @@ ChromeUtils::ShallowClone(GlobalObject& aGlobal,
 
   JS::RootedObject obj(cx);
   {
-    Maybe<JSAutoRealm> ar;
+    Maybe<JSAutoRealmAllowCCW> ar;
     if (aTarget) {
       JS::RootedObject target(cx, js::CheckedUnwrap(aTarget));
       if (!target) {
@@ -483,7 +483,7 @@ namespace module_getter {
 
     JS::RootedValue value(aCx);
     {
-      JSAutoRealm ar(aCx, moduleExports);
+      JSAutoRealmAllowCCW ar(aCx, moduleExports);
 
       if (!JS_GetPropertyById(aCx, moduleExports, id, &value)) {
         return false;
@@ -726,7 +726,7 @@ ChromeUtils::CreateError(const GlobalObject& aGlobal, const nsAString& aMessage,
     uint32_t line = 0;
     uint32_t column = 0;
 
-    Maybe<JSAutoRealm> ar;
+    Maybe<JSAutoRealmAllowCCW> ar;
     JS::RootedObject stack(cx);
     if (aStack) {
       stack = UncheckedUnwrap(aStack);

dom/base/Element.cpp

@@ -3747,7 +3747,7 @@ Element::Animate(const Nullable<ElementOrCSSPseudoElement>& aTarget,
 
   // Animation constructor follows the standard Xray calling convention and
   // needs to be called in the target element's realm.
-  Maybe<JSAutoRealm> ar;
+  Maybe<JSAutoRealmAllowCCW> ar;
   if (js::GetContextCompartment(aContext) !=
       js::GetObjectCompartment(ownerGlobal->GetGlobalJSObject())) {
     ar.emplace(aContext, ownerGlobal->GetGlobalJSObject());

dom/base/IntlUtils.cpp

@@ -88,7 +88,7 @@ IntlUtils::GetDisplayNames(const Sequence<nsString>& aLocales,
   }
 
   // Return the result as DisplayNameResult.
-  JSAutoRealm ar(cx, &retVal.toObject());
+  JSAutoRealmAllowCCW ar(cx, &retVal.toObject());
   if (!aResult.Init(cx, retVal)) {
     aError.Throw(NS_ERROR_FAILURE);
   }
@@ -135,7 +135,7 @@ IntlUtils::GetLocaleInfo(const Sequence<nsString>& aLocales,
   }
 
   // Return the result as LocaleInfo.
-  JSAutoRealm ar(cx, &retVal.toObject());
+  JSAutoRealmAllowCCW ar(cx, &retVal.toObject());
   if (!aResult.Init(cx, retVal)) {
     aError.Throw(NS_ERROR_FAILURE);
   }

dom/base/PostMessageEvent.cpp

@@ -77,7 +77,7 @@ PostMessageEvent::Run()
       targetWindow->IsDying())
     return NS_OK;
 
-  JSAutoRealm ar(cx, targetWindow->GetWrapper());
+  JSAutoRealmAllowCCW ar(cx, targetWindow->GetWrapper());
 
   // Ensure that any origin which might have been provided is the origin of this
   // window's document.  Note that we do this *now* instead of when postMessage

dom/base/StructuredCloneBlob.cpp

@@ -38,7 +38,7 @@ StructuredCloneBlob::Constructor(GlobalObject& aGlobal, JS::HandleValue aValue,
 
   RefPtr<StructuredCloneBlob> holder = StructuredCloneBlob::Create();
 
-  Maybe<JSAutoRealm> ar;
+  Maybe<JSAutoRealmAllowCCW> ar;
   JS::RootedValue value(cx, aValue);
 
   if (aTargetGlobal) {
@@ -87,7 +87,7 @@ StructuredCloneBlob::Deserialize(JSContext* aCx, JS::HandleObject aTargetScope,
   }
 
   {
-    JSAutoRealm ar(aCx, scope);
+    JSAutoRealmAllowCCW ar(aCx, scope);
 
     Read(xpc::NativeGlobal(scope), aCx, aResult, aRv);
     if (aRv.Failed()) {

dom/base/nsContentPermissionHelper.cpp

@@ -713,7 +713,7 @@ nsContentPermissionRequestProxy::Allow(JS::HandleValue aChoices)
 
       JSContext* cx = jsapi.cx();
       JS::Rooted<JSObject*> obj(cx, &aChoices.toObject());
-      JSAutoRealm ar(cx, obj);
+      JSAutoRealmAllowCCW ar(cx, obj);
 
       JS::Rooted<JS::Value> val(cx);
 

dom/base/nsContentUtils.cpp

@@ -7028,7 +7028,7 @@ nsContentUtils::IsPatternMatching(nsAString& aValue, nsAString& aPattern,
 
   // We can use the junk scope here, because we're just using it for
   // regexp evaluation, not actual script execution.
-  JSAutoRealm ar(cx, xpc::UnprivilegedJunkScope());
+  JSAutoRealmAllowCCW ar(cx, xpc::UnprivilegedJunkScope());
 
   // The pattern has to match the entire value.
   aPattern.InsertLiteral(u"^(?:", 0);