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

Unified Diff: webkit/fileapi/obfuscated_file_system_file_util.h

Issue 7470037: [Refactor] to rename and re-layer the file_util stack layers. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Rebased on the svn tree. Created 9 years, 3 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
« no previous file with comments | « webkit/fileapi/native_file_util.cc ('k') | webkit/fileapi/obfuscated_file_system_file_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/obfuscated_file_system_file_util.h
diff --git a/webkit/fileapi/obfuscated_file_system_file_util.h b/webkit/fileapi/obfuscated_file_system_file_util.h
deleted file mode 100644
index 782fc07f7abc09cb200595672260f12317ec6fae..0000000000000000000000000000000000000000
--- a/webkit/fileapi/obfuscated_file_system_file_util.h
+++ /dev/null
@@ -1,264 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef WEBKIT_FILEAPI_OBFUSCATED_FILE_SYSTEM_FILE_UTIL_H_
-#define WEBKIT_FILEAPI_OBFUSCATED_FILE_SYSTEM_FILE_UTIL_H_
-
-#include <map>
-#include <string>
-#include <vector>
-
-#include "base/file_path.h"
-#include "base/file_util_proxy.h"
-#include "base/memory/ref_counted.h"
-#include "base/platform_file.h"
-#include "base/timer.h"
-#include "webkit/fileapi/file_system_directory_database.h"
-#include "webkit/fileapi/file_system_file_util.h"
-#include "webkit/fileapi/file_system_origin_database.h"
-#include "webkit/fileapi/file_system_types.h"
-
-namespace base {
-struct PlatformFileInfo;
-class Time;
-}
-
-class GURL;
-
-namespace fileapi {
-
-class FileSystemOperationContext;
-
-// The overall implementation philosophy of this class is that partial failures
-// should leave us with an intact database; we'd prefer to leak the occasional
-// backing file than have a database entry whose backing file is missing. When
-// doing FSCK operations, if you find a loose backing file with no reference,
-// you may safely delete it.
-//
-// This class is RefCountedThreadSafe because it may gain a reference on the IO
-// thread, but must be deleted on the FILE thread because that's where
-// DropDatabases needs to be called. References will be held by the
-// SandboxMountPointProvider [and the task it uses to drop the reference] and
-// SandboxMountPointProvider::GetFileSystemRootPathTask. Without that last one,
-// we wouldn't need ref counting.
-//
-// TODO(ericu): We don't ever update directory mtimes; which operations should
-// do that?
-class ObfuscatedFileSystemFileUtil : public FileSystemFileUtil,
- public base::RefCountedThreadSafe<ObfuscatedFileSystemFileUtil> {
- public:
- // |underlying_file_util| is owned by the instance. It will be deleted by
- // the owner instance. For example, it can be instanciated as follows:
- // FileSystemFileUtil* file_system_file_util =
- // new ObfuscatedFileSystemFileUtil(new FileSystemFileUtil());
- ObfuscatedFileSystemFileUtil(
- const FilePath& file_system_directory,
- FileSystemFileUtil* underlying_file_util);
- virtual ~ObfuscatedFileSystemFileUtil();
-
- virtual base::PlatformFileError CreateOrOpen(
- FileSystemOperationContext* context,
- const FilePath& file_path,
- int file_flags,
- base::PlatformFile* file_handle,
- bool* created) OVERRIDE;
-
- virtual base::PlatformFileError EnsureFileExists(
- FileSystemOperationContext* context,
- const FilePath& file_path, bool* created) OVERRIDE;
-
- virtual base::PlatformFileError GetLocalFilePath(
- FileSystemOperationContext* context,
- const FilePath& virtual_file,
- FilePath* local_path) OVERRIDE;
-
- virtual base::PlatformFileError GetFileInfo(
- FileSystemOperationContext* context,
- const FilePath& file,
- base::PlatformFileInfo* file_info,
- FilePath* platform_file) OVERRIDE;
-
- virtual base::PlatformFileError ReadDirectory(
- FileSystemOperationContext* context,
- const FilePath& file_path,
- std::vector<base::FileUtilProxy::Entry>* entries) OVERRIDE;
-
- virtual base::PlatformFileError CreateDirectory(
- FileSystemOperationContext* context,
- const FilePath& file_path,
- bool exclusive,
- bool recursive) OVERRIDE;
-
- virtual base::PlatformFileError CopyOrMoveFile(
- FileSystemOperationContext* context,
- const FilePath& src_file_path,
- const FilePath& dest_file_path,
- bool copy) OVERRIDE;
-
- virtual PlatformFileError CopyInForeignFile(
- FileSystemOperationContext* context,
- const FilePath& src_file_path,
- const FilePath& dest_file_path) OVERRIDE;
-
- virtual base::PlatformFileError DeleteFile(
- FileSystemOperationContext* context,
- const FilePath& file_path) OVERRIDE;
-
- virtual base::PlatformFileError DeleteSingleDirectory(
- FileSystemOperationContext* context,
- const FilePath& file_path) OVERRIDE;
-
- virtual base::PlatformFileError Touch(
- FileSystemOperationContext* context,
- const FilePath& file_path,
- const base::Time& last_access_time,
- const base::Time& last_modified_time) OVERRIDE;
-
- virtual base::PlatformFileError Truncate(
- FileSystemOperationContext* context,
- const FilePath& path,
- int64 length) OVERRIDE;
-
- virtual bool PathExists(
- FileSystemOperationContext* context,
- const FilePath& file_path) OVERRIDE;
-
- virtual bool DirectoryExists(
- FileSystemOperationContext* context,
- const FilePath& file_path) OVERRIDE;
-
- virtual bool IsDirectoryEmpty(
- FileSystemOperationContext* context,
- const FilePath& file_path) OVERRIDE;
-
- // Gets the topmost directory specific to this origin and type. This will
- // contain both the directory database's files and all the backing file
- // subdirectories.
- FilePath GetDirectoryForOriginAndType(
- const GURL& origin, FileSystemType type, bool create);
-
- // Deletes the topmost directory specific to this origin and type. This will
- // delete its directory database.
- bool DeleteDirectoryForOriginAndType(const GURL& origin, FileSystemType type);
-
- // This will migrate a filesystem from the old passthrough sandbox into the
- // new obfuscated one. It won't obfuscate the old filenames [it will maintain
- // the old structure, but move it to a new root], but any new files created
- // will go into the new standard locations. This will be completely
- // transparent to the user. This migration is atomic in that it won't alter
- // the source data until it's done, and that will be with a single directory
- // move [the directory with the unguessable name will move into the new
- // filesystem storage directory]. However, if this fails partway through, it
- // might leave a seemingly-valid database for this origin. When it starts up,
- // it will clear any such database, just in case.
- bool MigrateFromOldSandbox(
- const GURL& origin, FileSystemType type, const FilePath& root);
-
- // TODO(ericu): This doesn't really feel like it belongs in this class.
- // The previous version lives in FileSystemPathManager, but perhaps
- // SandboxMountPointProvider would be better?
- static FilePath::StringType GetDirectoryNameForType(FileSystemType type);
-
- // Origin enumerator interface.
- // An instance of this interface is assumed to be called on the file thread.
- class AbstractOriginEnumerator {
- public:
- virtual ~AbstractOriginEnumerator() {}
-
- // Returns the next origin. Returns empty if there are no more origins.
- virtual GURL Next() = 0;
-
- // Returns the current origin's information.
- virtual bool HasFileSystemType(FileSystemType type) const = 0;
- };
-
- // This method and all methods of its returned class must be called only on
- // the FILE thread. The caller is responsible for deleting the returned
- // object.
- AbstractOriginEnumerator* CreateOriginEnumerator();
-
- virtual AbstractFileEnumerator* CreateFileEnumerator(
- FileSystemOperationContext* context,
- const FilePath& root_path) OVERRIDE;
-
- // Deletes a directory database from the database list in the ObfuscatedFSFU
- // and destroys the database on the disk.
- bool DestroyDirectoryDatabase(const GURL& origin, FileSystemType type);
-
- // Computes a cost for storing a given file in the obfuscated FSFU.
- // As the cost of a file is independent of the cost of its parent directories,
- // this ignores all but the BaseName of the supplied path. In order to
- // compute the cost of adding a multi-segment directory recursively, call this
- // on each path segment and add the results.
- static int64 ComputeFilePathCost(const FilePath& path);
-
- private:
- typedef FileSystemDirectoryDatabase::FileId FileId;
- typedef FileSystemDirectoryDatabase::FileInfo FileInfo;
-
- base::PlatformFileError GetFileInfoInternal(
- FileSystemDirectoryDatabase* db,
- FileSystemOperationContext* context,
- FileId file_id,
- FileInfo* local_info,
- base::PlatformFileInfo* file_info,
- FilePath* platform_file_path);
-
- // Creates a new file, both the underlying backing file and the entry in the
- // database. file_info is an in-out parameter. Supply the name and
- // parent_id; data_path is ignored. On success, data_path will
- // always be set to the relative path [from the root of the type-specific
- // filesystem directory] of a NEW backing file, and handle, if supplied, will
- // hold open PlatformFile for the backing file, which the caller is
- // responsible for closing. If you supply a path in source_path, it will be
- // used as a source from which to COPY data.
- // Caveat: do not supply handle if you're also supplying a data path. It was
- // easier not to support this, and no code has needed it so far, so it will
- // DCHECK and handle will hold base::kInvalidPlatformFileValue.
- base::PlatformFileError CreateFile(
- FileSystemOperationContext* context,
- const GURL& origin_url, FileSystemType type,
- const FilePath& source_path, FileInfo* file_info,
- int file_flags, base::PlatformFile* handle);
- // Given the filesystem's root URL and a virtual path, produces a real, full
- // local path to the underlying data file. This does a database lookup, and
- // verifies that the file exists.
- FilePath GetLocalPath(
- const GURL& origin_url,
- FileSystemType type,
- const FilePath& virtual_path);
- // This converts from a relative path [as is stored in the FileInfo.data_path
- // field] to an absolute local path that can be given to the operating system.
- // It does no checks as to whether the file actually exists; it's pure path
- // manipulation.
- FilePath DataPathToLocalPath(
- const GURL& origin, FileSystemType type, const FilePath& data_path);
- // This does the reverse of DataPathToLocalPath.
- FilePath LocalPathToDataPath(
- const GURL& origin, FileSystemType type, const FilePath& local_path);
- // This returns NULL if |create| flag is false and a filesystem does not
- // exist for the given |origin_url| and |type|.
- // For read operations |create| should be false.
- FileSystemDirectoryDatabase* GetDirectoryDatabase(
- const GURL& origin_url, FileSystemType type, bool create);
- // Gets the topmost directory specific to this origin. This will
- // contain both the filesystem type subdirectories.
- FilePath GetDirectoryForOrigin(const GURL& origin, bool create);
- void MarkUsed();
- void DropDatabases();
- bool InitOriginDatabase(bool create);
-
- typedef std::map<std::string, FileSystemDirectoryDatabase*> DirectoryMap;
- DirectoryMap directories_;
- scoped_ptr<FileSystemOriginDatabase> origin_database_;
- FilePath file_system_directory_;
- base::OneShotTimer<ObfuscatedFileSystemFileUtil> timer_;
- scoped_ptr<FileSystemFileUtil> underlying_file_util_;
-
- DISALLOW_COPY_AND_ASSIGN(ObfuscatedFileSystemFileUtil);
-};
-
-} // namespace fileapi
-
-#endif // WEBKIT_FILEAPI_OBFUSCATED_FILE_SYSTEM_FILE_UTIL_H_
« no previous file with comments | « webkit/fileapi/native_file_util.cc ('k') | webkit/fileapi/obfuscated_file_system_file_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698