From bc9d9e9e1cd4f9f080d5199f2b87fddc9de8287a Mon Sep 17 00:00:00 2001 From: "sdwilsh%shawnwilsher.com" Date: Tue, 12 Jun 2007 18:31:54 +0000 Subject: [PATCH] Bug 383807 - Explain why we are not checking the result of new in nsExternalProtocolHander.cpp. Patch by Ryan Jones . r=cbiesinger, sr=dmose git-svn-id: svn://10.0.0.236/trunk@227885 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/uriloader/exthandler/nsExternalProtocolHandler.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/mozilla/uriloader/exthandler/nsExternalProtocolHandler.cpp b/mozilla/uriloader/exthandler/nsExternalProtocolHandler.cpp index a1f695ac5f3..292946e302e 100644 --- a/mozilla/uriloader/exthandler/nsExternalProtocolHandler.cpp +++ b/mozilla/uriloader/exthandler/nsExternalProtocolHandler.cpp @@ -325,6 +325,8 @@ NS_IMETHODIMP nsExtProtocolChannel::AsyncOpen(nsIStreamListener *listener, nsISu nsCOMPtr event = new nsWebProtocolRedirect(mUrl, uriTemplate, listener, ctxt, this); + // We don't check if |event| was successfully created because + // |NS_DispatchToCurrentThread| will do that for us. rv = NS_DispatchToCurrentThread(event); if (NS_SUCCEEDED(rv)) { return rv;