Bug 1434206 - Remove unused and undefined functions. r=gcp

mProtocolV2 is still used to skip the caching of misses on V4:

https://searchfox.org/mozilla-central/rev/d4b9e50875ad7e5d20f2fee6a53418315f6dfcc0/toolkit/components/url-classifier/nsUrlClassifierDBService.cpp#1353-1357

MozReview-Commit-ID: 2cHh9JiZuHh

--HG--
extra : rebase_source : 8c571faf246da2666c6ce1e14d4a701e17d88f89

toolkit/components/url-classifier/Classifier.h

@@ -84,8 +84,6 @@ public:
    */
   nsresult ApplyFullHashes(nsTArray<TableUpdate*>* aUpdates);
 
-  nsresult CacheCompletions(const CacheResultArray& aResults);
-  uint32_t GetHashKey(void) { return mHashKey; }
   /*
    * Get a bunch of extra prefixes to query for completion
    * and mask the real entry being requested

toolkit/components/url-classifier/LookupCache.h

@@ -82,7 +82,6 @@ public:
   // True as long as this lookup is complete and hasn't expired.
   bool mConfirmed;
 
-  // TODO : Is this necessary
   bool mProtocolV2;
 };
 

toolkit/components/url-classifier/nsIUrlClassifierDBService.idl

@@ -7,11 +7,8 @@
 %{C++
 #include "Entries.h"
 #include "LookupCache.h"
-class nsUrlClassifierLookupResult;
 %}
 [ptr] native ResultArray(nsTArray<mozilla::safebrowsing::LookupResult>);
-[ptr] native CacheCompletionArray(nsTArray<mozilla::safebrowsing::CacheResult>);
-[ptr] native PrefixArray(mozilla::safebrowsing::PrefixArray);
 
 interface nsIUrlClassifierHashCompleter;
 interface nsIPrincipal;

toolkit/components/url-classifier/nsUrlClassifierDBService.h

@@ -146,10 +146,6 @@ private:
   // Close db connection and join the background thread if it exists.
   nsresult Shutdown();
 
-  // Check if the key is on a known-clean host.
-  nsresult CheckClean(const nsACString &lookupKey,
-                      bool *clean);
-
   nsresult ReadTablesFromPrefs();
 
   RefPtr<nsUrlClassifierDBServiceWorker> mWorker;