Simple Cache: DoomEntriesBetween()

net/disk_cache/simple/simple_backend_impl.h

Index: net/disk_cache/simple/simple_backend_impl.h
diff --git a/net/disk_cache/simple/simple_backend_impl.h b/net/disk_cache/simple/simple_backend_impl.h
index fd5c53a936cc938754acd42c74edd08487ec1c1b..d73a6413243cf496083137d3edc5f8cf182b291f 100644
--- a/net/disk_cache/simple/simple_backend_impl.h
+++ b/net/disk_cache/simple/simple_backend_impl.h
@@ -31,7 +31,8 @@ namespace disk_cache {
 
 class SimpleIndex;
 
-class NET_EXPORT_PRIVATE SimpleBackendImpl : public Backend {
+class NET_EXPORT_PRIVATE SimpleBackendImpl : public Backend,
+  public base::SupportsWeakPtr<SimpleBackendImpl> {

[gavinp, 2013/04/17 07:41:18]

Is this indented correctly?

[pasko-google - do not use, 2013/04/17 19:47:52]

unspecified? I've indented it to 4 spaces, let me know if your preference is
different.

  public:
   SimpleBackendImpl(const base::FilePath& path, int max_bytes,
                     net::CacheType type,
@@ -71,6 +72,12 @@ class NET_EXPORT_PRIVATE SimpleBackendImpl : public Backend {
   // Must run on IO Thread.
   void InitializeIndex(const CompletionCallback& callback, int result);
 
+  // Dooms all entries previously accessed between |initial_time| and
+  // |end_time|. Invoked when the index is ready.
+  void IndexReadyForDoom(const base::Time initial_time,

[gavinp, 2013/04/17 07:41:18]

this const has no meaning. I wouldn't pass a base::Time by reference (but a
grepfight shows that's definitely done), so might as well just use base::Time
here.

[pasko-google - do not use, 2013/04/17 19:47:52]

Sure, I just blindly copied it from somewhere.

Done.

+                         const base::Time end_time,
+                         const CompletionCallback& callback, int result);
+
   // Try to create the directory if it doesn't exist.
   // Must run on Cache Thread.
   static void CreateDirectory(