BufferedDataSource now caches redirects for subsequent requests

media/blink/buffered_data_source.cc

Index: media/blink/buffered_data_source.cc
diff --git a/media/blink/buffered_data_source.cc b/media/blink/buffered_data_source.cc
index 7b15b11c88b4c902ae687c0c5303262a85939852..c149c8ce5b99f3bd79c052fb61dd1ca813bb96ca 100644
--- a/media/blink/buffered_data_source.cc
+++ b/media/blink/buffered_data_source.cc
@@ -387,7 +387,15 @@ void BufferedDataSource::StartCallback(
     loader_->Stop();
     return;
   }
+
   response_original_url_ = loader_->response_original_url();
+  // The response original url is the URL of this resource after following
+  // redirects. Update |url_| to this so that we only follow redirects once for
+  // simplicity and efficiency.
+  // If it's empty, it was a response from a Service Worker, so keep the

[hubbe, 2016/04/28 19:34:17]

Why do we care if it's a service worker?

[watk, 2016/04/28 20:05:16]

I guess my phrasing is slightly misleading. It's not that it was a service
worker that's important, but that it's empty. The only reason I mentioned
Service Workers is because otherwise it's not obvious why we'd check for an
empty url.

Should I reword it?

+  // original URL.
+  if (!response_original_url_.is_empty())
+    url_ = response_original_url_;
 
   // All responses must be successful. Resources that are assumed to be fully
   // buffered must have a known content length.