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

Unified Diff: net/disk_cache/cache_util_posix.cc

Issue 10855002: Change the type of file_type parameter to int, as the parameter actually takes or-ed bitmasks, (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Addressed the comments about constant names. Created 8 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: net/disk_cache/cache_util_posix.cc
diff --git a/net/disk_cache/cache_util_posix.cc b/net/disk_cache/cache_util_posix.cc
index ec5f931b2ccfb143f8b2b5d72b90e00d178bb38f..e66795e2e5368025618debb9aef7e25f2b9a20ca 100644
--- a/net/disk_cache/cache_util_posix.cc
+++ b/net/disk_cache/cache_util_posix.cc
@@ -22,12 +22,9 @@ bool MoveCache(const FilePath& from_path, const FilePath& to_path) {
LOG(ERROR) << "Unable to create destination cache directory.";
return false;
}
- file_util::FileEnumerator iter(
- from_path,
- /* recursive */ false,
- static_cast<file_util::FileEnumerator::FileType>(
- file_util::FileEnumerator::DIRECTORIES |
- file_util::FileEnumerator::FILES));
+ file_util::FileEnumerator iter(from_path, false /* not recursive */,
+ (file_util::FileEnumerator::DIRECTORIES |
+ file_util::FileEnumerator::FILES));
for (FilePath name = iter.Next(); !name.value().empty(); name = iter.Next()) {
FilePath destination = to_path.Append(name.BaseName());
if (!file_util::Move(name, destination)) {

Powered by Google App Engine
This is Rietveld 408576698