Bug 1278505 - Remove unneeded NS_NOTREACHED from DeallocPPrintProgressDialogChild(). r=mconley

MozReview-Commit-ID: 7WvZM390h8W

--HG--
extra : rebase_source : 94131635467e64dab7f30148183454065cdd2712

embedding/components/printingui/ipc/nsPrintingProxy.cpp

@@ -226,11 +226,9 @@ nsPrintingProxy::AllocPPrintProgressDialogChild()
 bool
 nsPrintingProxy::DeallocPPrintProgressDialogChild(PPrintProgressDialogChild* aActor)
 {
-  // The parent process will never initiate the PPrintProgressDialog
-  // protocol connection, so no need to provide an deallocator here.
-  NS_NOTREACHED("Deallocator for PPrintProgressDialogChild should not be "
-                "called on nsPrintingProxy.");
-  return false;
+  // The PrintProgressDialogChild implements refcounting, and
+  // will take itself out.
+  return true;
 }
 
 PPrintSettingsDialogChild*