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

Unified Diff: webkit/fileapi/local_file_system_file_util.cc

Issue 7717020: Adding Size() to AbstractFileEnumerator (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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
Index: webkit/fileapi/local_file_system_file_util.cc
diff --git a/webkit/fileapi/local_file_system_file_util.cc b/webkit/fileapi/local_file_system_file_util.cc
index ca028e42cb11499d7c46fb51d58a6e2e0e80d610..68d5dd01faa25f75d633a4558e5e9e6b6a91231c 100644
--- a/webkit/fileapi/local_file_system_file_util.cc
+++ b/webkit/fileapi/local_file_system_file_util.cc
@@ -241,8 +241,9 @@ class LocalFileSystemFileEnumerator
~LocalFileSystemFileEnumerator() {}
- virtual FilePath Next();
- virtual bool IsDirectory();
+ virtual FilePath Next() OVERRIDE;
+ virtual int64 Size() OVERRIDE;
+ virtual bool IsDirectory() OVERRIDE;
private:
file_util::FileEnumerator file_enum_;
@@ -260,6 +261,12 @@ FilePath LocalFileSystemFileEnumerator::Next() {
return virtual_root_path_.Append(path);
}
+int64 LocalFileSystemFileEnumerator::Size() {
+ file_util::FileEnumerator::FindInfo file_util_info;
ericu 2011/08/25 18:29:30 Is there any reason that we shouldn't cache the re
tzik 2011/08/26 02:41:53 Done. No reason other than making the patch small
+ file_enum_.GetFindInfo(&file_util_info);
+ return file_util::FileEnumerator::GetFilesize(file_util_info);
+}
+
bool LocalFileSystemFileEnumerator::IsDirectory() {
file_util::FileEnumerator::FindInfo file_util_info;
file_enum_.GetFindInfo(&file_util_info);

Powered by Google App Engine
This is Rietveld 408576698