Convert implicit scoped_refptr constructor calls to explicit ones, part 2

webkit/appcache/mock_appcache_storage.cc

 // Copyright (c) 2010 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 "webkit/appcache/mock_appcache_storage.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/ref_counted.h"
 #include "base/stl_util-inl.h"
@@ skipping 372 matching lines @@
 void MockAppCacheStorage::RunOnePendingTask() {
   DCHECK(!pending_tasks_.empty());
   Task* task = pending_tasks_.front();
   pending_tasks_.pop_front();
   task->Run();
   delete task;
 }
 
 void MockAppCacheStorage::AddStoredCache(AppCache* cache) {
   int64 cache_id = cache->cache_id();
-  if (stored_caches_.find(cache_id) == stored_caches_.end())
-    stored_caches_.insert(StoredCacheMap::value_type(cache_id, cache));
+  if (stored_caches_.find(cache_id) == stored_caches_.end()) {
+    stored_caches_.insert(
+        StoredCacheMap::value_type(cache_id, make_scoped_refptr(cache)));
+  }
 }
 
 void MockAppCacheStorage::RemoveStoredCache(AppCache* cache) {
   // Do not remove from the working set, active caches are still usable
   // and may be looked up by id until they fall out of use.
   stored_caches_.erase(cache->cache_id());
 }
 
 void MockAppCacheStorage::RemoveStoredCaches(
     const AppCacheGroup::Caches& caches) {
   AppCacheGroup::Caches::const_iterator it = caches.begin();
   while (it != caches.end()) {
     RemoveStoredCache(*it);
     ++it;
   }
 }
 
 void MockAppCacheStorage::AddStoredGroup(AppCacheGroup* group) {
   const GURL& url = group->manifest_url();
-  if (stored_groups_.find(url) == stored_groups_.end())
-    stored_groups_.insert(StoredGroupMap::value_type(url, group));
+  if (stored_groups_.find(url) == stored_groups_.end()) {
+    stored_groups_.insert(
+        StoredGroupMap::value_type(url, make_scoped_refptr(group)));
+  }
 }
 
 void MockAppCacheStorage::RemoveStoredGroup(AppCacheGroup* group) {
   stored_groups_.erase(group->manifest_url());
 }
 
 bool MockAppCacheStorage::ShouldGroupLoadAppearAsync(
     const AppCacheGroup* group) {
   // We'll have to query the database to see if a group for the
   // manifest_url exists on disk. So return true for async.
@@ skipping 29 matching lines @@
 bool MockAppCacheStorage::ShouldCacheLoadAppearAsync(const AppCache* cache) {
   if (!cache)
     return true;
 
   // If the 'stored' ref is the only ref, real storage will have to load from
   // the database.
   return IsCacheStored(cache) && cache->HasOneRef();
 }
 
 }  // namespace appcache