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

Unified Diff: webkit/browser/fileapi/file_system_url.h

Issue 442383002: Move storage-related files from webkit/ to new top-level directory storage/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 4 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/browser/fileapi/file_system_quota_util.h ('k') | webkit/browser/fileapi/file_system_url.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/browser/fileapi/file_system_url.h
diff --git a/webkit/browser/fileapi/file_system_url.h b/webkit/browser/fileapi/file_system_url.h
deleted file mode 100644
index ead076254b1f059bab9901e2b395421a830651d1..0000000000000000000000000000000000000000
--- a/webkit/browser/fileapi/file_system_url.h
+++ /dev/null
@@ -1,180 +0,0 @@
-// Copyright (c) 2012 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_BROWSER_FILEAPI_FILE_SYSTEM_URL_H_
-#define WEBKIT_BROWSER_FILEAPI_FILE_SYSTEM_URL_H_
-
-#include <set>
-#include <string>
-
-#include "base/files/file_path.h"
-#include "url/gurl.h"
-#include "webkit/browser/webkit_storage_browser_export.h"
-#include "webkit/common/fileapi/file_system_mount_option.h"
-#include "webkit/common/fileapi/file_system_types.h"
-
-namespace fileapi {
-
-// A class representing a filesystem URL which consists of origin URL,
-// type and an internal path used inside the filesystem.
-//
-// When a FileSystemURL instance is created for a GURL (for filesystem: scheme),
-// each accessor method would return following values:
-//
-// Example: For a URL 'filesystem:http://foo.com/temporary/foo/bar':
-// origin() returns 'http://foo.com',
-// mount_type() returns kFileSystemTypeTemporary,
-// virtual_path() returns 'foo/bar',
-// type() returns the same value as mount_type(),
-// path() returns the same value as virtual_path(),
-//
-// All other accessors return empty or invalid value.
-//
-// FileSystemURL can also be created to represent a 'cracked' filesystem URL if
-// the original URL's type/path is pointing to a mount point which can be
-// further resolved to a lower filesystem type/path.
-//
-// Example: Assume a path '/media/removable' is mounted at mount name
-// 'mount_name' with type kFileSystemTypeFoo as an external file system.
-//
-// The original URL would look like:
-// 'filesystem:http://bar.com/external/mount_name/foo/bar':
-//
-// FileSystemURL('http://bar.com',
-// kFileSystemTypeExternal,
-// 'mount_name/foo/bar'
-// 'mount_name',
-// kFileSystemTypeFoo,
-// '/media/removable/foo/bar');
-// would create a FileSystemURL whose accessors return:
-//
-// origin() returns 'http://bar.com',
-// mount_type() returns kFileSystemTypeExternal,
-// virtual_path() returns 'mount_name/foo/bar',
-// type() returns the kFileSystemTypeFoo,
-// path() returns '/media/removable/foo/bar',
-//
-// Note that in either case virtual_path() always returns the path part after
-// 'type' part in the original URL, and mount_type() always returns the 'type'
-// part in the original URL.
-//
-// Additionally, following accessors would return valid values:
-// filesystem_id() returns 'mount_name'.
-//
-// It is impossible to directly create a valid FileSystemURL instance (except by
-// using CreatedForTest methods, which should not be used in production code).
-// To get a valid FileSystemURL, one of the following methods can be used:
-// <Friend>::CrackURL, <Friend>::CreateCrackedFileSystemURL, where <Friend> is
-// one of the friended classes.
-//
-// TODO(ericu): Look into making virtual_path() [and all FileSystem API virtual
-// paths] just an std::string, to prevent platform-specific base::FilePath
-// behavior from getting invoked by accident. Currently the base::FilePath
-// returned here needs special treatment, as it may contain paths that are
-// illegal on the current platform.
-// To avoid problems, use VirtualPath::BaseName and
-// VirtualPath::GetComponents instead of the base::FilePath methods.
-class WEBKIT_STORAGE_BROWSER_EXPORT FileSystemURL {
- public:
- FileSystemURL();
- ~FileSystemURL();
-
- // Methods for creating FileSystemURL without attempting to crack them.
- // Should be used only in tests.
- static FileSystemURL CreateForTest(const GURL& url);
- static FileSystemURL CreateForTest(const GURL& origin,
- FileSystemType mount_type,
- const base::FilePath& virtual_path);
-
- // Returns true if this instance represents a valid FileSystem URL.
- bool is_valid() const { return is_valid_; }
-
- // Returns the origin part of this URL. See the class comment for details.
- const GURL& origin() const { return origin_; }
-
- // Returns the type part of this URL. See the class comment for details.
- FileSystemType type() const { return type_; }
-
- // Returns the cracked path of this URL. See the class comment for details.
- const base::FilePath& path() const { return path_; }
-
- // Returns the original path part of this URL.
- // See the class comment for details.
- // TODO(kinuko): this must return std::string.
- const base::FilePath& virtual_path() const { return virtual_path_; }
-
- // Returns the filesystem ID/mount name for isolated/external filesystem URLs.
- // See the class comment for details.
- const std::string& filesystem_id() const { return filesystem_id_; }
- const std::string& mount_filesystem_id() const {
- return mount_filesystem_id_;
- }
-
- FileSystemType mount_type() const { return mount_type_; }
-
- const FileSystemMountOption& mount_option() const { return mount_option_; }
-
- // Returns the formatted URL of this instance.
- GURL ToGURL() const;
-
- std::string DebugString() const;
-
- // Returns true if this URL is a strict parent of the |child|.
- bool IsParent(const FileSystemURL& child) const;
-
- bool IsInSameFileSystem(const FileSystemURL& other) const;
-
- bool operator==(const FileSystemURL& that) const;
-
- bool operator!=(const FileSystemURL& that) const {
- return !(*this == that);
- }
-
- struct WEBKIT_STORAGE_BROWSER_EXPORT Comparator {
- bool operator() (const FileSystemURL& lhs, const FileSystemURL& rhs) const;
- };
-
- private:
- friend class FileSystemContext;
- friend class ExternalMountPoints;
- friend class IsolatedContext;
-
- explicit FileSystemURL(const GURL& filesystem_url);
- FileSystemURL(const GURL& origin,
- FileSystemType mount_type,
- const base::FilePath& virtual_path);
- // Creates a cracked FileSystemURL.
- FileSystemURL(const GURL& origin,
- FileSystemType mount_type,
- const base::FilePath& virtual_path,
- const std::string& mount_filesystem_id,
- FileSystemType cracked_type,
- const base::FilePath& cracked_path,
- const std::string& filesystem_id,
- const FileSystemMountOption& mount_option);
-
- bool is_valid_;
-
- // Values parsed from the original URL.
- GURL origin_;
- FileSystemType mount_type_;
- base::FilePath virtual_path_;
-
- // Values obtained by cracking URLs.
- // |mount_filesystem_id_| is retrieved from the first round of cracking,
- // and the rest of the fields are from recursive cracking. Permission
- // checking on the top-level mount information should be done with the former,
- // and the low-level file operation should be implemented with the latter.
- std::string mount_filesystem_id_;
- FileSystemType type_;
- base::FilePath path_;
- std::string filesystem_id_;
- FileSystemMountOption mount_option_;
-};
-
-typedef std::set<FileSystemURL, FileSystemURL::Comparator> FileSystemURLSet;
-
-} // namespace fileapi
-
-#endif // WEBKIT_BROWSER_FILEAPI_FILE_SYSTEM_URL_H_
« no previous file with comments | « webkit/browser/fileapi/file_system_quota_util.h ('k') | webkit/browser/fileapi/file_system_url.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698