Convert implicit scoped_refptr constructor calls to explicit ones, part 1

webkit/appcache/appcache_host_unittest.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 "base/callback.h"
 #include "base/scoped_ptr.h"
 #include "net/url_request/url_request.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webkit/appcache/appcache.h"
 #include "webkit/appcache/appcache_backend_impl.h"
@@ skipping 237 matching lines @@
   // Callback should have fired upon completing the group load.
   EXPECT_EQ(UNCACHED, last_status_result_);
   EXPECT_EQ(reinterpret_cast<void*>(1), last_callback_param_);
 }
 
 TEST_F(AppCacheHostTest, SetSwappableCache) {
   AppCacheHost host(1, &mock_frontend_, &service_);
   host.SetSwappableCache(NULL);
   EXPECT_FALSE(host.swappable_cache_.get());
 
-  scoped_refptr<AppCacheGroup> group1 =
-      new AppCacheGroup(&service_, GURL(), service_.storage()->NewGroupId());
+  scoped_refptr<AppCacheGroup> group1(
+      new AppCacheGroup(&service_, GURL(), service_.storage()->NewGroupId()));
   host.SetSwappableCache(group1);
   EXPECT_FALSE(host.swappable_cache_.get());
 
   AppCache* cache1 = new AppCache(&service_, 111);
   cache1->set_complete(true);
   group1->AddCache(cache1);
   host.SetSwappableCache(group1);
   EXPECT_EQ(cache1, host.swappable_cache_.get());
 
   mock_frontend_.last_host_id_ = -222;  // to verify we received OnCacheSelected
 
   host.AssociateCache(cache1);
   EXPECT_FALSE(host.swappable_cache_.get());  // was same as associated cache
   EXPECT_EQ(appcache::IDLE, host.GetStatus());
   // verify OnCacheSelected was called
   EXPECT_EQ(host.host_id(), mock_frontend_.last_host_id_);
   EXPECT_EQ(cache1->cache_id(), mock_frontend_.last_cache_id_);
   EXPECT_EQ(appcache::IDLE, mock_frontend_.last_status_);
 
   AppCache* cache2 = new AppCache(&service_, 222);
   cache2->set_complete(true);
   group1->AddCache(cache2);
   EXPECT_EQ(cache2, host.swappable_cache_.get());  // updated to newest
 
-  scoped_refptr<AppCacheGroup> group2 =
+  scoped_refptr<AppCacheGroup> group2(
       new AppCacheGroup(&service_, GURL("http://foo.com"),
-                        service_.storage()->NewGroupId());
+                        service_.storage()->NewGroupId()));
   AppCache* cache3 = new AppCache(&service_, 333);
   cache3->set_complete(true);
   group2->AddCache(cache3);
 
   AppCache* cache4 = new AppCache(&service_, 444);
   cache4->set_complete(true);
   group2->AddCache(cache4);
   EXPECT_EQ(cache2, host.swappable_cache_.get());  // unchanged
 
   host.AssociateCache(cache3);
@@ skipping 50 matching lines @@
 
   // Simulate the parent being torn down.
   backend_impl.UnregisterHost(kParentHostId);
   parent_host = NULL;
   EXPECT_EQ(NULL, backend_impl.GetHost(kParentHostId));
   EXPECT_EQ(NULL, worker_host->GetParentAppCacheHost());
 }
 
 }  // namespace appcache