Bug 1405496 - Remove unnecessary assertion. r=bagder, a=debug-only
authorLiang-Heng Chen <xeonchen@gmail.com>
Thu, 05 Oct 2017 10:59:11 +0800
changeset 434738 7b0cf325bfb44acd8abc3fb2c05c257b3fa00d76
parent 434737 f906462b31279a4ef232d2b938fcce4aef96ae9b
child 434739 47964993f1d0073d9ed23ea8d8f07e1e67bd0bc1
push id1567
push userjlorenzo@mozilla.com
push dateThu, 02 Nov 2017 12:36:05 +0000
treeherdermozilla-release@e512c14a0406 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbagder, debug-only
bugs1405496
milestone57.0
Bug 1405496 - Remove unnecessary assertion. r=bagder, a=debug-only The assertion does not have to be true. MozReview-Commit-ID: HrXpPsO0rQd
netwerk/base/nsProtocolProxyService.cpp
--- a/netwerk/base/nsProtocolProxyService.cpp
+++ b/netwerk/base/nsProtocolProxyService.cpp
@@ -361,17 +361,16 @@ public:
                                                              pacUri);
 
         return NS_DispatchToMainThread(event);
     }
 
 private:
     ~AsyncGetPACURIRequest()
     {
-        MOZ_ASSERT(NS_IsMainThread() == mIsMainThreadOnly);
         NS_ReleaseOnMainThreadSystemGroup(
           "AsyncGetPACURIRequest::mServiceHolder", mServiceHolder.forget());
     }
 
     bool mIsMainThreadOnly;
 
     nsProtocolProxyService* mService; // ref-count is hold by mServiceHolder
     nsCOMPtr<nsIProtocolProxyService2> mServiceHolder;