diff options
| author | skerner@chromium.org <skerner@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-01 20:24:09 +0000 |
|---|---|---|
| committer | skerner@chromium.org <skerner@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-01 20:24:09 +0000 |
| commit | 4cd2d520d95db38a157fd2f86161ea457eab7af2 (patch) | |
| tree | 5e9f69b9eb22b9e4db9853080a3b5c6500b5922a /base/file_util.h | |
| parent | 8bc2f4747ecc9ac8522e109440dba9a0e1fabff3 (diff) | |
| download | chromium_src-4cd2d520d95db38a157fd2f86161ea457eab7af2.zip chromium_src-4cd2d520d95db38a157fd2f86161ea457eab7af2.tar.gz chromium_src-4cd2d520d95db38a157fd2f86161ea457eab7af2.tar.bz2 | |
Loosen permission on extension temp dir when a flag is used.
Issue 35198 can not be reproduced locally. To enable users to do experiments, three command line flags are added to chrome:
--issue35198-crxdir-browser: Have the browser process create the directory in which the extension will be unzipped.
--issue35198-logging: Enable log messages from directory creation in the utility process to be moved to the browser process.
--issue35198-permission: Use the most permissive file permissions possible on the extension unpack directory.
BUG=35198
TEST=manual
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51231
Review URL: http://codereview.chromium.org/2802018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51416 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/file_util.h')
| -rw-r--r-- | base/file_util.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/base/file_util.h b/base/file_util.h index 48f431f..6277748 100644 --- a/base/file_util.h +++ b/base/file_util.h @@ -265,8 +265,13 @@ bool CreateTemporaryFileInDir(const FilePath& dir, // Create a directory within another directory. // Extra characters will be appended to |name_tmpl| to ensure that the // new directory does not have the same name as an existing directory. +// If |loosen_permissions| is true, the new directory will be readable +// and writable to all users on windows. It is ignored on other platforms. +// |loosen_permissions| exists to allow debugging of crbug/35198, and will +// be removed when the issue is understood. bool CreateTemporaryDirInDir(const FilePath& base_dir, const FilePath::StringType& prefix, + bool loosen_permissions, FilePath* new_dir); // Create a new directory under TempPath. If prefix is provided, the new |