diff options
| author | tommycli@chromium.org <tommycli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-01 05:08:59 +0000 |
|---|---|---|
| committer | tommycli@chromium.org <tommycli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-01 05:08:59 +0000 |
| commit | e4843d3d1a0a249a8f7299faae307ffc4d29add6 (patch) | |
| tree | 29d0b1a4a610ebb1af716143f030129e38973fc2 /content/browser/child_process_security_policy_impl.cc | |
| parent | b18e06b22fe5721e4f1bfe4d0b62b6f2fdf18380 (diff) | |
| download | chromium_src-e4843d3d1a0a249a8f7299faae307ffc4d29add6.zip chromium_src-e4843d3d1a0a249a8f7299faae307ffc4d29add6.tar.gz chromium_src-e4843d3d1a0a249a8f7299faae307ffc4d29add6.tar.bz2 | |
ChildProcessSecurityPolicy: Remove CanWriteFile and CanCreateFile.
We currently have some checks that are more fine-grained than the grants. There's no reason for this, so I'm removing them and replacing them with the broader check.
I would also like to remove some of the Can*FileSystemFile checks, but I currently cannot, as that goes into checking the permissions for the filesystem as a whole in the case of isolated file systems.
That will have to be saved for a different CL.
This CL depends on:
https://codereview.chromium.org/24406003/
This is part of Step 2 of this refactoring plan:
https://docs.google.com/a/google.com/document/d/1QGkGWuwgSuaRqovz4wyb0upqPKDVsgYOFKt44E7gmOE/edit?usp=sharing
BUG=262142, 263150
Review URL: https://codereview.chromium.org/24439002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226148 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/child_process_security_policy_impl.cc')
| -rw-r--r-- | content/browser/child_process_security_policy_impl.cc | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/content/browser/child_process_security_policy_impl.cc b/content/browser/child_process_security_policy_impl.cc index b0a2b6e..c6f9757 100644 --- a/content/browser/child_process_security_policy_impl.cc +++ b/content/browser/child_process_security_policy_impl.cc @@ -614,16 +614,6 @@ bool ChildProcessSecurityPolicyImpl::CanReadFile(int child_id, return HasPermissionsForFile(child_id, file, kReadFilePermissions); } -bool ChildProcessSecurityPolicyImpl::CanWriteFile(int child_id, - const base::FilePath& file) { - return HasPermissionsForFile(child_id, file, kWriteFilePermissions); -} - -bool ChildProcessSecurityPolicyImpl::CanCreateFile(int child_id, - const base::FilePath& file) { - return HasPermissionsForFile(child_id, file, kCreateFilePermissions); -} - bool ChildProcessSecurityPolicyImpl::CanCreateReadWriteFile( int child_id, const base::FilePath& file) { |