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

Unified Diff: chrome/browser/chromeos/drive/file_cache_metadata_unittest.cc

Issue 15690021: drive: Move temporary file removal responsiblity to FileCache (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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/drive/file_cache_metadata_unittest.cc
diff --git a/chrome/browser/chromeos/drive/file_cache_metadata_unittest.cc b/chrome/browser/chromeos/drive/file_cache_metadata_unittest.cc
index bd5dee4039415cffe0e6afe4d8b1f3881f15ea07..fbc17948ab5a6e5c59281d326e7c3302ae9b1117 100644
--- a/chrome/browser/chromeos/drive/file_cache_metadata_unittest.cc
+++ b/chrome/browser/chromeos/drive/file_cache_metadata_unittest.cc
@@ -232,46 +232,6 @@ TEST_F(FileCacheMetadataTest, CorruptDB) {
EXPECT_FALSE(metadata_->GetCacheEntry("id_quux", "md5qux", &cache_entry));
}
-// Test FileCacheMetadata::RemoveTemporaryFiles.
-TEST_F(FileCacheMetadataTest, RemoveTemporaryFiles) {
- SetUpCacheMetadata();
-
- {
- FileCacheEntry cache_entry;
- cache_entry.set_md5("<md5>");
- cache_entry.set_is_present(true);
- metadata_->AddOrUpdateCacheEntry("<resource_id_1>", cache_entry);
- }
- {
- FileCacheEntry cache_entry;
- cache_entry.set_md5("<md5>");
- cache_entry.set_is_present(true);
- cache_entry.set_is_persistent(true);
- metadata_->AddOrUpdateCacheEntry("<resource_id_2>", cache_entry);
- }
- {
- FileCacheEntry cache_entry;
- cache_entry.set_md5("<md5>");
- cache_entry.set_is_present(true);
- cache_entry.set_is_persistent(true);
- metadata_->AddOrUpdateCacheEntry("<resource_id_3>", cache_entry);
- }
- {
- FileCacheEntry cache_entry;
- cache_entry.set_md5("<md5>");
- cache_entry.set_is_present(true);
- metadata_->AddOrUpdateCacheEntry("<resource_id_4>", cache_entry);
- }
-
- metadata_->RemoveTemporaryFiles();
- // resource 1 and 4 should be gone, as these are temporary.
- FileCacheEntry cache_entry;
- EXPECT_FALSE(metadata_->GetCacheEntry("<resource_id_1>", "", &cache_entry));
- EXPECT_TRUE(metadata_->GetCacheEntry("<resource_id_2>", "", &cache_entry));
- EXPECT_TRUE(metadata_->GetCacheEntry("<resource_id_3>", "", &cache_entry));
- EXPECT_FALSE(metadata_->GetCacheEntry("<resource_id_4>", "", &cache_entry));
-}
-
// Don't use TEST_F, as we don't want SetUp() and TearDown() for this test.
TEST(FileCacheMetadataExtraTest, CannotOpenDB) {
// Create nonexistent cache paths, so the initialization fails due to the
« no previous file with comments | « chrome/browser/chromeos/drive/file_cache_metadata.cc ('k') | chrome/browser/chromeos/drive/file_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698