[CacheStorage] Initialize the cache backend immediately upon opening

content/browser/cache_storage/cache_storage_manager_unittest.cc

 // Copyright 2014 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 "content/browser/cache_storage/cache_storage_manager.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <utility>
@@ skipping 632 matching lines @@
   EXPECT_EQ(1u, callback_strings_.size());
   EXPECT_STREQ("foo", callback_strings_[0].c_str());
 }
 
 // With a persistent cache if the client drops its reference to a
 // CacheStorageCache it should be deleted.
 TEST_F(CacheStorageManagerTest, DropReference) {
   EXPECT_TRUE(Open(origin1_, "foo"));
   base::WeakPtr<CacheStorageCache> cache =
       callback_cache_handle_->value()->AsWeakPtr();
+  // Run a cache operation to ensure that the cache has finished initializing so
+  // that when the handle is dropped it can close immediately.
+  EXPECT_FALSE(CacheMatch(callback_cache_handle_->value(),
+                          GURL("http://example.com/foo")));
+
   callback_cache_handle_ = nullptr;
   EXPECT_FALSE(cache);
 }
 
 // A cache continues to work so long as there is a handle to it. Only after the
 // last cache handle is deleted can the cache be freed.
 TEST_P(CacheStorageManagerTestP, CacheWorksAfterDelete) {
   const GURL kFooURL("http://example.com/foo");
   const GURL kBarURL("http://example.com/bar");
   const GURL kBazURL("http://example.com/baz");
@@ skipping 224 matching lines @@
     legacy_path_ = CacheStorageManager::ConstructLegacyOriginPath(
         cache_manager_->root_path(), origin1_);
     new_path_ = CacheStorageManager::ConstructOriginPath(
         cache_manager_->root_path(), origin1_);
 
     ASSERT_FALSE(base::DirectoryExists(legacy_path_));
     ASSERT_FALSE(base::DirectoryExists(new_path_));
     ASSERT_TRUE(Open(origin1_, cache1_));
     ASSERT_TRUE(Open(origin1_, cache2_));
     callback_cache_handle_.reset();
+
+    // Let the caches shut down.
+    base::RunLoop().RunUntilIdle();
+
     ASSERT_FALSE(base::DirectoryExists(legacy_path_));
     ASSERT_TRUE(base::DirectoryExists(new_path_));
 
     quota_manager_proxy_->SimulateQuotaManagerDestroyed();
     cache_manager_ = CacheStorageManager::Create(cache_manager_.get());
 
     ASSERT_TRUE(base::Move(new_path_, legacy_path_));
     ASSERT_TRUE(base::DirectoryExists(legacy_path_));
     ASSERT_FALSE(base::DirectoryExists(new_path_));
   }
@@ skipping 334 matching lines @@
 
 INSTANTIATE_TEST_CASE_P(CacheStorageManagerTests,
                         CacheStorageManagerTestP,
                         ::testing::Values(false, true));
 
 INSTANTIATE_TEST_CASE_P(CacheStorageQuotaClientTests,
                         CacheStorageQuotaClientTestP,
                         ::testing::Values(false, true));
 
 }  // namespace content