[CacheStorage] Give ownership of all CacheStorageCaches to CacheStorage

content/browser/cache_storage/cache_storage_cache_handle.cc

Index: content/browser/cache_storage/cache_storage_cache_handle.cc
diff --git a/content/browser/cache_storage/cache_storage_cache_handle.cc b/content/browser/cache_storage/cache_storage_cache_handle.cc
new file mode 100644
index 0000000000000000000000000000000000000000..12ff51e48c109645f8669d61cf8e5d7cb572728f
--- /dev/null
+++ b/content/browser/cache_storage/cache_storage_cache_handle.cc
@@ -0,0 +1,29 @@
+// Copyright 2016 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_cache_handle.h"
+
+namespace content {
+
+CacheStorageCacheHandle::~CacheStorageCacheHandle() {
+  if (cache_storage_ && cache_storage_cache_)
+    cache_storage_->DropCacheHandleRef(cache_storage_cache_.get());
+}
+
+std::unique_ptr<CacheStorageCacheHandle> CacheStorageCacheHandle::Clone() {
+  return std::unique_ptr<CacheStorageCacheHandle>(
+      new CacheStorageCacheHandle(cache_storage_cache_, cache_storage_));
+}
+
+CacheStorageCacheHandle::CacheStorageCacheHandle(
+    base::WeakPtr<CacheStorageCache> cache_storage_cache,
+    base::WeakPtr<CacheStorage> cache_storage)
+    : cache_storage_cache_(cache_storage_cache), cache_storage_(cache_storage) {
+  DCHECK(cache_storage);
+  DCHECK(cache_storage_cache_);
+  if (cache_storage_cache_)

[nhiroki, 2016/06/14 05:30:24]

We should not handle the DCHECK failure case:

"A consequence of this is that you should not handle DCHECK() failures, even if
failure would result in a crash."
http://www.chromium.org/developers/coding-style#TOC-CHECK-DCHECK-and-NOTREACHED-

[jkarlin, 2016/06/14 06:41:08]

Good catch, thanks! Done.

+    cache_storage_->AddCacheHandleRef(cache_storage_cache_.get());
+}
+
+}  // namespace content