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

Unified Diff: webkit/fileapi/file_system_url.h

Issue 12717014: Launch packaged app with files on Drive on Chrome OS. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update comment of the new fs type. Created 7 years, 9 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: webkit/fileapi/file_system_url.h
diff --git a/webkit/fileapi/file_system_url.h b/webkit/fileapi/file_system_url.h
index 29e6feed51a55811f1eeecf812813b0a62e82933..d25b2ec4007e713209013e267be697491960d2dc 100644
--- a/webkit/fileapi/file_system_url.h
+++ b/webkit/fileapi/file_system_url.h
@@ -61,7 +61,7 @@ namespace fileapi {
// Additionally, following accessors would return valid values:
// filesystem_id() returns 'mount_name'.
kinuko 2013/03/21 16:47:51 can you add a brief explanation about mount_filesy
kinaba 2013/03/22 04:58:33 Done.
//
-// It is imposible to directly create a valid FileSystemURL instance (except by
+// 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
@@ -112,6 +112,9 @@ class WEBKIT_STORAGE_EXPORT FileSystemURL {
// 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_; }
@@ -139,21 +142,23 @@ class WEBKIT_STORAGE_EXPORT FileSystemURL {
FileSystemURL(const GURL& origin,
FileSystemType mount_type,
const base::FilePath& virtual_path,
- const std::string& filesystem_id,
+ const std::string& mount_filesystem_id,
FileSystemType cracked_type,
- const base::FilePath& cracked_path);
+ const base::FilePath& cracked_path,
+ const std::string& filesystem_id);
bool is_valid_;
+ // Values parsed from the original URL.
GURL origin_;
- FileSystemType type_;
FileSystemType mount_type_;
- base::FilePath path_;
-
- // Values specific to cracked URLs.
- std::string filesystem_id_;
base::FilePath virtual_path_;
+ // Values obtained by cracking URLs.
+ std::string mount_filesystem_id_;
+ FileSystemType type_;
+ base::FilePath path_;
+ std::string filesystem_id_;
};
typedef std::set<FileSystemURL, FileSystemURL::Comparator> FileSystemURLSet;

Powered by Google App Engine
This is Rietveld 408576698