Bug 1563313 - AsyncApplyFilters must set mProxyInfo only when if it has not already been canceled, r=mayhemer

Otherwise, in the CTOR, we crash becaues of MOZ_ASSERT(!mProxyInfo);

Differential Revision: https://phabricator.services.mozilla.com/D37162

--HG--
extra : moz-landing-system : lando

netwerk/base/nsProtocolProxyService.cpp

@@ -533,7 +533,6 @@ nsAsyncResolveRequest::AsyncApplyFilters::OnProxyFilterResult(
   }
 
   mFilterCalledBack = true;
-  mProxyInfo = aProxyInfo;
 
   if (mProcessingInLoop) {
     // No need to call/dispatch ProcessNextFilter(), we are in a control
@@ -548,6 +547,8 @@ nsAsyncResolveRequest::AsyncApplyFilters::OnProxyFilterResult(
     return NS_OK;
   }
 
+  mProxyInfo = aProxyInfo;
+
   if (mNextFilterIndex == mFiltersCopy.Length()) {
     // We are done, all filters have been called on!
     Finish();