Bug 1021001 - Remove AutoCheckCannotGC from NamespaceImports.h, and require users to use the sole definition in GCAPI.h. r=terrence

js/src/NamespaceImports.h

@@ -42,8 +42,6 @@ class MOZ_STACK_CLASS SourceBufferHolder;
 
 class HandleValueArray;
 
-class AutoCheckCannotGC;
-
 }
 
 // Do the importing.
@@ -135,8 +133,6 @@ using JS::HandleValueArray;
 
 using JS::Zone;
 
-using JS::AutoCheckCannotGC;
-
 } /* namespace js */
 
 #endif /* NamespaceImports_h */

js/src/jit/ParallelFunctions.cpp

@@ -17,6 +17,8 @@
 using namespace js;
 using namespace jit;
 
+using JS::AutoCheckCannotGC;
+
 using parallel::Spew;
 using parallel::SpewOps;
 using parallel::SpewBailouts;

js/src/jsarray.cpp

@@ -49,6 +49,8 @@ using mozilla::DebugOnly;
 using mozilla::IsNaN;
 using mozilla::PointerRangeSize;
 
+using JS::AutoCheckCannotGC;
+
 bool
 js::GetLengthProperty(JSContext *cx, HandleObject obj, uint32_t *lengthp)
 {

js/src/jsinferinlines.h

@@ -203,7 +203,7 @@ IdToTypeId(jsid id)
 
     if (JSID_IS_STRING(id)) {
         JSAtom *atom = JSID_TO_ATOM(id);
-        AutoCheckCannotGC nogc;
+        JS::AutoCheckCannotGC nogc;
         bool isNumeric = atom->hasLatin1Chars()
                          ? IdIsNumericTypeId(atom->latin1Range(nogc))
                          : IdIsNumericTypeId(atom->twoByteRange(nogc));

js/src/jsnum.cpp

@@ -46,6 +46,8 @@ using mozilla::NegativeInfinity;
 using mozilla::PodCopy;
 using mozilla::PositiveInfinity;
 using mozilla::RangedPtr;
+
+using JS::AutoCheckCannotGC;
 using JS::GenericNaN;
 
 /*

js/src/jsopcode.cpp

@@ -45,6 +45,8 @@
 using namespace js;
 using namespace js::gc;
 
+using JS::AutoCheckCannotGC;
+
 using js::frontend::IsIdentifier;
 
 /*

js/src/jsstr.cpp

@@ -73,6 +73,8 @@ using mozilla::PodCopy;
 using mozilla::PodEqual;
 using mozilla::SafeCast;
 
+using JS::AutoCheckCannotGC;
+
 typedef Handle<JSLinearString*> HandleLinearString;
 
 static JSLinearString *

js/src/vm/SelfHosting.cpp

@@ -33,6 +33,8 @@
 using namespace js;
 using namespace js::selfhosted;
 
+using JS::AutoCheckCannotGC;
+
 static void
 selfHosting_ErrorReporter(JSContext *cx, const char *message, JSErrorReport *report)
 {

js/src/vm/String-inl.h

@@ -88,7 +88,7 @@ NewFatInlineString(ExclusiveContext *cx, Handle<JSLinearString*> base, size_t st
     if (!s)
         return nullptr;
 
-    AutoCheckCannotGC nogc;
+    JS::AutoCheckCannotGC nogc;
     mozilla::PodCopy(chars, base->chars<CharT>(nogc) + start, length);
     chars[length] = 0;
     return s;

js/src/vm/String.cpp

@@ -24,6 +24,8 @@ using mozilla::PodCopy;
 using mozilla::RangedPtr;
 using mozilla::RoundUpPow2;
 
+using JS::AutoCheckCannotGC;
+
 size_t
 JSString::sizeOfExcludingThis(mozilla::MallocSizeOf mallocSizeOf)
 {

js/src/vm/TypedArrayObject.h

@@ -188,7 +188,7 @@ IsTypedArrayIndex(jsid id, uint64_t *indexp)
     if (MOZ_UNLIKELY(!JSID_IS_STRING(id)))
         return false;
 
-    AutoCheckCannotGC nogc;
+    JS::AutoCheckCannotGC nogc;
     JSAtom *atom = JSID_TO_ATOM(id);
     size_t length = atom->length();