Fixed playing video in component extensions.

net/url_request/url_range_request_job.h

+// Copyright (c) 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.
+
+#ifndef NET_URL_REQUEST_URL_RANGE_REQUEST_JOB_H_
+#define NET_URL_REQUEST_URL_RANGE_REQUEST_JOB_H_
+
+#include "net/base/net_export.h"
+#include "net/http/http_byte_range.h"
+#include "net/url_request/url_request_job.h"
+
+namespace net {
+
+class HttpRequestHeaders;
+
+class NET_EXPORT URLRangeRequestJob : public URLRequestJob {

[mmenke, 2014/01/07 18:51:39]

Should have a comment here.

I suggest something like:

"URLRequestJob with support for parsing range requests.  It is up to subclasses
to handle the response, and pass any errors parsing the ranges after Start() has
been called."

[kirr, 2014/01/09 07:33:21]

Done.

+ public:
+  URLRangeRequestJob(URLRequest* request,
+                     NetworkDelegate* delegate);
+
+  virtual void SetExtraRequestHeaders(
+      const HttpRequestHeaders& headers) OVERRIDE;
+
+  const std::vector<HttpByteRange>& ranges() const { return ranges_; }
+  int range_parse_error() const { return range_parse_error_; }

[mmenke, 2014/01/07 18:51:39]

I suggest range_parse_result_ instead, since "OK" isn't exactly an error.

Should also comment that this is "OK" if no range has been specified.

[kirr, 2014/01/09 07:33:21]

Done.

+
+ protected:
+  virtual ~URLRangeRequestJob();
+
+ private:
+  std::vector<HttpByteRange> ranges_;
+  int range_parse_error_;
+};
+
+}  // namespace net
+
+#endif  // NET_URL_REQUEST_URL_RANGE_REQUEST_JOB_H_