Bug 1139225 - Followup - Remove duplicate multiPartChannel variable.
authorSeth Fowler <seth@mozilla.com>
Tue, 24 Mar 2015 14:47:01 -0700
changeset 235395 8ae8156e09569481842093c54eb4371925c0dd62
parent 235394 8f5b359fc9fcacb564a043e5dcef5ffb88a379d7
child 235396 129094fb7ce9dad99a22a096fd104a1958300e79
push id57417
push usermfowler@mozilla.com
push dateTue, 24 Mar 2015 21:47:15 +0000
treeherdermozilla-inbound@8ae8156e0956 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1139225
milestone39.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1139225 - Followup - Remove duplicate multiPartChannel variable.
image/src/imgRequest.cpp
--- a/image/src/imgRequest.cpp
+++ b/image/src/imgRequest.cpp
@@ -693,17 +693,16 @@ NS_IMETHODIMP imgRequest::OnStartRequest
   /*
    * If mRequest is null here, then we need to set it so that we'll be able to
    * cancel it if our Cancel() method is called.  Note that this can only
    * happen for multipart channels.  We could simply not null out mRequest for
    * non-last parts, if GetIsLastPart() were reliable, but it's not.  See
    * https://bugzilla.mozilla.org/show_bug.cgi?id=339610
    */
   if (!mRequest) {
-    nsCOMPtr<nsIMultiPartChannel> multiPartChannel = do_QueryInterface(aRequest);
     MOZ_ASSERT(multiPartChannel, "Should have mRequest unless we're multipart");
     nsCOMPtr<nsIChannel> baseChannel;
     multiPartChannel->GetBaseChannel(getter_AddRefs(baseChannel));
     mRequest = baseChannel;
   }
 
   nsCOMPtr<nsIChannel> channel(do_QueryInterface(aRequest));
   if (channel) {