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

media/blink/multibuffer_data_source_unittest.cc

Index: media/blink/multibuffer_data_source_unittest.cc
diff --git a/media/blink/multibuffer_data_source_unittest.cc b/media/blink/multibuffer_data_source_unittest.cc
index 85794b75ea5e43e716bb2c549258dc451436d8c4..f265046163ba688773acbf7e17bc19254084233f 100644
--- a/media/blink/multibuffer_data_source_unittest.cc
+++ b/media/blink/multibuffer_data_source_unittest.cc
@@ -973,8 +973,10 @@ TEST_F(MultibufferDataSourceTest, Http_Read) {
 
   ReadAt(kDataSize, kDataSize);
   EXPECT_CALL(*this, ReadCallback(kDataSize));
-  EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
+  EXPECT_CALL(host_,
+              AddBufferedByteRange(kDataSize, kDataSize + kDataSize / 2));
   ReceiveData(kDataSize / 2);
+  EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
   ReceiveData(kDataSize / 2);

[wolenetz, 2016/05/12 23:41:17]

I'm unfamiliar with this code. Naively, it looks like we've simulated the
fetching of just kDataSize across these two ReceiveData() calls. Why then do we
appear to expect to have buffered more than just kDataSize?

[hubbe, 2016/05/13 00:05:14]

InitializeWith206Response() also calls receivedata, so we've already received
one block.

 
   EXPECT_TRUE(data_source_->downloading());
@@ -989,8 +991,10 @@ TEST_F(MultibufferDataSourceTest, Http_ShareData) {
 
   ReadAt(kDataSize, kDataSize);
   EXPECT_CALL(*this, ReadCallback(kDataSize));
-  EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
+  EXPECT_CALL(host_,
+              AddBufferedByteRange(kDataSize, kDataSize + kDataSize / 2));
   ReceiveData(kDataSize / 2);
+  EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 2));
   ReceiveData(kDataSize / 2);
 
   EXPECT_TRUE(data_source_->downloading());
@@ -1313,6 +1317,7 @@ TEST_F(MultibufferDataSourceTest, SeekPastEOF) {
   EXPECT_CALL(*this, ReadCallback(kDataSize));
   ReadAt(0);
 
+  EXPECT_CALL(host_, AddBufferedByteRange(kDataSize, kDataSize + 1));
   ReceiveData(1);
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize * 3));
   FinishLoading();