Use range request for CORS access check on Android.

content/renderer/media/android/media_info_loader.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 "content/renderer/media/android/media_info_loader.h"
 
 #include "base/bits.h"
 #include "base/callback_helpers.h"
 #include "base/metrics/histogram.h"
 #include "third_party/WebKit/public/platform/WebURLError.h"
@@ skipping 32 matching lines @@
 
   start_time_ = base::TimeTicks::Now();
   first_party_url_ = frame->document().firstPartyForCookies();
 
   // Prepare the request.
   WebURLRequest request(url_);
   // TODO(mkwst): Split this into video/audio.
   request.setRequestContext(WebURLRequest::RequestContextVideo);
   frame->setReferrerForRequest(request, blink::WebURL());
 
+  // Since we don't actually care about the media data at this time, use a one
+  // byte range request to avoid unnecessarily downloading resources.  Not all
+  // servers support HEAD unfortunately, so use a range request; which is no
+  // worse than the previous request+cancel code.  See http://crbug.com/400788
+  request.addHTTPHeaderField("Range", "bytes=0-1");

[acolwell GONE FROM CHROMIUM, 2014/08/25 19:40:08]

nit: This is a 2 byte range request. Please update the comment.

[DaleCurtis, 2014/08/25 19:43:18]

Done.

+
   scoped_ptr<WebURLLoader> loader;
   if (test_loader_) {
     loader = test_loader_.Pass();
   } else {
     WebURLLoaderOptions options;
     if (cors_mode_ == blink::WebMediaPlayer::CORSModeUnspecified) {
       options.allowCredentials = true;
       options.crossOriginRequestPolicy =
           WebURLLoaderOptions::CrossOriginRequestPolicyAllow;
       allow_stored_credentials_ = true;
@@ skipping 53 matching lines @@
            << (response.httpVersion() == WebURLResponse::HTTP_0_9 ? "0.9" :
                response.httpVersion() == WebURLResponse::HTTP_1_0 ? "1.0" :
                response.httpVersion() == WebURLResponse::HTTP_1_1 ? "1.1" :
                "Unknown")
            << " " << response.httpStatusCode();
   DCHECK(active_loader_.get());
   if (!url_.SchemeIs("http") && !url_.SchemeIs("https")) {
       DidBecomeReady(kOk);
       return;
   }
   if (response.httpStatusCode() == kHttpOK) {

[acolwell GONE FROM CHROMIUM, 2014/08/25 19:40:08]

Range requests result in a 206 response so you'll need to update the code to
handle that.

[DaleCurtis, 2014/08/25 19:43:19]

Done.

     DidBecomeReady(kOk);
     return;
   }
   loader_failed_ = true;
   DidBecomeReady(kFailed);
 }
 
 void MediaInfoLoader::didReceiveData(
     WebURLLoader* loader,
     const char* data,
@@ skipping 56 matching lines @@
 void MediaInfoLoader::DidBecomeReady(Status status) {
   UMA_HISTOGRAM_TIMES("Media.InfoLoadDelay",
                       base::TimeTicks::Now() - start_time_);
   active_loader_.reset();
   if (!ready_cb_.is_null())
     base::ResetAndReturn(&ready_cb_).Run(status, url_, first_party_url_,
                                          allow_stored_credentials_);
 }
 
 }  // namespace content