Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(158)

Side by Side Diff: net/url_request/url_range_request_job.cc

Issue 104513002: Fixed playing video in component extensions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added URLRequestSimpleJobTest. More clear ignoring incorrect range headers. Created 6 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
mmenke 2014/01/07 18:51:39 nit: New files shouldn't have the "(c)"
kirr 2014/01/09 07:33:21 Done.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "net/url_request/url_range_request_job.h"
6
7 #include "net/base/net_errors.h"
8 #include "net/http/http_request_headers.h"
9 #include "net/http/http_util.h"
10
11 namespace net {
12
13 URLRangeRequestJob::URLRangeRequestJob(URLRequest* request,
14 NetworkDelegate* delegate)
15 : URLRequestJob(request, delegate), range_parse_error_(false) {
mmenke 2014/01/07 18:51:39 range_parse_error_ should be initialized to "OK".
kirr 2014/01/09 07:33:21 Done.
16 }
17
18 URLRangeRequestJob::~URLRangeRequestJob() {
19 }
20
21 void URLRangeRequestJob::SetExtraRequestHeaders(
22 const HttpRequestHeaders& headers) {
23 std::string range_header;
24 if (headers.GetHeader(HttpRequestHeaders::kRange, &range_header)) {
25 if (!HttpUtil::ParseRangeHeader(range_header, &ranges_)) {
26 range_parse_error_ = ERR_REQUEST_RANGE_NOT_SATISFIABLE;
27 }
28 }
29 }
30
31 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698