Bug 1373109 - Don't call MaybeCloseWindow twice when diverting. r=mconley

nsExternalHelperAppService::OnStartRequest calls MaybeCloseWindow
unconditionally after it finishes looking at the request headers. The call
from DidDivertRequest is redundant (and possibly left over from when we didn't
call OnStartRequest properly).

MozReview-Commit-ID: Cm5YhCt5XLY

--HG--
extra : rebase_source : 628100c860459d338cbabc65f557d7058b91f26d

uriloader/exthandler/nsExternalHelperAppService.cpp

@@ -1287,7 +1287,6 @@ nsExternalAppHandler::DidDivertRequest(nsIRequest *request)
   MOZ_ASSERT(XRE_IsContentProcess(), "in child process");
   // Remove our request from the child loadGroup
   RetargetLoadNotifications(request);
-  MaybeCloseWindow();
 }
 
 NS_IMETHODIMP nsExternalAppHandler::SetWebProgressListener(nsIWebProgressListener2 * aWebProgressListener)