Make SimpleEntryImpl ref counted.

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..eab0164521d9ba12497bc4bdcfda030a2d334920 100644
--- a/net/disk_cache/simple/simple_backend_impl.h
+++ b/net/disk_cache/simple/simple_backend_impl.h
@@ -11,9 +11,11 @@
 
 #include "base/compiler_specific.h"
 #include "base/files/file_path.h"
+#include "base/memory/ref_counted.h"
 #include "base/task_runner.h"
 #include "net/base/cache_type.h"
 #include "net/disk_cache/disk_cache.h"
+#include "net/disk_cache/simple/simple_index.h"
 
 namespace base {
 class MessageLoopProxy;
@@ -29,8 +31,6 @@ namespace disk_cache {
 
 // See http://www.chromium.org/developers/design-documents/network-stack/disk-cache/very-simple-backend
 
-class SimpleIndex;
-
 class NET_EXPORT_PRIVATE SimpleBackendImpl : public Backend {
  public:
   SimpleBackendImpl(const base::FilePath& path, int max_bytes,
@@ -79,7 +79,7 @@ class NET_EXPORT_PRIVATE SimpleBackendImpl : public Backend {
       const InitializeIndexCallback& initialize_index_callback);
 
   const base::FilePath path_;
-  scoped_ptr<SimpleIndex> index_;
+  scoped_refptr<SimpleIndex> index_;

[rvargas (doing something else), 2013/04/17 19:56:38]

Isn't another way to handle the index? refcounting muddles the ownership and
lifetime of objects so it would be better if the  index has a single and clearly
defined owner (and lifetime determined by the backend).

   const scoped_refptr<base::TaskRunner> cache_thread_;
 };