Bug 1735296 - Part 4: Remove unused public APIs for sweeping pointers r=sfink

This should make it hader for users of the public API to put unbarriered
pointers in hash tables.

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

Author: jonco3

js/public/GCPolicyAPI.h

@@ -111,18 +111,6 @@ struct GCPointerPolicy {
     // marking.
     UnsafeTraceRoot(trc, vp, name);
   }
-  static bool needsSweep(T* vp) {
-    if (*vp) {
-      return js::gc::IsAboutToBeFinalizedUnbarriered(vp);
-    }
-    return false;
-  }
-  static bool traceWeak(JSTracer* trc, T* vp) {
-    if (*vp) {
-      return js::TraceManuallyBarrieredWeakEdge(trc, vp, "traceWeak");
-    }
-    return true;
-  }
   static bool isTenured(T v) { return !js::gc::IsInsideNursery(v); }
   static bool isValid(T v) { return js::gc::IsCellPointerValidOrNull(v); }
 };
@@ -165,12 +153,6 @@ struct GCPolicy<JS::Heap<T>> {
   static bool needsSweep(JS::Heap<T>* thingp) {
     return *thingp && js::gc::EdgeNeedsSweep(thingp);
   }
-  static bool traceWeak(JSTracer* trc, JS::Heap<T>* thingp) {
-    if (*thingp) {
-      return js::TraceWeakEdge(trc, thingp, "traceWeak");
-    }
-    return true;
-  }
 };
 
 // GCPolicy<UniquePtr<T>> forwards the contained pointer to GCPolicy<T>.

js/public/TracingAPI.h

@@ -505,30 +505,12 @@ extern JS_PUBLIC_API void UnsafeTraceManuallyBarrieredEdge(JSTracer* trc,
                                                            JSObject** edgep,
                                                            const char* name);
 
-// Not part of the public API, but declared here so we can use it in
-// GCPolicyAPI.h
-template <typename T>
-inline bool TraceManuallyBarrieredWeakEdge(JSTracer* trc, T* thingp,
-                                           const char* name);
-
-template <typename T>
-class WeakHeapPtr;
-
-template <typename T>
-inline bool TraceWeakEdge(JSTracer* trc, WeakHeapPtr<T>* thingp,
-                          const char* name);
-
 namespace gc {
 
 // Return true if the given edge is not live and is about to be swept.
 template <typename T>
 extern JS_PUBLIC_API bool EdgeNeedsSweep(JS::Heap<T>* edgep);
 
-// Not part of the public API, but declared here so we can use it in GCPolicy
-// which is.
-template <typename T>
-bool IsAboutToBeFinalizedUnbarriered(T* thingp);
-
 }  // namespace gc
 
 #ifdef DEBUG