diff options
Diffstat (limited to 'base')
-rw-r--r-- | base/base_paths.h | 9 | ||||
-rw-r--r-- | base/base_paths_linux.cc | 2 | ||||
-rw-r--r-- | base/base_paths_mac.mm | 2 | ||||
-rw-r--r-- | base/path_service.cc | 8 | ||||
-rw-r--r-- | base/path_service.h | 5 | ||||
-rw-r--r-- | base/path_service_unittest.cc | 6 |
6 files changed, 7 insertions, 25 deletions
diff --git a/base/base_paths.h b/base/base_paths.h index 0f9be04..72cff15 100644 --- a/base/base_paths.h +++ b/base/base_paths.h @@ -33,10 +33,11 @@ enum { // for tests that need to locate various resources. It // should not be used outside of test code. #if defined(OS_POSIX) - DIR_USER_CACHE, // Directory where user cache data resides. The Chromium - // browser cache can be a subdirectory of DIR_USER_CACHE. - // This is $XDG_CACHE_HOME on Linux and ~/Library/Caches - // on Mac. + DIR_CACHE, // Directory where to put cache data. Note this is + // *not* where the browser cache lives, but the + // browser cache can be a subdirectory. + // This is $XDG_CACHE_HOME on Linux and + // ~/Library/Caches on Mac. #endif PATH_END diff --git a/base/base_paths_linux.cc b/base/base_paths_linux.cc index 6d837f2..701b983 100644 --- a/base/base_paths_linux.cc +++ b/base/base_paths_linux.cc @@ -105,7 +105,7 @@ bool PathProviderPosix(int key, FilePath* result) { << "Try running from your chromium/src directory."; return false; } - case base::DIR_USER_CACHE: + case base::DIR_CACHE: scoped_ptr<base::Environment> env(base::Environment::Create()); FilePath cache_dir(base::nix::GetXDGDirectory(env.get(), "XDG_CACHE_HOME", ".cache")); diff --git a/base/base_paths_mac.mm b/base/base_paths_mac.mm index b9225d8..97413ba 100644 --- a/base/base_paths_mac.mm +++ b/base/base_paths_mac.mm @@ -47,7 +47,7 @@ bool PathProviderMac(int key, FilePath* result) { case base::FILE_MODULE: { return GetNSExecutablePath(result); } - case base::DIR_USER_CACHE: + case base::DIR_CACHE: return mac_util::GetUserDirectory(NSCachesDirectory, result); case base::DIR_APP_DATA: return mac_util::GetUserDirectory(NSApplicationSupportDirectory, result); diff --git a/base/path_service.cc b/base/path_service.cc index 9eed56d..8660c42 100644 --- a/base/path_service.cc +++ b/base/path_service.cc @@ -215,14 +215,6 @@ bool PathService::Get(int key, std::wstring* result) { } #endif -// TODO(evan): remove me -- see comments in header. -bool PathService::IsOverridden(int key) { - PathData* path_data = GetPathData(); - DCHECK(path_data); - AutoLock scoped_lock(path_data->lock); - return path_data->overrides.find(key) != path_data->overrides.end(); -} - bool PathService::Override(int key, const FilePath& path) { PathData* path_data = GetPathData(); DCHECK(path_data); diff --git a/base/path_service.h b/base/path_service.h index 6873e1f..4d99cdc 100644 --- a/base/path_service.h +++ b/base/path_service.h @@ -45,11 +45,6 @@ class PathService { // over the lifetime of the app, so this method should be used with caution. static bool Override(int key, const FilePath& path); - // Return whether a path was overridden. - // TODO(evan): temporarily restoring this to quick-fix a regression. - // Remove me again once it's fixed properly. - static bool IsOverridden(int key); - // To extend the set of supported keys, you can register a path provider, // which is just a function mirroring PathService::Get. The ProviderFunc // returns false if it cannot provide a non-empty path for the given key. diff --git a/base/path_service_unittest.cc b/base/path_service_unittest.cc index c65f7a5..4c41aed 100644 --- a/base/path_service_unittest.cc +++ b/base/path_service_unittest.cc @@ -21,12 +21,6 @@ namespace { bool ReturnsValidPath(int dir_type) { FilePath path; bool result = PathService::Get(dir_type, &path); -#if defined(OS_POSIX) - // If chromium has never been started on this account, the cache path will not - // exist. - if (dir_type == base::DIR_USER_CACHE) - return result && !path.value().empty(); -#endif return result && !path.value().empty() && file_util::PathExists(path); } |