Bug 1572782 - Remove unused js::FreeOp alias and remaining references r=tcampbell?

Differential Revision: https://phabricator.services.mozilla.com/D41413

--HG--
extra : moz-landing-system : lando

js/public/TypeDecls.h

@@ -34,7 +34,6 @@ class JSScript;
 class JSString;
 
 namespace js {
-using FreeOp = JSFreeOp;
 class TempAllocPolicy;
 };  // namespace js
 

js/public/Utility.h

@@ -490,7 +490,7 @@ static inline void js_free(void* p) {
  * - For deallocations during GC finalization, use one of the following
  *   operations on the JSFreeOp provided to the finalizer:
  *
- *     FreeOp::{free_,delete_}
+ *     JSFreeOp::{free_,delete_}
  */
 
 /*

js/src/debugger/DebugScript.cpp

@@ -17,7 +17,7 @@
 #include "debugger/Debugger.h"  // for BreakpointSite, Breakpoint
 #include "gc/Barrier.h"         // for GCPtrNativeObject, WriteBarriered
 #include "gc/Cell.h"            // for TenuredCell
-#include "gc/FreeOp.h"          // for FreeOp
+#include "gc/FreeOp.h"          // for JSFreeOp
 #include "gc/GCEnum.h"          // for MemoryUse, MemoryUse::BreakpointSite
 #include "gc/Marking.h"         // for IsAboutToBeFinalized
 #include "gc/Zone.h"            // for Zone
@@ -30,7 +30,7 @@
 #include "vm/Runtime.h"         // for ReportOutOfMemory
 #include "vm/Stack.h"           // for ActivationIterator, Activation
 
-#include "gc/FreeOp-inl.h"     // for FreeOp::free_
+#include "gc/FreeOp-inl.h"     // for JSFreeOp::free_
 #include "gc/GC-inl.h"         // for ZoneCellIter
 #include "gc/Marking-inl.h"    // for CheckGCThingAfterMovingGC
 #include "vm/JSContext-inl.h"  // for JSContext::check

js/src/debugger/DebugScript.h

@@ -21,7 +21,6 @@ namespace js {
 
 class BreakpointSite;
 class Debugger;
-class FreeOp;
 
 // DebugScript manages the internal debugger state for a JSScript, which may be
 // associated with multiple Debuggers.

js/src/debugger/Debugger.cpp

@@ -49,7 +49,7 @@
 #include "frontend/ParseContext.h"       // for UsedNameTracker
 #include "frontend/Parser.h"             // for Parser
 #include "gc/Barrier.h"                  // for GCPtrNativeObject
-#include "gc/FreeOp.h"                   // for FreeOp
+#include "gc/FreeOp.h"                   // for JSFreeOp
 #include "gc/GC.h"                       // for IterateLazyScripts
 #include "gc/GCMarker.h"                 // for GCMarker
 #include "gc/GCRuntime.h"                // for GCRuntime, AutoEnterIteration

js/src/debugger/Debugger.h

@@ -65,7 +65,6 @@ class AutoRealm;
 class CrossCompartmentKey;
 class Debugger;
 class DebuggerEnvironment;
-class FreeOp;
 class PromiseObject;
 namespace gc {
 struct Cell;

js/src/debugger/Frame.cpp

@@ -34,7 +34,7 @@
 #include "debugger/Script.h"               // for DebuggerScript
 #include "frontend/BytecodeCompilation.h"  // for CompileEvalScript
 #include "gc/Barrier.h"                    // for HeapPtr
-#include "gc/FreeOp.h"                     // for FreeOp
+#include "gc/FreeOp.h"                     // for JSFreeOp
 #include "gc/GC.h"                         // for MemoryUse
 #include "gc/Marking.h"                    // for IsAboutToBeFinalized
 #include "gc/Rooting.h"                    // for RootedDebuggerFrame

js/src/debugger/Frame.h

@@ -28,7 +28,6 @@
 namespace js {
 
 class AbstractGeneratorObject;
-class FreeOp;
 class GlobalObject;
 
 /*

js/src/gc/FreeOp.h

@@ -88,7 +88,7 @@ class JSFreeOp {
   // Queue memory that was associated with a GC thing using js::AddCellMemory to
   // be freed when the JSFreeOp is destroyed.
   //
-  // This should not be called on the default FreeOps returned by
+  // This should not be called on the default JSFreeOps returned by
   // JSRuntime/JSContext::defaultFreeOp() since these are not destroyed until
   // the runtime itself is destroyed.
   //
@@ -97,7 +97,7 @@ class JSFreeOp {
   void freeLater(Cell* cell, void* p, size_t nbytes, MemoryUse use);
 
   bool appendJitPoisonRange(const js::jit::JitPoisonRange& range) {
-    // FreeOps other than the defaultFreeOp() are constructed on the stack,
+    // JSFreeOps other than the defaultFreeOp() are constructed on the stack,
     // and won't hold onto the pointers to free indefinitely.
     MOZ_ASSERT(!isDefaultFreeOp());
 
@@ -173,8 +173,4 @@ class JSFreeOp {
   void queueForFreeLater(void* p);
 };
 
-namespace js {
-using FreeOp = JSFreeOp;
-}  // namespace js
-
 #endif  // gc_FreeOp_h

js/src/gc/ZoneAllocator.h

@@ -324,7 +324,7 @@ inline void RemoveCellMemory(gc::Cell* cell, size_t nbytes, MemoryUse use,
 // Initialize an object's reserved slot with a private value pointing to
 // malloc-allocated memory and associate the memory with the object.
 //
-// This call should be matched with a call to FreeOp::free_/delete_ in the
+// This call should be matched with a call to JSFreeOp::free_/delete_ in the
 // object's finalizer to free the memory and update the memory accounting.
 
 inline void InitReservedSlot(NativeObject* obj, uint32_t slot, void* ptr,
@@ -341,7 +341,7 @@ inline void InitReservedSlot(NativeObject* obj, uint32_t slot, T* ptr,
 // Initialize an object's private slot with a pointer to malloc-allocated memory
 // and associate the memory with the object.
 //
-// This call should be matched with a call to FreeOp::free_/delete_ in the
+// This call should be matched with a call to JSFreeOp::free_/delete_ in the
 // object's finalizer to free the memory and update the memory accounting.
 
 inline void InitObjectPrivate(NativeObject* obj, void* ptr, size_t nbytes,

js/xpconnect/idl/nsIXPCScriptable.idl

@@ -14,7 +14,6 @@
 
 #include "js/TypeDecls.h"
 
-struct JSFreeOp;
 namespace js {
 struct Class;
 }