Asynchronous initialization in Simple Index.

net/disk_cache/simple/simple_backend_impl.cc

 // 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.
 
 #include "net/disk_cache/simple/simple_backend_impl.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/location.h"
@@ skipping 27 matching lines @@
 namespace disk_cache {
 
 SimpleBackendImpl::SimpleBackendImpl(
     const FilePath& path,
     int max_bytes,
     net::CacheType type,
     const scoped_refptr<base::TaskRunner>& cache_thread,
     net::NetLog* net_log)
   : path_(path),
     cache_thread_(cache_thread) {
-  index_.reset(new SimpleIndex(cache_thread, path));
+  index_.reset(new SimpleIndex(cache_thread,

[gavinp, 2013/04/11 09:56:32]

Move this into the ctor list.

[felipeg, 2013/04/11 11:25:45]

Done.

[felipeg, 2013/04/11 11:25:45]

Done.

+                               MessageLoopProxy::current(),  // io_thread
+                               path));
+
 }
 
-int SimpleBackendImpl::Init(const CompletionCallback& callback) {
+int SimpleBackendImpl::Init(const CompletionCallback& completition_callback) {
+  InitializeIndexCallback initialize_index_callback =
+      base::Bind(&SimpleBackendImpl::InitializeIndex,
+                 base::Unretained(this),
+                 completition_callback);

[pasko-google - do not use, 2013/04/10 17:51:01]

s/completition_callback/completion_callback/

[felipeg, 2013/04/11 09:41:35]

Done.

   cache_thread_->PostTask(FROM_HERE,
-                          base::Bind(&SimpleBackendImpl::InitializeIndex,
-                                     base::Unretained(this),
-                                     MessageLoopProxy::current(),
-                                     callback));
+                          base::Bind(&SimpleBackendImpl::CreateDirectory,
+                                     MessageLoopProxy::current(),  // io_thread
+                                     path_,
+                                     initialize_index_callback));
   return net::ERR_IO_PENDING;
 }
 
 SimpleBackendImpl::~SimpleBackendImpl() {
-  index_->Cleanup();
+  index_->WriteToDisk();
 }
 
 net::CacheType SimpleBackendImpl::GetCacheType() const {
   return net::DISK_CACHE;
 }
 
 int32 SimpleBackendImpl::GetEntryCount() const {
   NOTIMPLEMENTED();
   return 0;
 }
@@ skipping 50 matching lines @@
 
 void SimpleBackendImpl::GetStats(
     std::vector<std::pair<std::string, std::string> >* stats) {
   NOTIMPLEMENTED();
 }
 
 void SimpleBackendImpl::OnExternalCacheHit(const std::string& key) {
   NOTIMPLEMENTED();
 }
 
-void SimpleBackendImpl::InitializeIndex(MessageLoopProxy* io_thread,
-                                        const CompletionCallback& callback) {
+void SimpleBackendImpl::InitializeIndex(
+    const CompletionCallback& callback, int rv) {
+  if (rv == net::OK)
+    index_->Initialize();
+  callback.Run(rv);
+}
+
+// static
+void SimpleBackendImpl::CreateDirectory(
+    MessageLoopProxy* io_thread,
+    const base::FilePath& path,
+    const InitializeIndexCallback& initialize_index_callback) {
   int rv = net::OK;
-  if (!file_util::PathExists(path_) && !file_util::CreateDirectory(path_)) {
-    LOG(ERROR) << "Simple Cache Backend: failed to create: " << path_.value();
+  if (!file_util::PathExists(path) && !file_util::CreateDirectory(path)) {
+    LOG(ERROR) << "Simple Cache Backend: failed to create: " << path.value();
     rv = net::ERR_FAILED;
-  } else
-    rv = index_->Initialize() ? net::OK : net::ERR_FAILED;
-  io_thread->PostTask(FROM_HERE, base::Bind(callback, rv));
+  }
+
+  io_thread->PostTask(FROM_HERE, base::Bind(initialize_index_callback, rv));
 }
 
 }  // namespace disk_cache