net/disk_cache/simple: Reduce size of EntrySet nodes.

net/disk_cache/simple/simple_index_file_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/hash.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/pickle.h"
 #include "base/stringprintf.h"
 #include "base/time.h"
@@ skipping 34 matching lines @@
   EXPECT_EQ(new_index_metadata.GetNumberOfEntries(),
             index_metadata.GetNumberOfEntries());
   EXPECT_EQ(new_index_metadata.cache_size_, index_metadata.cache_size_);
 
   EXPECT_TRUE(new_index_metadata.CheckIndexMetadata());
 }
 
 class SimpleIndexFileTest : public testing::Test {
  public:
   bool CompareTwoEntryMetadata(const EntryMetadata& a, const EntryMetadata& b) {
-    return a.hash_key_ == b.hash_key_ &&
-        a.last_used_time_ == b.last_used_time_ &&
-        a.entry_size_ == b.entry_size_;
+    return a.last_used_time_ == b.last_used_time_ &&
+           a.entry_size_ == b.entry_size_;
   }
 
 };
 
 TEST_F(SimpleIndexFileTest, Serialize) {
   SimpleIndex::EntrySet entries;
+  uint64 hashes_array[] = { 11, 22, 33 };
   EntryMetadata entries_array[] = {

[pasko, 2013/05/16 18:13:21]

Since we have hashes_array now, to avoid repeating the numbers it would be
better to create a vector of EntryMetadata in the loop below, and use that.

[digit1, 2013/05/17 08:45:38]

Done.

-    EntryMetadata(11, Time::FromInternalValue(22), 33),
-    EntryMetadata(22, Time::FromInternalValue(33), 44),
-    EntryMetadata(33, Time::FromInternalValue(44), 55)
+    EntryMetadata(Time::FromInternalValue(22), 33),
+    EntryMetadata(Time::FromInternalValue(33), 44),
+    EntryMetadata(Time::FromInternalValue(44), 55)
   };
   SimpleIndexFile::IndexMetadata index_metadata(arraysize(entries_array), 456);
   for (uint32 i = 0; i < arraysize(entries_array); ++i) {
-    SimpleIndex::InsertInEntrySet(entries_array[i], &entries);
+    SimpleIndex::InsertInEntrySet(hashes_array[i], entries_array[i], &entries);
   }
 
   scoped_ptr<Pickle> pickle = SimpleIndexFile::Serialize(
       index_metadata, entries);
   EXPECT_TRUE(pickle.get() != NULL);
 
   scoped_ptr<SimpleIndex::EntrySet> new_entries = SimpleIndexFile::Deserialize(
       reinterpret_cast<const char*>(pickle->data()),
       pickle->size());
   EXPECT_TRUE(new_entries.get() != NULL);
   EXPECT_EQ(entries.size(), new_entries->size());
 
   for (uint32 i = 0; i < arraysize(entries_array); ++i) {
-    SimpleIndex::EntrySet::iterator it =
-        new_entries->find(entries_array[i].GetHashKey());
+    SimpleIndex::EntrySet::iterator it = new_entries->find(hashes_array[i]);
     EXPECT_TRUE(new_entries->end() != it);
     EXPECT_TRUE(CompareTwoEntryMetadata(it->second, entries_array[i]));
   }
 }
 
 }