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

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

Issue 2786583002: chromeos: Check both original and absolute paths for file: scheme (Closed)
Patch Set: disable checks in sub classes Created 3 years, 8 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
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/url_request/url_request_context_builder.h" 5 #include "net/url_request/url_request_context_builder.h"
6 6
7 #include <string> 7 #include <string>
8 #include <utility> 8 #include <utility>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 return true; 117 return true;
118 } 118 }
119 119
120 bool OnCanSetCookie(const URLRequest& request, 120 bool OnCanSetCookie(const URLRequest& request,
121 const std::string& cookie_line, 121 const std::string& cookie_line,
122 CookieOptions* options) override { 122 CookieOptions* options) override {
123 return true; 123 return true;
124 } 124 }
125 125
126 bool OnCanAccessFile(const URLRequest& request, 126 bool OnCanAccessFile(const URLRequest& request,
127 const base::FilePath& path) const override { 127 const base::FilePath& original_path,
128 const base::FilePath& absolute_path) const override {
128 return true; 129 return true;
129 } 130 }
130 131
131 DISALLOW_COPY_AND_ASSIGN(BasicNetworkDelegate); 132 DISALLOW_COPY_AND_ASSIGN(BasicNetworkDelegate);
132 }; 133 };
133 134
134 // Define a context class that can self-manage the ownership of its components 135 // Define a context class that can self-manage the ownership of its components
135 // via a UrlRequestContextStorage object. 136 // via a UrlRequestContextStorage object.
136 class ContainerURLRequestContext : public URLRequestContext { 137 class ContainerURLRequestContext : public URLRequestContext {
137 public: 138 public:
(...skipping 382 matching lines...) Expand 10 before | Expand all | Expand 10 after
520 } 521 }
521 url_request_interceptors_.clear(); 522 url_request_interceptors_.clear();
522 } 523 }
523 storage->set_job_factory(std::move(top_job_factory)); 524 storage->set_job_factory(std::move(top_job_factory));
524 // TODO(willchan): Support sdch. 525 // TODO(willchan): Support sdch.
525 526
526 return std::move(context); 527 return std::move(context);
527 } 528 }
528 529
529 } // namespace net 530 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698