Bug 1612447, part 1 - Remove some XPCMap Remove methods that are never called. r=bzbarsky

Things are now removed from these maps using iterators.

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

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

js/xpconnect/src/XPCMaps.h

@@ -127,18 +127,6 @@ class Native2WrappedNativeMap {
     return wrapper;
   }
 
-  inline void Remove(XPCWrappedNative* wrapper) {
-    MOZ_ASSERT(wrapper, "bad param");
-#ifdef DEBUG
-    XPCWrappedNative* wrapperInMap = Find(wrapper->GetIdentityObject());
-    MOZ_ASSERT(!wrapperInMap || wrapperInMap == wrapper,
-               "About to remove a different wrapper with the same "
-               "nsISupports identity! This will most likely cause serious "
-               "problems!");
-#endif
-    mTable.Remove(wrapper->GetIdentityObject());
-  }
-
   inline void Clear() { mTable.Clear(); }
 
   inline uint32_t Count() { return mTable.EntryCount(); }
@@ -294,11 +282,6 @@ class ClassInfo2WrappedNativeProtoMap {
     return proto;
   }
 
-  inline void Remove(nsIClassInfo* info) {
-    MOZ_ASSERT(info, "bad param");
-    mTable.Remove(info);
-  }
-
   inline void Clear() { mTable.Clear(); }
 
   inline uint32_t Count() { return mTable.EntryCount(); }
@@ -404,11 +387,6 @@ class XPCWrappedNativeProtoMap {
     return proto;
   }
 
-  inline void Remove(XPCWrappedNativeProto* proto) {
-    MOZ_ASSERT(proto, "bad param");
-    mTable.Remove(proto);
-  }
-
   inline uint32_t Count() { return mTable.EntryCount(); }
 
   PLDHashTable::Iterator Iter() { return mTable.Iter(); }