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

Unified Diff: chrome/browser/extensions/api/file_system/file_system_api.cc

Issue 314113010: Remove deprecated permissions functions from Extension (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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: chrome/browser/extensions/api/file_system/file_system_api.cc
diff --git a/chrome/browser/extensions/api/file_system/file_system_api.cc b/chrome/browser/extensions/api/file_system/file_system_api.cc
index 2cd8ddfc5d4d74428896ce44228d45b56e741fbd..ddec04b85c31cfec27f9448279967d82e337245c 100644
--- a/chrome/browser/extensions/api/file_system/file_system_api.cc
+++ b/chrome/browser/extensions/api/file_system/file_system_api.cc
@@ -35,6 +35,7 @@
#include "extensions/browser/extension_prefs.h"
#include "extensions/browser/extension_system.h"
#include "extensions/common/permissions/api_permission.h"
+#include "extensions/common/permissions/permissions_data.h"
#include "grit/generated_resources.h"
#include "net/base/mime_util.h"
#include "ui/base/l10n/l10n_util.h"
@@ -386,7 +387,8 @@ bool FileSystemGetWritableEntryFunction::RunAsync() {
void FileSystemGetWritableEntryFunction::CheckPermissionAndSendResponse() {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
if (is_directory_ &&
- !extension_->HasAPIPermission(APIPermission::kFileSystemDirectory)) {
+ !extension_->permissions_data()->HasAPIPermission(
+ APIPermission::kFileSystemDirectory)) {
error_ = kRequiresFileSystemDirectoryError;
SendResponse(false);
}
@@ -869,7 +871,8 @@ bool FileSystemChooseEntryFunction::RunAsync() {
picker_type = ui::SelectFileDialog::SELECT_SAVEAS_FILE;
} else if (options->type == file_system::CHOOSE_ENTRY_TYPE_OPENDIRECTORY) {
is_directory_ = true;
- if (!extension_->HasAPIPermission(APIPermission::kFileSystemDirectory)) {
+ if (!extension_->permissions_data()->HasAPIPermission(
+ APIPermission::kFileSystemDirectory)) {
error_ = kRequiresFileSystemDirectoryError;
return false;
}

Powered by Google App Engine
This is Rietveld 408576698