Fix progress reporting for multibuffers to report progress for every byte we receive.

media/blink/multibuffer_data_source_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 921 matching lines @@
 }
 
 TEST_F(MultibufferDataSourceTest, Http_Read) {
   InitializeWith206Response();
 
   EXPECT_CALL(*this, ReadCallback(kDataSize));
   ReadAt(0, kDataSize * 2);
 
   ReadAt(kDataSize, kDataSize);
   EXPECT_CALL(*this, ReadCallback(kDataSize));
+  EXPECT_CALL(host_,
+              AddBufferedByteRange(kDataSize, kDataSize + kDataSize / 2));
+  ReceiveData(kDataSize / 2);
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
   ReceiveData(kDataSize / 2);
-  ReceiveData(kDataSize / 2);
 
   EXPECT_TRUE(data_source_->downloading());
   Stop();
 }
 
 TEST_F(MultibufferDataSourceTest, Http_ShareData) {
   InitializeWith206Response();
 
   EXPECT_CALL(*this, ReadCallback(kDataSize));
   ReadAt(0, kDataSize * 2);
 
   ReadAt(kDataSize, kDataSize);
   EXPECT_CALL(*this, ReadCallback(kDataSize));
+  EXPECT_CALL(host_,
+              AddBufferedByteRange(kDataSize, kDataSize + kDataSize / 2));
+  ReceiveData(kDataSize / 2);
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
   ReceiveData(kDataSize / 2);
-  ReceiveData(kDataSize / 2);
 
   EXPECT_TRUE(data_source_->downloading());
 
   StrictMock<MockBufferedDataSourceHost> host2;
   MockMultibufferDataSource source2(
       GURL(kHttpUrl), message_loop_.task_runner(), url_index_,
       view_->mainFrame()->toWebLocalFrame(), &host2);
   source2.SetPreload(preload_);
 
   EXPECT_CALL(*this, OnInitialize(true));
@@ skipping 301 matching lines @@
 
   EXPECT_CALL(host_, SetTotalBytes(response_generator_->content_length()));
   Respond(response_generator_->Generate206(0));
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize));
   ReceiveData(kDataSize);
 
   // Read a bit from the beginning.
   EXPECT_CALL(*this, ReadCallback(kDataSize));
   ReadAt(0);
 
+  EXPECT_CALL(host_, AddBufferedByteRange(kDataSize, kDataSize + 1));
   ReceiveData(1);
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 3));
   FinishLoading();
   EXPECT_CALL(*this, ReadCallback(0));
 
   ReadAt(kDataSize + 5, kDataSize * 2);
   Stop();
 }
 
 TEST_F(MultibufferDataSourceTest, Http_RetryThenRedirect) {
@@ skipping 68 matching lines @@
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
   FinishLoading();
 
   // Done loading, now we should know the length.
   EXPECT_TRUE(data_source_->GetSize(&len));
   EXPECT_EQ(kDataSize, len);
   Stop();
 }
 
 }  // namespace media