Bug 1253054 - Stop warning if request has already failed. r=bkelly
authorEric Rahm <erahm@mozilla.com>
Fri, 11 Mar 2016 11:18:07 -0800
changeset 339600 ce018c5ee5c15e76c640f0c0e2414ab7c6a04657
parent 339599 3f3abdbf2d06fef628c8ddc71624de3acb801c23
child 339601 da961dd18d914695ebff4f35eb19d36e3fa665fa
push id12762
push userbmo:rail@mozilla.com
push dateFri, 11 Mar 2016 19:47:45 +0000
reviewersbkelly
bugs1253054
milestone48.0a1
Bug 1253054 - Stop warning if request has already failed. r=bkelly
dom/fetch/FetchDriver.cpp
--- a/dom/fetch/FetchDriver.cpp
+++ b/dom/fetch/FetchDriver.cpp
@@ -476,17 +476,17 @@ FetchDriver::OnStartRequest(nsIRequest* 
   workers::AssertIsOnMainThread();
 
   // Note, this can be called multiple times if we are doing an opaqueredirect.
   // In that case we will get a simulated OnStartRequest() and then the real
   // channel will call in with an errored OnStartRequest().
 
   nsresult rv;
   aRequest->GetStatus(&rv);
-  if (NS_WARN_IF(NS_FAILED(rv))) {
+  if (NS_FAILED(rv)) {
     FailWithNetworkError();
     return rv;
   }
 
   // We should only get to the following code once.
   MOZ_ASSERT(!mPipeOutputStream);
   MOZ_ASSERT(mObserver);