diff options
| author | mtomasz <mtomasz@chromium.org> | 2015-03-17 23:45:27 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-18 06:46:18 +0000 |
| commit | a6775bea054270edd81d57f769ff1e22edb2fe13 (patch) | |
| tree | a77c76a300736b4707078e52b0049d16bbcf6ff6 /chrome/browser/chromeos/file_manager/open_util.cc | |
| parent | e232abb7d549a2acff45aa610efe5c4373a7b409 (diff) | |
| download | chromium_src-a6775bea054270edd81d57f769ff1e22edb2fe13.zip chromium_src-a6775bea054270edd81d57f769ff1e22edb2fe13.tar.gz chromium_src-a6775bea054270edd81d57f769ff1e22edb2fe13.tar.bz2 | |
Remove granting permissions on behalf of Files app.
Files app is not available for some tests, and granting full permissions was an
overkill, as we grant only permissions to requested resources.
TEST=All current tests pass.
BUG=440674
Review URL: https://codereview.chromium.org/1013963002
Cr-Commit-Position: refs/heads/master@{#321088}
Diffstat (limited to 'chrome/browser/chromeos/file_manager/open_util.cc')
| -rw-r--r-- | chrome/browser/chromeos/file_manager/open_util.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/chromeos/file_manager/open_util.cc b/chrome/browser/chromeos/file_manager/open_util.cc index b21834e..9eeb9cc4 100644 --- a/chrome/browser/chromeos/file_manager/open_util.cc +++ b/chrome/browser/chromeos/file_manager/open_util.cc @@ -158,6 +158,7 @@ void OpenItem(Profile* profile, const platform_util::OpenOperationCallback& callback) { DCHECK_CURRENTLY_ON(BrowserThread::UI); + // This is unfortunately necessary as file browser handlers operate on URLs. GURL url; if (!ConvertAbsoluteFilePathToFileSystemUrl(profile, file_path, kFileManagerAppId, &url)) { @@ -166,7 +167,7 @@ void OpenItem(Profile* profile, } GetMetadataForPath( - GetFileSystemContextForExtensionId(profile, kFileManagerAppId), url, + GetFileSystemContextForExtensionId(profile, kFileManagerAppId), file_path, base::Bind(&OpenItemWithMetadata, profile, file_path, url, expected_type, callback)); } |