Simple Cache: DoomEntriesBetween()

net/disk_cache/simple/simple_backend_impl.h

 // Copyright (c) 2013 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.
 
 #ifndef NET_DISK_CACHE_SIMPLE_SIMPLE_BACKEND_IMPL_H_
 #define NET_DISK_CACHE_SIMPLE_SIMPLE_BACKEND_IMPL_H_
 
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 13 matching lines @@
 // SimpleBackendImpl is a new cache backend that stores entries in individual
 // files.
 
 // It is currently a work in progress, missing many features of a real cache,
 // such as eviction.
 
 // See http://www.chromium.org/developers/design-documents/network-stack/disk-cache/very-simple-backend
 
 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,
                     const scoped_refptr<base::TaskRunner>& cache_thread,
                     net::NetLog* net_log);
 
   virtual ~SimpleBackendImpl();
 
   // Must run on IO Thread.
   int Init(const CompletionCallback& completion_callback);
@@ skipping 19 matching lines @@
   virtual void GetStats(
       std::vector<std::pair<std::string, std::string> >* stats) OVERRIDE;
   virtual void OnExternalCacheHit(const std::string& key) OVERRIDE;
 
  private:
   typedef base::Callback<void(int result)> InitializeIndexCallback;
 
   // 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(
       base::MessageLoopProxy* io_thread,
       const base::FilePath& path,
       const InitializeIndexCallback& initialize_index_callback);
 
   const base::FilePath path_;
   scoped_ptr<SimpleIndex> index_;
   const scoped_refptr<base::TaskRunner> cache_thread_;
 };
 
 }  // namespace disk_cache
 
 #endif  // NET_DISK_CACHE_SIMPLE_SIMPLE_BACKEND_IMPL_H_