fix a semi-rare crash in multibuffers

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 509 matching lines @@
 }
 
 TEST_F(MultibufferDataSourceTest, Range_NotSupported) {
   InitializeWith200Response();
 
   EXPECT_TRUE(loading());
   EXPECT_TRUE(data_source_->IsStreaming());
   Stop();
 }
 
+TEST_F(MultibufferDataSourceTest, Range_NotSatisfiable) {
+  Initialize(kHttpUrl, true);
+  EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize));
+  Respond(response_generator_->Generate404(416));
+  EXPECT_FALSE(loading());
+  Stop();
+}
+
 // Special carve-out for Apache versions that choose to return a 200 for
 // Range:0- ("because it's more efficient" than a 206)
 TEST_F(MultibufferDataSourceTest, Range_SupportedButReturned200) {
   Initialize(kHttpUrl, true);
   EXPECT_CALL(host_, SetTotalBytes(response_generator_->content_length()));
   WebURLResponse response = response_generator_->Generate200();
   response.setHTTPHeaderField(WebString::fromUTF8("Accept-Ranges"),
                               WebString::fromUTF8("bytes"));
   Respond(response);
 
@@ skipping 829 matching lines @@
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize));
   ReceiveData(kDataSize);
 
   EXPECT_FALSE(data_source_->IsStreaming());
 
   FinishLoading();
   EXPECT_FALSE(loading());
   Stop();
 }
 
+TEST_F(MultibufferDataSourceTest, Http_RangeNotSatisfiableAfterRedirect) {
+  Initialize(kHttpUrl, true);
+
+  // Server responds with a redirect.
+  blink::WebURLRequest request((GURL(kHttpDifferentPathUrl)));
+  blink::WebURLResponse response((GURL(kHttpUrl)));
+  response.setHTTPStatusCode(307);
+  data_provider()->willFollowRedirect(url_loader(), request, response);
+
+  EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize));
+  Respond(response_generator_->Generate404(416));
+  Stop();
+}
+
 TEST_F(MultibufferDataSourceTest, LengthKnownAtEOF) {
   Initialize(kHttpUrl, true);
   // Server responds without content-length.
   WebURLResponse response = response_generator_->Generate200();
   response.clearHTTPHeaderField(WebString::fromUTF8("Content-Length"));
   response.setExpectedContentLength(kPositionNotSpecified);
   Respond(response);
   EXPECT_CALL(host_, AddBufferedByteRange(0, kDataSize));
   ReceiveData(kDataSize);
   int64_t len;
   EXPECT_FALSE(data_source_->GetSize(&len));
   EXPECT_TRUE(data_source_->IsStreaming());
   EXPECT_CALL(*this, ReadCallback(kDataSize));
   ReadAt(0);
 
   ReadAt(kDataSize);
   EXPECT_CALL(host_, SetTotalBytes(kDataSize));
   EXPECT_CALL(*this, ReadCallback(0));
   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