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

Unified Diff: net/url_request/url_request.h

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
Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index d73d2fec6a34b05861710e347ab248404af8bb18..4f66cb24cdc9890d26f936d7f27075516dae8389 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -302,9 +302,14 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
// Profile.
static bool IsHandledURL(const GURL& url);
- // Allow access to file:// on ChromeOS for tests.
- static void AllowFileAccess();
- static bool IsFileAccessAllowed();
+ // Allow access to all file:// URLs (used for tests).
+ static void AllowAccessToAllFiles();
+
+ // Tests to see if access to |path| is allowed. If AllowAccessToAllFiles has
+ // been called, then this will return true. Otherwise it uses the list of
+ // restricted directories set by AllowFileAccessTo to decide. If neither
+ // has been called, will return false.
+ bool IsFileAccessAllowed(const FilePath& path);
willchan no longer on Chromium 2012/04/19 21:12:24 This is only used by the URLRequestFileJob, right?
Greg Spencer (Chromium) 2012/04/20 00:05:54 Done. I originally missed how to get at the netwo
// The original url is the url used to initialize the request, and it may
// differ from the url if the request was redirected.

Powered by Google App Engine
This is Rietveld 408576698