diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-07 22:51:31 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-07 22:51:31 +0000 |
| commit | fcb524ead399b591b4e6587c2fbe712e45126320 (patch) | |
| tree | 77da32bc4e3ab916de166e0f9e52e0ad61629efa /tools | |
| parent | 3afef01773fcfe63523ebede8ad5a4dfed10088b (diff) | |
| download | chromium_src-fcb524ead399b591b4e6587c2fbe712e45126320.zip chromium_src-fcb524ead399b591b4e6587c2fbe712e45126320.tar.gz chromium_src-fcb524ead399b591b4e6587c2fbe712e45126320.tar.bz2 | |
Move FileEnumerator to its own file, do some refactoring.
It creates a class FileInfo to contain the details rather than using a platform-specific typedef. This allows the accessors GetName, GetSize, etc. to be moved directly to this class (previously they were static helpers on the FileEnumerator class) which makes a bunch of code much cleaner. It also gives reasonable getting and initialization which the previous version lacked.
BUG=175002
R=rvargas@chromium.org
Review URL: https://codereview.chromium.org/13165005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198820 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/android/md5sum/md5sum.cc | 5 | ||||
| -rw-r--r-- | tools/heapcheck/suppressions.txt | 6 | ||||
| -rw-r--r-- | tools/valgrind/tsan/suppressions.txt | 4 |
3 files changed, 8 insertions, 7 deletions
diff --git a/tools/android/md5sum/md5sum.cc b/tools/android/md5sum/md5sum.cc index ceee7d5..8543fb6 100644 --- a/tools/android/md5sum/md5sum.cc +++ b/tools/android/md5sum/md5sum.cc @@ -11,6 +11,7 @@ #include <string> #include "base/file_util.h" +#include "base/files/file_enumerator.h" #include "base/files/file_path.h" #include "base/logging.h" #include "base/md5.h" @@ -53,8 +54,8 @@ std::set<std::string> MakeFileSet(const char** files) { for (const char** file = files; *file; ++file) { base::FilePath file_path(*file); if (file_util::DirectoryExists(file_path)) { - file_util::FileEnumerator file_enumerator( - file_path, true /* recurse */, file_util::FileEnumerator::FILES); + base::FileEnumerator file_enumerator( + file_path, true /* recurse */, base::FileEnumerator::FILES); for (base::FilePath child, empty; (child = file_enumerator.Next()) != empty; ) { // If the path contains /.svn/, ignore it. diff --git a/tools/heapcheck/suppressions.txt b/tools/heapcheck/suppressions.txt index d9320f8..5f3de13 100644 --- a/tools/heapcheck/suppressions.txt +++ b/tools/heapcheck/suppressions.txt @@ -595,8 +595,8 @@ fun:std::string::_S_construct_aux fun:std::string::_S_construct fun:*basic_string* - fun:file_util::FileEnumerator::ReadDirectory - fun:file_util::FileEnumerator::Next + fun:base::FileEnumerator::ReadDirectory + fun:base::FileEnumerator::Next fun:::RelayReadDirectory::RunWork fun:::MessageLoopRelay::ProcessOnTargetThread } @@ -1859,4 +1859,4 @@ fun:net::URLRequest::StartJob fun:net::URLRequest::BeforeRequestComplete fun:net::URLRequest::Start -}
\ No newline at end of file +} diff --git a/tools/valgrind/tsan/suppressions.txt b/tools/valgrind/tsan/suppressions.txt index 9d20cef..5d57017 100644 --- a/tools/valgrind/tsan/suppressions.txt +++ b/tools/valgrind/tsan/suppressions.txt @@ -901,8 +901,8 @@ ThreadSanitizer:Race fun:__alloc_dir fun:opendir - fun:file_util::FileEnumerator::ReadDirectory - fun:file_util::FileEnumerator::Next + fun:base::FileEnumerator::ReadDirectory + fun:base::FileEnumerator::Next fun:::GetPrefsCandidateFilesFromFolder fun:ExternalPrefLoader::ReadStandaloneExtensionPrefFiles fun:ExternalPrefLoader::LoadOnFileThread |