Bug 1310127 - Part 7: Use MOZ_MUST_USE in netwerk/protocol/http r?jst draft
authorWei-Cheng Pan <wpan@mozilla.com>
Tue, 20 Dec 2016 15:02:46 +0800
changeset 494454 3f7549a732127d038f07e044489160a3f290b2e9
parent 494453 bdcddd90761c7bcef11e1801dccc138f52dede99
child 494455 b0c1cf913ccc1721b0f4b6ec8b1def308f217817
push id48029
push userbmo:wpan@mozilla.com
push dateTue, 07 Mar 2017 03:35:29 +0000
reviewersjst
bugs1310127
milestone54.0a1
Bug 1310127 - Part 7: Use MOZ_MUST_USE in netwerk/protocol/http r?jst MozReview-Commit-ID: NpX8rph1Ke
dom/webbrowserpersist/nsWebBrowserPersist.cpp
--- a/dom/webbrowserpersist/nsWebBrowserPersist.cpp
+++ b/dom/webbrowserpersist/nsWebBrowserPersist.cpp
@@ -1381,28 +1381,31 @@ nsresult nsWebBrowserPersist::SaveURIInt
             encodedChannel->SetApplyConversion(false);
         }
     }
 
     if (mPersistFlags & PERSIST_FLAGS_FORCE_ALLOW_COOKIES)
     {
         nsCOMPtr<nsIHttpChannelInternal> httpChannelInternal =
                 do_QueryInterface(inputChannel);
-        if (httpChannelInternal)
-            httpChannelInternal->SetThirdPartyFlags(nsIHttpChannelInternal::THIRD_PARTY_FORCE_ALLOW);
+        if (httpChannelInternal) {
+            rv = httpChannelInternal->SetThirdPartyFlags(nsIHttpChannelInternal::THIRD_PARTY_FORCE_ALLOW);
+            MOZ_ASSERT(NS_SUCCEEDED(rv));
+        }
     }
 
     // Set the referrer, post data and headers if any
     nsCOMPtr<nsIHttpChannel> httpChannel(do_QueryInterface(inputChannel));
     if (httpChannel)
     {
         // Referrer
         if (aReferrer)
         {
-            httpChannel->SetReferrerWithPolicy(aReferrer, aReferrerPolicy);
+            rv = httpChannel->SetReferrerWithPolicy(aReferrer, aReferrerPolicy);
+            MOZ_ASSERT(NS_SUCCEEDED(rv));
         }
 
         // Post data
         if (aPostData)
         {
             nsCOMPtr<nsISeekableStream> stream(do_QueryInterface(aPostData));
             if (stream)
             {