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

Side by Side Diff: net/proxy/proxy_script_fetcher_impl_unittest.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/proxy/proxy_script_fetcher_impl.h" 5 #include "net/proxy/proxy_script_fetcher_impl.h"
6 6
7 #include <string> 7 #include <string>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 return true; 187 return true;
188 } 188 }
189 189
190 bool OnCanSetCookie(const URLRequest& request, 190 bool OnCanSetCookie(const URLRequest& request,
191 const std::string& cookie_line, 191 const std::string& cookie_line,
192 CookieOptions* options) override { 192 CookieOptions* options) override {
193 return true; 193 return true;
194 } 194 }
195 195
196 bool OnCanAccessFile(const URLRequest& request, 196 bool OnCanAccessFile(const URLRequest& request,
197 const base::FilePath& path) const override { 197 const base::FilePath& original_path,
198 const base::FilePath& absolute_path) const override {
198 return true; 199 return true;
199 } 200 }
200 201
201 DISALLOW_COPY_AND_ASSIGN(BasicNetworkDelegate); 202 DISALLOW_COPY_AND_ASSIGN(BasicNetworkDelegate);
202 }; 203 };
203 204
204 class ProxyScriptFetcherImplTest : public PlatformTest { 205 class ProxyScriptFetcherImplTest : public PlatformTest {
205 public: 206 public:
206 ProxyScriptFetcherImplTest() { 207 ProxyScriptFetcherImplTest() {
207 test_server_.AddDefaultHandlers(base::FilePath(kDocRoot)); 208 test_server_.AddDefaultHandlers(base::FilePath(kDocRoot));
(...skipping 284 matching lines...) Expand 10 before | Expand all | Expand 10 after
492 base::string16 text; 493 base::string16 text;
493 TestCompletionCallback callback; 494 TestCompletionCallback callback;
494 int result = pac_fetcher.Fetch(url, &text, callback.callback()); 495 int result = pac_fetcher.Fetch(url, &text, callback.callback());
495 EXPECT_THAT(result, IsError(ERR_FAILED)); 496 EXPECT_THAT(result, IsError(ERR_FAILED));
496 } 497 }
497 } 498 }
498 499
499 } // namespace 500 } // namespace
500 501
501 } // namespace net 502 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698