Disk Cache: Make Stats independent of the backend implementation.

net/disk_cache/addr.h

 // Copyright (c) 2012 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.
 
 // This is an internal class that handles the address of a cache record.
 // See net/disk_cache/disk_cache.h for the public interface of the cache.
 
 #ifndef NET_DISK_CACHE_ADDR_H_
 #define NET_DISK_CACHE_ADDR_H_
 
@@ skipping 109 matching lines @@
     if (size < 1024)
       return BLOCK_256;
     else if (size < 4096)
       return BLOCK_1K;
     else if (size <= 4096 * 4)
       return BLOCK_4K;
     else
       return EXTERNAL;
   }
 
+  static int RequiredBlocks(int size, FileType file_type) {

[gavinp, 2013/05/23 15:32:33]

This long list of inline functions seems not in keeping with our coding style.

[rvargas (doing something else), 2013/05/23 19:39:43]

They are all trivial functions and I want to hint the compiler that they should
be inlined.

+    int block_size = BlockSizeForFileType(file_type);
+    return (size + block_size - 1) / block_size;
+  }
+
   // Returns true if this address looks like a valid one.
   bool SanityCheck() const;
   bool SanityCheckForEntry() const;
   bool SanityCheckForRankings() const;
 
  private:
   static const uint32 kInitializedMask    = 0x80000000;
   static const uint32 kFileTypeMask       = 0x70000000;
   static const uint32 kFileTypeOffset     = 28;
   static const uint32 kNumBlocksMask      = 0x03000000;
   static const uint32 kNumBlocksOffset    = 24;
   static const uint32 kFileSelectorMask   = 0x00ff0000;
   static const uint32 kFileSelectorOffset = 16;
   static const uint32 kStartBlockMask     = 0x0000FFFF;
   static const uint32 kFileNameMask       = 0x0FFFFFFF;
 
   CacheAddr value_;
 };
 
 }  // namespace disk_cache
 
 #endif  // NET_DISK_CACHE_ADDR_H_