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

Unified Diff: content/browser/loader/resource_dispatcher_host_impl.cc

Issue 23760004: ChildProcessSecurityPolicy: Port FileAPIMessageFilter to use new checks (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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: content/browser/loader/resource_dispatcher_host_impl.cc
diff --git a/content/browser/loader/resource_dispatcher_host_impl.cc b/content/browser/loader/resource_dispatcher_host_impl.cc
index 2a4ad9ac684eaf013e8717688027dc894c13aa1b..dcd2d12bc3b04e8c1a5d0fb3d4e1083816ec3181 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.cc
+++ b/content/browser/loader/resource_dispatcher_host_impl.cc
@@ -217,8 +217,7 @@ bool ShouldServiceRequest(int process_type,
}
if (iter->type() == ResourceRequestBody::Element::TYPE_FILE_FILESYSTEM) {
fileapi::FileSystemURL url = file_system_context->CrackURL(iter->url());
- if (!policy->HasPermissionsForFileSystemFile(
- child_id, url, fileapi::kReadFilePermissions)) {
+ if (!policy->CanReadFileSystemFile(child_id, url)) {
NOTREACHED() << "Denied unauthorized upload of "
<< iter->url().spec();
return false;

Powered by Google App Engine
This is Rietveld 408576698