media: Allow suspend on HTTP 200 videos

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 583 matching lines @@
 
   EXPECT_FALSE(loading());
   Stop();
 }
 
 // Test the case where the initial response from the server indicates that
 // Range requests are supported, but a later request prove otherwise.
 TEST_F(MultibufferDataSourceTest, Range_ServerLied) {
   InitializeWith206Response();
 
-  // Read causing a new request to be made -- we'll expect it to error.
+  // Read causing a new request to be made, we will discard the data that
+  // was already read in the first request.
   ReadAt(kFarReadPosition);
 
   // Return a 200 in response to a range request.
   EXPECT_CALL(*this, ReadCallback(media::DataSource::kReadError));
   Respond(response_generator_->Generate200());
 
-  EXPECT_FALSE(loading());
+  EXPECT_TRUE(loading());
   Stop();
 }
 
 TEST_F(MultibufferDataSourceTest, Http_AbortWhileReading) {
   InitializeWith206Response();
 
   // Make sure there's a pending read -- we'll expect it to error.
   ReadAt(kFileSize);
 
   // Abort!!!
@@ skipping 999 matching lines @@
                      base::Bind(&MultibufferDataSourceTest::ReadCallback,
                                 base::Unretained(this)));
   base::RunLoop().RunUntilIdle();
 
   // Make sure we're not downloading anymore.
   EXPECT_EQ(data_source_->downloading(), false);
   Stop();
 }
 
 }  // namespace media