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

Unified Diff: chrome/browser/chromeos/gdata/gdata_cache.cc

Issue 10765020: gdata: Simplify semantics of sub_dir_type in CacheEntry. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/gdata/gdata_cache.cc
diff --git a/chrome/browser/chromeos/gdata/gdata_cache.cc b/chrome/browser/chromeos/gdata/gdata_cache.cc
index c47c1835f8c113eae0852a6bab629d91a45c4627..64f0d7e5640caeac620030d5696d9a83cce487a3 100644
--- a/chrome/browser/chromeos/gdata/gdata_cache.cc
+++ b/chrome/browser/chromeos/gdata/gdata_cache.cc
@@ -932,10 +932,9 @@ void GDataCache::Pin(const std::string& resource_id,
dest_path = FilePath::FromUTF8Unsafe(util::kSymLinkToDevNull);
source_path = dest_path;
- // Set sub_dir_type to PINNED to indicate that the file doesn't exist.
- // When the file is finally downloaded and StoreToCache called, it will be
- // moved to persistent directory.
- sub_dir_type = CACHE_TYPE_PINNED;
+ // Set sub_dir_type to TMP. The file will be first downloaded in 'tmp',
+ // then moved to 'persistent'.
+ sub_dir_type = CACHE_TYPE_TMP;
} else { // File exists in cache, determines destination path.
cache_state |= cache_entry->cache_state;

Powered by Google App Engine
This is Rietveld 408576698