Deprecate MountPointProvider::IsAccessAllowed in favor of GetPermissionPolicy

chrome/browser/chromeos/extensions/file_handler_util.cc

Index: chrome/browser/chromeos/extensions/file_handler_util.cc
diff --git a/chrome/browser/chromeos/extensions/file_handler_util.cc b/chrome/browser/chromeos/extensions/file_handler_util.cc
index b6b56e4c97f31656e1a6116e5744107320fc8d2a..a688f78a9a9d2c25f67fa69f2ade85fbade1272e 100644
--- a/chrome/browser/chromeos/extensions/file_handler_util.cc
+++ b/chrome/browser/chromeos/extensions/file_handler_util.cc
@@ -659,7 +659,9 @@ bool FileTaskExecutor::FileBrowserHasAccessPermissionForFiles(
       return false;
 
     if (!chromeos::CrosMountPointProvider::CanHandleURL(files[i]) ||
-        !external_provider->IsAccessAllowed(files[i])) {
+        (external_provider->GetPermissionPolicy(
+            files[i], fileapi::kReadFilePermissions) ==
+            fileapi::FILE_PERMISSION_ALWAYS_DENY)) {

[kinuko, 2013/02/04 08:58:25]

Maybe this change needs some more care too

[tbarzic, 2013/02/05 00:09:34]

hmm, this should be ok.. to be honest I've started thinking about removing this
part of code and replacing it with something like:
CHECK(TheExtensionHasFileBrowserPrivatePermission(file_browser_id_), but that's
not much related to this cl.

       return false;
     }
   }