Mandate unique_ptr for base::IDMap in IDMapOwnPointer mode.

content/browser/cache_storage/cache_storage_cache.cc

Index: content/browser/cache_storage/cache_storage_cache.cc
diff --git a/content/browser/cache_storage/cache_storage_cache.cc b/content/browser/cache_storage/cache_storage_cache.cc
index 6ca5786b882581b11095984e4565ee02f7e691fc..52dbd039d935868b0765f55946fc68d8a7444af5 100644
--- a/content/browser/cache_storage/cache_storage_cache.cc
+++ b/content/browser/cache_storage/cache_storage_cache.cc
@@ -1133,7 +1133,8 @@ void CacheStorageCache::PutDidWriteHeaders(
   CacheStorageBlobToDiskCache* blob_to_cache =
       new CacheStorageBlobToDiskCache();
   BlobToDiskCacheIDMap::KeyType blob_to_cache_key =
-      active_blob_to_disk_cache_writers_.Add(blob_to_cache);
+      active_blob_to_disk_cache_writers_.Add(
+          std::unique_ptr<CacheStorageBlobToDiskCache>(blob_to_cache));

[aelias_OOO_until_Jul13, 2016/11/10 06:04:48]

base::MakeUnique<CacheStorageBlobToDiskCache>

[rlanday, 2016/11/10 18:20:01]

This is another weird one, blob_to_cache is inserted into the IDMap, then used
again below in an expression that depends on the key it was assigned. I think I
might be able to insert into the IDMap and then immediately look it up again, is
that preferable?

 
   std::unique_ptr<storage::BlobDataHandle> blob_data_handle =
       std::move(put_context->blob_data_handle);