Revert "Track and persist what file entries an extension has access to."

This reverts commit 4fe61bc24f343a6a1381a782dec4fb45b2f6c437 (r184878).

This was causing test failures in FileSystemApiOpenBackgroundTest and
FileSystemApiSaveBackgroundTest on linux (example
http://build.chromium.org/p/chromium.linux/buildstatus?builder=Linux%20Tests%20%28dbg%29%281%29&number=23657 )

TBR=koz@chromium.org
BUG=

Review URL: https://codereview.chromium.org/12319140

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184900 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 0 insertions, 76 deletions

diff --git a/chrome/browser/extensions/api/file_handlers/app_file_handler_util.cc b/chrome/browser/extensions/api/file_handlers/app_file_handler_util.cc
index d4ff715..22d3fa4 100644
--- a/chrome/browser/extensions/api/file_handlers/app_file_handler_util.cc
+++ b/chrome/browser/extensions/api/file_handlers/app_file_handler_util.cc
@@ -4,13 +4,7 @@
 
 #include "chrome/browser/extensions/api/file_handlers/app_file_handler_util.h"
 
-#include "chrome/browser/extensions/extension_prefs.h"
-#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/extensions/extension_system.h"
-#include "content/public/browser/child_process_security_policy.h"
 #include "net/base/mime_util.h"
-#include "webkit/fileapi/file_system_types.h"
-#include "webkit/fileapi/isolated_context.h"
 
 namespace extensions {
 
@@ -88,42 +82,6 @@ bool FileHandlerCanHandleFileWithMimeType(
   return false;
 }
 
-GrantedFileEntry CreateFileEntry(
-    Profile* profile,
-    const std::string& extension_id,
-    int renderer_id,
-    const base::FilePath& path,
-    bool writable) {
-  GrantedFileEntry result;
-  fileapi::IsolatedContext* isolated_context =
-      fileapi::IsolatedContext::GetInstance();
-  DCHECK(isolated_context);
-
-  result.filesystem_id = isolated_context->RegisterFileSystemForPath(
-      fileapi::kFileSystemTypeNativeLocal, path, &result.registered_name);
-
-  content::ChildProcessSecurityPolicy* policy =
-      content::ChildProcessSecurityPolicy::GetInstance();
-  policy->GrantReadFileSystem(renderer_id, result.filesystem_id);
-  if (writable)
-    policy->GrantWriteFileSystem(renderer_id, result.filesystem_id);
-
-  result.id = result.filesystem_id + ":" + result.registered_name;
-
-  // We only need file level access for reading FileEntries. Saving FileEntries
-  // just needs the file system to have read/write access, which is granted
-  // above if required.
-  if (!policy->CanReadFile(renderer_id, path))
-    policy->GrantReadFile(renderer_id, path);
-
-  ExtensionPrefs* prefs = extensions::ExtensionSystem::Get(profile)->
-      extension_service()->extension_prefs();
-  // Save this file entry in the prefs.
-  prefs->AddSavedFileEntry(extension_id, result.id, path, writable);
-
-  return result;
-}
-
 }  // namespace app_file_handler_util
 
 }  // namespace extensions
diff --git a/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h b/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
index a73a0c7..5bbb90af 100644
--- a/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
+++ b/chrome/browser/extensions/api/file_handlers/app_file_handler_util.h
@@ -12,8 +12,6 @@
 #include "chrome/common/extensions/api/file_handlers/file_handlers_parser.h"
 #include "chrome/common/extensions/extension.h"
 
-class Profile;
-
 namespace extensions {
 
 // TODO(benwells): move this to platform_apps namespace.
@@ -36,38 +34,6 @@ FindFileHandlersForMimeTypes(const Extension& extension,
 bool FileHandlerCanHandleFileWithMimeType(const FileHandlerInfo& handler,
                                           const std::string& mime_type);
 
-// Represents a file entry that a user has given an extension permission to
-// access. Intended to be persisted to disk (in the Preferences file), so should
-// remain serializable.
-struct SavedFileEntry {
-  SavedFileEntry(const std::string& id,
-                 const base::FilePath& path,
-                 bool writable)
-      : id(id),
-        path(path),
-        writable(writable) {}
-
-  std::string id;
-  base::FilePath path;
-  bool writable;
-};
-
-// Refers to a file entry that a renderer has been given access to.
-struct GrantedFileEntry {
-  std::string id;
-  std::string filesystem_id;
-  std::string registered_name;
-};
-
-// Creates a new file entry and allows |renderer_id| to access |path|. This
-// registers a new file system for |path|.
-GrantedFileEntry CreateFileEntry(
-    Profile* profile,
-    const std::string& extension_id,
-    int renderer_id,
-    const base::FilePath& path,
-    bool writable);
-
 }  // namespace app_file_handler_util
 
 }  // namespace extensions