Backed out changeset 263b1b07c2ac (bug 1325651) for being a possible cause of valgrind leaks a=backout

dom/ipc/ContentParent.cpp

@@ -4793,7 +4793,6 @@ ContentParent::RecvPURLClassifierConstructor(PURLClassifierParent* aActor,
 {
   MOZ_ASSERT(NS_IsMainThread());
   MOZ_ASSERT(aActor);
-  *aSuccess = false;
 
   auto* actor = static_cast<URLClassifierParent*>(aActor);
   nsCOMPtr<nsIPrincipal> principal(aPrincipal);

dom/ipc/URLClassifierParent.cpp

@@ -18,7 +18,6 @@ URLClassifierParent::StartClassify(nsIPrincipal* aPrincipal,
                                    bool aUseTrackingProtection,
                                    bool* aSuccess)
 {
-  *aSuccess = false;
   nsresult rv = NS_OK;
   // Note that in safe mode, the URL classifier service isn't available, so we
   // should handle the service not being present gracefully.
@@ -35,7 +34,6 @@ URLClassifierParent::StartClassify(nsIPrincipal* aPrincipal,
     // without ever calling out callback in both cases.
     // This means that code using this in the child process will only get a hit
     // on its callback if some classification actually happens.
-    *aSuccess = false;
     ClassificationFailed();
   }
   return IPC_OK();