Bug 1398659. P8 - remove the call to mCacheStream.NotifyDataLength() from OpenChannel(). draft
authorJW Wang <jwwang@mozilla.com>
Thu, 07 Sep 2017 15:23:57 +0800
changeset 662184 df0a1b0ecde95befcf0fcdbf30f99fd4220e65d7
parent 662183 88463f412b0cb9b4a7df50e38692c9a62f69898e
child 662185 45e3b84d1a945713f5eecb02c3a8b301c55a5eb4
push id78979
push userjwwang@mozilla.com
push dateMon, 11 Sep 2017 05:57:05 +0000
bugs1398659
milestone57.0a1
Bug 1398659. P8 - remove the call to mCacheStream.NotifyDataLength() from OpenChannel(). MediaCacheStream::mStreamLength has been set either in Init() or InitAsClone(). MozReview-Commit-ID: L259ecDgjN7
dom/media/MediaResource.cpp
--- a/dom/media/MediaResource.cpp
+++ b/dom/media/MediaResource.cpp
@@ -544,28 +544,16 @@ ChannelMediaResource::Open(nsIStreamList
 
 nsresult
 ChannelMediaResource::OpenChannel()
 {
   MOZ_ASSERT(NS_IsMainThread());
   MOZ_ASSERT(mChannel);
   MOZ_ASSERT(!mListener, "Listener should have been removed by now");
 
-  // Set the content length, if it's available as an HTTP header.
-  // This ensures that MediaResource wrapping objects for platform libraries
-  // that expect to know the length of a resource can get it before
-  // OnStartRequest() fires.
-  nsCOMPtr<nsIHttpChannel> hc = do_QueryInterface(mChannel);
-  if (hc && !IsPayloadCompressed(hc)) {
-    int64_t cl = -1;
-    if (NS_SUCCEEDED(hc->GetContentLength(&cl)) && cl != -1) {
-      mCacheStream.NotifyDataLength(cl);
-    }
-  }
-
   mListener = new Listener(this);
   nsresult rv = mChannel->SetNotificationCallbacks(mListener.get());
   NS_ENSURE_SUCCESS(rv, rv);
 
   rv = SetupChannelHeaders();
   NS_ENSURE_SUCCESS(rv, rv);
 
   rv = mChannel->AsyncOpen2(mListener);