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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/base/network_delegate.cc

Issue 10068021: Fix file access on Chrome for ChromeOS on Linux (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed clang problem Created 8 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 side-by-side diff with in-line comments
Download patch
« net/base/network_delegate.h ('K') | « net/base/network_delegate.h ('k') | net/proxy/network_delegate_error_observer_unittest.cc » ('j') | net/url_request/url_request.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/network_delegate.cc
diff --git a/net/base/network_delegate.cc b/net/base/network_delegate.cc
index 6b857cf83100a414459101cc3915c189bf712ca2..ef6d3d719c96381d22c7b6755c8dc4872ad143ee 100644
--- a/net/base/network_delegate.cc
+++ b/net/base/network_delegate.cc
@@ -105,8 +105,16 @@ bool NetworkDelegate::NotifySettingCookie(
const std::string& cookie_line,
CookieOptions* options) {
DCHECK(CalledOnValidThread());
+ DCHECK(request);
DCHECK(!(request->load_flags() & net::LOAD_DO_NOT_SAVE_COOKIES));
return CanSetCookie(request, cookie_line, options);
}
+bool NetworkDelegate::NotifyFileAccessRequested(const URLRequest* request,
+ const FilePath& path){
willchan no longer on Chromium 2012/04/19 21:12:24 Missing a whitespace before the '{'
Greg Spencer (Chromium) 2012/04/20 00:05:54 Done.
+ DCHECK(CalledOnValidThread());
+ DCHECK(request);
+ return CanAccessFile(request, path);
+}
+
} // namespace net
« net/base/network_delegate.h ('K') | « net/base/network_delegate.h ('k') | net/proxy/network_delegate_error_observer_unittest.cc » ('j') | net/url_request/url_request.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698