The cross-origin checks in the multibuffer code are not sufficient, as they only trigger when a red…

media/blink/resource_multibuffer_data_provider_unittest.cc

Index: media/blink/resource_multibuffer_data_provider_unittest.cc
diff --git a/media/blink/resource_multibuffer_data_provider_unittest.cc b/media/blink/resource_multibuffer_data_provider_unittest.cc
index 1055a2b0fcf834eb8b94117e8eaea3285cca4eea..413b1b6e543b6e34cf1e27dc4956de386437b9ed 100644
--- a/media/blink/resource_multibuffer_data_provider_unittest.cc
+++ b/media/blink/resource_multibuffer_data_provider_unittest.cc
@@ -254,7 +254,6 @@ TEST_F(ResourceMultiBufferDataProviderTest, BadHttpResponse) {
   response.setHTTPStatusCode(404);
   response.setHTTPStatusText("Not Found\n");
   loader_->didReceiveResponse(url_loader_, response);
-  StopWhenLoad();
 }
 
 // Tests that partial content is requested but not fulfilled.
@@ -262,7 +261,6 @@ TEST_F(ResourceMultiBufferDataProviderTest, NotPartialResponse) {
   Initialize(kHttpUrl, 100);
   Start();
   FullResponse(1024, false);
-  StopWhenLoad();
 }
 
 // Tests that a 200 response is received.
@@ -319,7 +317,6 @@ TEST_F(ResourceMultiBufferDataProviderTest, InvalidPartialResponse) {
   response.setExpectedContentLength(10);
   response.setHTTPStatusCode(kHttpPartialContent);
   loader_->didReceiveResponse(url_loader_, response);
-  StopWhenLoad();
 }
 
 TEST_F(ResourceMultiBufferDataProviderTest, TestRedirects) {