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

Side by Side Diff: webkit/browser/fileapi/file_system_usage_cache.cc

Issue 16950028: Move file_util::Delete to the base namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "webkit/browser/fileapi/file_system_usage_cache.h" 5 #include "webkit/browser/fileapi/file_system_usage_cache.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/debug/trace_event.h" 9 #include "base/debug/trace_event.h"
10 #include "base/file_util.h" 10 #include "base/file_util.h"
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 bool FileSystemUsageCache::Exists(const base::FilePath& usage_file_path) { 148 bool FileSystemUsageCache::Exists(const base::FilePath& usage_file_path) {
149 TRACE_EVENT0("FileSystem", "UsageCache::Exists"); 149 TRACE_EVENT0("FileSystem", "UsageCache::Exists");
150 DCHECK(CalledOnValidThread()); 150 DCHECK(CalledOnValidThread());
151 return file_util::PathExists(usage_file_path); 151 return file_util::PathExists(usage_file_path);
152 } 152 }
153 153
154 bool FileSystemUsageCache::Delete(const base::FilePath& usage_file_path) { 154 bool FileSystemUsageCache::Delete(const base::FilePath& usage_file_path) {
155 TRACE_EVENT0("FileSystem", "UsageCache::Delete"); 155 TRACE_EVENT0("FileSystem", "UsageCache::Delete");
156 DCHECK(CalledOnValidThread()); 156 DCHECK(CalledOnValidThread());
157 CloseCacheFiles(); 157 CloseCacheFiles();
158 return file_util::Delete(usage_file_path, true); 158 return base::Delete(usage_file_path, true);
159 } 159 }
160 160
161 void FileSystemUsageCache::CloseCacheFiles() { 161 void FileSystemUsageCache::CloseCacheFiles() {
162 TRACE_EVENT0("FileSystem", "UsageCache::CloseCacheFiles"); 162 TRACE_EVENT0("FileSystem", "UsageCache::CloseCacheFiles");
163 DCHECK(CalledOnValidThread()); 163 DCHECK(CalledOnValidThread());
164 for (CacheFiles::iterator itr = cache_files_.begin(); 164 for (CacheFiles::iterator itr = cache_files_.begin();
165 itr != cache_files_.end(); ++itr) { 165 itr != cache_files_.end(); ++itr) {
166 if (itr->second != base::kInvalidPlatformFileValue) 166 if (itr->second != base::kInvalidPlatformFileValue)
167 base::ClosePlatformFile(itr->second); 167 base::ClosePlatformFile(itr->second);
168 } 168 }
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
306 return !task_runner_.get() || task_runner_->RunsTasksOnCurrentThread(); 306 return !task_runner_.get() || task_runner_->RunsTasksOnCurrentThread();
307 } 307 }
308 308
309 bool FileSystemUsageCache::HasCacheFileHandle(const base::FilePath& file_path) { 309 bool FileSystemUsageCache::HasCacheFileHandle(const base::FilePath& file_path) {
310 DCHECK(CalledOnValidThread()); 310 DCHECK(CalledOnValidThread());
311 DCHECK_LE(cache_files_.size(), kMaxHandleCacheSize); 311 DCHECK_LE(cache_files_.size(), kMaxHandleCacheSize);
312 return ContainsKey(cache_files_, file_path); 312 return ContainsKey(cache_files_, file_path);
313 } 313 }
314 314
315 } // namespace fileapi 315 } // namespace fileapi
OLDNEW
« no previous file with comments | « webkit/browser/dom_storage/session_storage_database.cc ('k') | webkit/browser/fileapi/native_file_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698