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

Unified Diff: net/disk_cache/backend_unittest.cc

Issue 266243004: Clang format slam. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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/backend_unittest.cc
diff --git a/net/disk_cache/backend_unittest.cc b/net/disk_cache/backend_unittest.cc
index e118dc6a9bd6e1c2e1fdd91df3aedfc3e78b3dab..33f87ac3d0ab57265dba463e37869bffbd2484bf 100644
--- a/net/disk_cache/backend_unittest.cc
+++ b/net/disk_cache/backend_unittest.cc
@@ -277,7 +277,7 @@ bool DiskCacheBackendTest::EnumerateAndMatchKeys(
void DiskCacheBackendTest::BackendBasics() {
InitCache();
- disk_cache::Entry *entry1 = NULL, *entry2 = NULL;
+ disk_cache::Entry* entry1 = NULL, *entry2 = NULL;
EXPECT_NE(net::OK, OpenEntry("the first key", &entry1));
ASSERT_EQ(net::OK, CreateEntry("the first key", &entry1));
ASSERT_TRUE(NULL != entry1);
@@ -349,7 +349,7 @@ void DiskCacheBackendTest::BackendKeying() {
InitCache();
const char* kName1 = "the first key";
const char* kName2 = "the first Key";
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
ASSERT_EQ(net::OK, CreateEntry(kName1, &entry1));
ASSERT_EQ(net::OK, CreateEntry(kName2, &entry2));
@@ -367,7 +367,7 @@ void DiskCacheBackendTest::BackendKeying() {
EXPECT_TRUE(entry1 == entry2);
entry2->Close();
- base::strlcpy(buffer + 3, kName1, arraysize(buffer) - 3);
+ base::strlcpy(buffer + 3, kName1, arraysize(buffer) - 3);
ASSERT_EQ(net::OK, OpenEntry(buffer + 3, &entry2));
EXPECT_TRUE(entry1 == entry2);
entry2->Close();
@@ -442,8 +442,12 @@ TEST_F(DiskCacheTest, CreateBackend) {
rv = disk_cache::CreateCacheBackend(net::MEMORY_CACHE,
net::CACHE_BACKEND_DEFAULT,
- base::FilePath(), 0,
- false, NULL, NULL, &cache,
+ base::FilePath(),
+ 0,
+ false,
+ NULL,
+ NULL,
+ &cache,
cb.callback());
ASSERT_EQ(net::OK, cb.GetResult(rv));
ASSERT_TRUE(cache.get());
@@ -558,10 +562,16 @@ TEST_F(DiskCacheBackendTest, MultipleInstancesWithPendingFileIO) {
net::TestCompletionCallback cb;
scoped_ptr<disk_cache::Backend> extra_cache;
- int rv = disk_cache::CreateCacheBackend(
- net::DISK_CACHE, net::CACHE_BACKEND_DEFAULT, store.path(), 0,
- false, base::MessageLoopProxy::current().get(), NULL,
- &extra_cache, cb.callback());
+ int rv =
+ disk_cache::CreateCacheBackend(net::DISK_CACHE,
+ net::CACHE_BACKEND_DEFAULT,
+ store.path(),
+ 0,
+ false,
+ base::MessageLoopProxy::current().get(),
+ NULL,
+ &extra_cache,
+ cb.callback());
ASSERT_EQ(net::OK, cb.GetResult(rv));
ASSERT_TRUE(extra_cache.get() != NULL);
@@ -640,7 +650,7 @@ void DiskCacheBackendTest::BackendShutdownWithPendingCreate(bool fast) {
base::Thread::Options(base::MessageLoop::TYPE_IO, 0)));
disk_cache::BackendFlags flags =
- fast ? disk_cache::kNone : disk_cache::kNoRandom;
+ fast ? disk_cache::kNone : disk_cache::kNoRandom;
CreateBackend(flags, &cache_thread);
disk_cache::Entry* entry;
@@ -840,7 +850,7 @@ TEST_F(DiskCacheBackendTest, ShaderCacheLoad) {
// Tests the chaining of an entry to the current head.
void DiskCacheBackendTest::BackendChain() {
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
InitCache();
@@ -1297,7 +1307,7 @@ void DiskCacheBackendTest::BackendEnumerations2() {
InitCache();
const std::string first("first");
const std::string second("second");
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
ASSERT_EQ(net::OK, CreateEntry(first, &entry1));
entry1->Close();
ASSERT_EQ(net::OK, CreateEntry(second, &entry2));
@@ -1368,7 +1378,7 @@ TEST_F(DiskCacheBackendTest, ShaderCacheEnumerationReadData) {
InitCache();
const std::string first("first");
const std::string second("second");
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
const int kSize = 50;
scoped_refptr<net::IOBuffer> buffer1(new net::IOBuffer(kSize));
@@ -1403,7 +1413,7 @@ void DiskCacheBackendTest::BackendInvalidEntryEnumeration() {
InitCache();
std::string key("Some key");
- disk_cache::Entry *entry, *entry1, *entry2;
+ disk_cache::Entry* entry, *entry1, *entry2;
ASSERT_EQ(net::OK, CreateEntry(key, &entry1));
const int kSize = 50;
@@ -1463,7 +1473,7 @@ void DiskCacheBackendTest::BackendFixEnumerators() {
}
EXPECT_EQ(kNumEntries, cache_->GetEntryCount());
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
void* iter1 = NULL;
void* iter2 = NULL;
ASSERT_EQ(net::OK, OpenNextEntry(&iter1, &entry1));
@@ -1514,7 +1524,7 @@ TEST_F(DiskCacheBackendTest, NewEvictionFixEnumerators) {
void DiskCacheBackendTest::BackendDoomRecent() {
InitCache();
- disk_cache::Entry *entry;
+ disk_cache::Entry* entry;
ASSERT_EQ(net::OK, CreateEntry("first", &entry));
entry->Close();
ASSERT_EQ(net::OK, CreateEntry("second", &entry));
@@ -1592,7 +1602,7 @@ TEST_F(DiskCacheBackendTest, DoomAllSparse) {
void DiskCacheBackendTest::BackendDoomBetween() {
InitCache();
- disk_cache::Entry *entry;
+ disk_cache::Entry* entry;
ASSERT_EQ(net::OK, CreateEntry("first", &entry));
entry->Close();
FlushQueueForTest();
@@ -1673,7 +1683,8 @@ TEST_F(DiskCacheBackendTest, DoomEntriesBetweenSparse) {
}
void DiskCacheBackendTest::BackendTransaction(const std::string& name,
- int num_entries, bool load) {
+ int num_entries,
+ bool load) {
success_ = false;
ASSERT_TRUE(CopyTestCache(name));
DisableFirstCleanup();
@@ -1847,7 +1858,7 @@ class BadEntropyProvider : public base::FieldTrial::EntropyProvider {
TEST_F(DiskCacheTest, SimpleCacheControlJoin) {
base::Thread cache_thread("CacheThread");
ASSERT_TRUE(cache_thread.StartWithOptions(
- base::Thread::Options(base::MessageLoop::TYPE_IO, 0)));
+ base::Thread::Options(base::MessageLoop::TYPE_IO, 0)));
scoped_ptr<disk_cache::BackendImpl> cache =
CreateExistingEntryCache(cache_thread, cache_path_);
@@ -1887,7 +1898,7 @@ TEST_F(DiskCacheTest, SimpleCacheControlRestart) {
base::Thread cache_thread("CacheThread");
ASSERT_TRUE(cache_thread.StartWithOptions(
- base::Thread::Options(base::MessageLoop::TYPE_IO, 0)));
+ base::Thread::Options(base::MessageLoop::TYPE_IO, 0)));
scoped_ptr<disk_cache::BackendImpl> cache =
CreateExistingEntryCache(cache_thread, cache_path_);
@@ -1990,7 +2001,7 @@ void DiskCacheBackendTest::BackendInvalidEntry2() {
DisableFirstCleanup();
InitCache();
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
ASSERT_EQ(net::OK, OpenEntry("the first key", &entry1));
EXPECT_NE(net::OK, OpenEntry("some other key", &entry2));
entry1->Close();
@@ -2010,7 +2021,7 @@ TEST_F(DiskCacheBackendTest, NewEvictionInvalidEntry2) {
// Tests that we don't crash or hang when enumerating this cache.
void DiskCacheBackendTest::BackendInvalidEntry3() {
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
DisableFirstCleanup();
InitCache();
@@ -2038,7 +2049,7 @@ TEST_F(DiskCacheBackendTest, NewEvictionInvalidEntry3) {
// the same key, and with hash collisions.
TEST_F(DiskCacheBackendTest, InvalidEntry4) {
ASSERT_TRUE(CopyTestCache("dirty_entry3"));
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
DisableFirstCleanup();
InitCache();
@@ -2051,7 +2062,7 @@ TEST_F(DiskCacheBackendTest, InvalidEntry4) {
TEST_F(DiskCacheBackendTest, InvalidEntry5) {
ASSERT_TRUE(CopyTestCache("dirty_entry4"));
SetNewEviction();
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
DisableFirstCleanup();
InitCache();
@@ -2061,7 +2072,7 @@ TEST_F(DiskCacheBackendTest, InvalidEntry5) {
TEST_F(DiskCacheBackendTest, InvalidEntry6) {
ASSERT_TRUE(CopyTestCache("dirty_entry5"));
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
DisableFirstCleanup();
InitCache();
@@ -2078,7 +2089,7 @@ TEST_F(DiskCacheBackendTest, InvalidEntry6) {
// The test cache could be a result of bug 69135.
TEST_F(DiskCacheBackendTest, BadNextEntry1) {
ASSERT_TRUE(CopyTestCache("list_loop2"));
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
DisableFirstCleanup();
InitCache();
@@ -2101,7 +2112,7 @@ TEST_F(DiskCacheBackendTest, BadNextEntry1) {
// The test cache could be a result of bug 69135.
TEST_F(DiskCacheBackendTest, BadNextEntry2) {
ASSERT_TRUE(CopyTestCache("list_loop3"));
- SetMask(0x1); // 2-entry table.
+ SetMask(0x1); // 2-entry table.
SetMaxSize(0x3000); // 12 kB.
DisableFirstCleanup();
InitCache();
@@ -2466,7 +2477,7 @@ void DiskCacheBackendTest::BackendInvalidRankings2() {
DisableFirstCleanup();
InitCache();
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
EXPECT_NE(net::OK, OpenEntry("the first key", &entry1));
ASSERT_EQ(net::OK, OpenEntry("some other key", &entry2));
entry2->Close();
@@ -2531,7 +2542,7 @@ TEST_F(DiskCacheBackendTest, NewEvictionInvalidRankingsFailure) {
// If the LRU is corrupt and we have open entries, we disable the cache.
void DiskCacheBackendTest::BackendDisable() {
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
void* iter = NULL;
ASSERT_EQ(net::OK, OpenNextEntry(&iter, &entry1));
@@ -2630,7 +2641,7 @@ TEST_F(DiskCacheBackendTest, NewEvictionDisableFailure2) {
// If the index size changes when we disable the cache, we should not crash.
void DiskCacheBackendTest::BackendDisable3() {
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
void* iter = NULL;
EXPECT_EQ(2, cache_->GetEntryCount());
ASSERT_EQ(net::OK, OpenNextEntry(&iter, &entry1));
@@ -2664,7 +2675,7 @@ TEST_F(DiskCacheBackendTest, NewEvictionDisableSuccess3) {
// If we disable the cache, already open entries should work as far as possible.
void DiskCacheBackendTest::BackendDisable4() {
- disk_cache::Entry *entry1, *entry2, *entry3, *entry4;
+ disk_cache::Entry* entry1, *entry2, *entry3, *entry4;
void* iter = NULL;
ASSERT_EQ(net::OK, OpenNextEntry(&iter, &entry1));
@@ -2788,7 +2799,7 @@ TEST_F(DiskCacheBackendTest, Backend_UsageStats) {
void DiskCacheBackendTest::BackendDoomAll() {
InitCache();
- disk_cache::Entry *entry1, *entry2;
+ disk_cache::Entry* entry1, *entry2;
ASSERT_EQ(net::OK, CreateEntry("first", &entry1));
ASSERT_EQ(net::OK, CreateEntry("second", &entry2));
entry1->Close();
@@ -2804,7 +2815,7 @@ void DiskCacheBackendTest::BackendDoomAll() {
// We should stop posting tasks at some point (if we post any).
base::MessageLoop::current()->RunUntilIdle();
- disk_cache::Entry *entry3, *entry4;
+ disk_cache::Entry* entry3, *entry4;
EXPECT_NE(net::OK, OpenEntry("third", &entry3));
ASSERT_EQ(net::OK, CreateEntry("third", &entry3));
ASSERT_EQ(net::OK, CreateEntry("fourth", &entry4));
@@ -2952,8 +2963,7 @@ TEST_F(DiskCacheTest, AutomaticMaxSize) {
disk_cache::PreferredCacheSize(large_size * 10 - 1));
// Region 3: expected = available * 0.1
- EXPECT_EQ(kDefaultCacheSize,
- disk_cache::PreferredCacheSize(large_size * 10));
+ EXPECT_EQ(kDefaultCacheSize, disk_cache::PreferredCacheSize(large_size * 10));
EXPECT_EQ((kDefaultCacheSize * 25 - 1) / 10,
disk_cache::PreferredCacheSize(large_size * 25 - 1));
@@ -2975,10 +2985,8 @@ TEST_F(DiskCacheTest, AutomaticMaxSize) {
disk_cache::PreferredCacheSize(largest_size * 100 - 1));
// Region 6: expected = largest possible size
- EXPECT_EQ(largest_size,
- disk_cache::PreferredCacheSize(largest_size * 100));
- EXPECT_EQ(largest_size,
- disk_cache::PreferredCacheSize(largest_size * 10000));
+ EXPECT_EQ(largest_size, disk_cache::PreferredCacheSize(largest_size * 100));
+ EXPECT_EQ(largest_size, disk_cache::PreferredCacheSize(largest_size * 10000));
}
// Tests that we can "migrate" a running instance from one experiment group to
@@ -3073,8 +3081,8 @@ TEST_F(DiskCacheBackendTest, FileSharing) {
EXPECT_FALSE(file2.IsValid());
sharing |= FILE_SHARE_DELETE;
- file2.Set(CreateFile(name.value().c_str(), access, sharing, NULL,
- OPEN_EXISTING, 0, NULL));
+ file2.Set(CreateFile(
+ name.value().c_str(), access, sharing, NULL, OPEN_EXISTING, 0, NULL));
EXPECT_TRUE(file2.IsValid());
#endif
@@ -3219,7 +3227,7 @@ TEST_F(DiskCacheBackendTest, DISABLED_SimpleCacheSetSize) {
// MacOS has a default open file limit of 256 files, which is incompatible with
// this simple cache test.
#if defined(OS_MACOSX)
-#define SIMPLE_MAYBE_MACOS(TestName) DISABLED_ ## TestName
+#define SIMPLE_MAYBE_MACOS(TestName) DISABLED_##TestName
#else
#define SIMPLE_MAYBE_MACOS(TestName) TestName
#endif
@@ -3329,8 +3337,8 @@ TEST_F(DiskCacheBackendTest, SimpleCacheOpenBadFile) {
header.initial_magic_number = GG_UINT64_C(0xbadf00d);
EXPECT_EQ(
implicit_cast<int>(sizeof(header)),
- base::WriteFile(entry_file1_path, reinterpret_cast<char*>(&header),
- sizeof(header)));
+ base::WriteFile(
+ entry_file1_path, reinterpret_cast<char*>(&header), sizeof(header)));
ASSERT_EQ(net::ERR_FAILED, OpenEntry(key, &entry));
}
@@ -3422,14 +3430,11 @@ TEST_F(DiskCacheBackendTest, SimpleCacheEnumerationBasics) {
count = 0;
disk_cache::Entry* entry_opened_before;
ASSERT_EQ(net::OK, OpenEntry(*(key_pool.begin()), &entry_opened_before));
- ASSERT_TRUE(EnumerateAndMatchKeys(key_pool.size()/2,
- &iter,
- &keys_to_match,
- &count));
+ ASSERT_TRUE(EnumerateAndMatchKeys(
+ key_pool.size() / 2, &iter, &keys_to_match, &count));
disk_cache::Entry* entry_opened_middle;
- ASSERT_EQ(net::OK,
- OpenEntry(*(keys_to_match.begin()), &entry_opened_middle));
+ ASSERT_EQ(net::OK, OpenEntry(*(keys_to_match.begin()), &entry_opened_middle));
ASSERT_TRUE(EnumerateAndMatchKeys(-1, &iter, &keys_to_match, &count));
cache_->EndEnumeration(&iter);
entry_opened_before->Close();
@@ -3451,10 +3456,8 @@ TEST_F(DiskCacheBackendTest, SimpleCacheEnumerationWhileDoomed) {
std::set<std::string> keys_to_match(key_pool);
void* iter = NULL;
size_t count = 0;
- ASSERT_TRUE(EnumerateAndMatchKeys(key_pool.size()/2,
- &iter,
- &keys_to_match,
- &count));
+ ASSERT_TRUE(EnumerateAndMatchKeys(
+ key_pool.size() / 2, &iter, &keys_to_match, &count));
std::string key_to_delete = *(keys_to_match.begin());
DoomEntry(key_to_delete);
@@ -3490,8 +3493,8 @@ TEST_F(DiskCacheBackendTest, SimpleCacheEnumerationCorruption) {
ASSERT_EQ(kSize, ReadData(corrupted_entry, 0, 0, buffer.get(), kSize));
corrupted_entry->Close();
- EXPECT_TRUE(disk_cache::simple_util::CreateCorruptFileForTests(
- key, cache_path_));
+ EXPECT_TRUE(
+ disk_cache::simple_util::CreateCorruptFileForTests(key, cache_path_));
EXPECT_EQ(key_pool.size() + 1,
implicit_cast<size_t>(cache_->GetEntryCount()));

Powered by Google App Engine
This is Rietveld 408576698