Remove implicit scoped_refptr operator T*

net/cookies/cookie_monster_perftest.cc

 // Copyright (c) 2012 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 <algorithm>
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/perftimer.h"
 #include "base/string_util.h"
@@ skipping 131 matching lines @@
     cookies.push_back(base::StringPrintf("a%03d=b", i));
   }
 
   SetCookieCallback setCookieCallback;
 
   // Add a bunch of cookies on a single host
   PerfTimeLogger timer("Cookie_monster_add_single_host");
 
   for (std::vector<std::string>::const_iterator it = cookies.begin();
        it != cookies.end(); ++it) {
-    setCookieCallback.SetCookie(cm, kUrlGoogle, *it);
+    setCookieCallback.SetCookie(cm.get(), kUrlGoogle, *it);
   }
   timer.Done();
 
   GetCookiesCallback getCookiesCallback;
 
   PerfTimeLogger timer2("Cookie_monster_query_single_host");
   for (std::vector<std::string>::const_iterator it = cookies.begin();
        it != cookies.end(); ++it) {
-    getCookiesCallback.GetCookies(cm, kUrlGoogle);
+    getCookiesCallback.GetCookies(cm.get(), kUrlGoogle);
   }
   timer2.Done();
 
   PerfTimeLogger timer3("Cookie_monster_deleteall_single_host");
   cm->DeleteAllAsync(CookieMonster::DeleteCallback());
   MessageLoop::current()->RunAllPending();
   timer3.Done();
 }
 
 TEST_F(CookieMonsterTest, TestAddCookieOnManyHosts) {
   scoped_refptr<CookieMonster> cm(new CookieMonster(NULL, NULL));
   std::string cookie(kCookieLine);
   std::vector<GURL> gurls;  // just wanna have ffffuunnn
   for (int i = 0; i < kNumCookies; ++i) {
     gurls.push_back(GURL(base::StringPrintf("https://a%04d.izzle", i)));
   }
 
   SetCookieCallback setCookieCallback;
 
   // Add a cookie on a bunch of host
   PerfTimeLogger timer("Cookie_monster_add_many_hosts");
   for (std::vector<GURL>::const_iterator it = gurls.begin();
        it != gurls.end(); ++it) {
-    setCookieCallback.SetCookie(cm, *it, cookie);
+    setCookieCallback.SetCookie(cm.get(), *it, cookie);
   }
   timer.Done();
 
   GetCookiesCallback getCookiesCallback;
 
   PerfTimeLogger timer2("Cookie_monster_query_many_hosts");
   for (std::vector<GURL>::const_iterator it = gurls.begin();
        it != gurls.end(); ++it) {
-    getCookiesCallback.GetCookies(cm, *it);
+    getCookiesCallback.GetCookies(cm.get(), *it);
   }
   timer2.Done();
 
   PerfTimeLogger timer3("Cookie_monster_deleteall_many_hosts");
   cm->DeleteAllAsync(CookieMonster::DeleteCallback());
   MessageLoop::current()->RunAllPending();
   timer3.Done();
 }
 
 TEST_F(CookieMonsterTest, TestGetCookiesWithInfo) {
   scoped_refptr<CookieMonster> cm(new CookieMonster(NULL, NULL));
 
   std::vector<GURL> gurls;
   for (int i = 0; i < kNumCookies; ++i)
     gurls.push_back(GURL(base::StringPrintf("https://a%04d.izzle", i)));
 
   SetCookieCallback setCookieCallback;
 
   for (std::vector<GURL>::const_iterator it = gurls.begin();
        it != gurls.end(); ++it) {
-    setCookieCallback.SetCookie(cm, *it, kCookieLine);
+    setCookieCallback.SetCookie(cm.get(), *it, kCookieLine);
   }
 
   GetCookiesWithInfoCallback getCookiesCallback;
 
   PerfTimeLogger timer("Cookie_monster_get_cookie_info");
   for (std::vector<GURL>::const_iterator it = gurls.begin();
        it != gurls.end(); ++it) {
-    getCookiesCallback.GetCookiesWithInfo(cm, *it);
+    getCookiesCallback.GetCookiesWithInfo(cm.get(), *it);
   }
   timer.Done();
 }
 
 static int CountInString(const std::string& str, char c) {
   return std::count(str.begin(), str.end(), c);
 }
 
 TEST_F(CookieMonsterTest, TestDomainTree) {
   scoped_refptr<CookieMonster> cm(new CookieMonster(NULL, NULL));
@@ skipping 27 matching lines @@
     }
   }
 
 
   EXPECT_EQ(31u, domain_list.size());
   for (std::vector<std::string>::const_iterator it = domain_list.begin();
        it != domain_list.end(); it++) {
     GURL gurl("https://" + *it + "/");
     const std::string cookie = base::StringPrintf(domain_cookie_format_tree,
                                                   it->c_str());
-    setCookieCallback.SetCookie(cm, gurl, cookie);
+    setCookieCallback.SetCookie(cm.get(), gurl, cookie);
   }
   EXPECT_EQ(31u, cm->GetAllCookies().size());
 
   GURL probe_gurl("https://b.a.b.a.top.com/");
-  std::string cookie_line = getCookiesCallback.GetCookies(cm, probe_gurl);
+  std::string cookie_line = getCookiesCallback.GetCookies(cm.get(), probe_gurl);
   EXPECT_EQ(5, CountInString(cookie_line, '=')) << "Cookie line: " <<
       cookie_line;
   PerfTimeLogger timer("Cookie_monster_query_domain_tree");
   for (int i = 0; i < kNumCookies; i++) {
-    getCookiesCallback.GetCookies(cm, probe_gurl);
+    getCookiesCallback.GetCookies(cm.get(), probe_gurl);
   }
   timer.Done();
 }
 
 TEST_F(CookieMonsterTest, TestDomainLine) {
   scoped_refptr<CookieMonster> cm(new CookieMonster(NULL, NULL));
   SetCookieCallback setCookieCallback;
   GetCookiesCallback getCookiesCallback;
   std::vector<std::string> domain_list;
   GURL probe_gurl("https://b.a.b.a.top.com/");
@@ skipping 11 matching lines @@
   domain_list.push_back("b.a.b.a.top.com");
   EXPECT_EQ(4u, domain_list.size());
 
   const char* domain_cookie_format_line = "a%03d=b; domain=%s";
   for (int i = 0; i < 8; i++) {
     for (std::vector<std::string>::const_iterator it = domain_list.begin();
          it != domain_list.end(); it++) {
       GURL gurl("https://" + *it + "/");
       const std::string cookie = base::StringPrintf(domain_cookie_format_line,
                                                     i, it->c_str());
-      setCookieCallback.SetCookie(cm, gurl, cookie);
+      setCookieCallback.SetCookie(cm.get(), gurl, cookie);
     }
   }
 
-  cookie_line = getCookiesCallback.GetCookies(cm, probe_gurl);
+  cookie_line = getCookiesCallback.GetCookies(cm.get(), probe_gurl);
   EXPECT_EQ(32, CountInString(cookie_line, '='));
   PerfTimeLogger timer2("Cookie_monster_query_domain_line");
   for (int i = 0; i < kNumCookies; i++) {
-    getCookiesCallback.GetCookies(cm, probe_gurl);
+    getCookiesCallback.GetCookies(cm.get(), probe_gurl);
   }
   timer2.Done();
 }
 
 TEST_F(CookieMonsterTest, TestImport) {
   scoped_refptr<MockPersistentCookieStore> store(new MockPersistentCookieStore);
   std::vector<CanonicalCookie*> initial_cookies;
   GetCookiesCallback getCookiesCallback;
 
   // We want to setup a fairly large backing store, with 300 domains of 50
   // cookies each.  Creation times must be unique.
   int64 time_tick(base::Time::Now().ToInternalValue());
 
   for (int domain_num = 0; domain_num < 300; domain_num++) {
     std::string domain_name(base::StringPrintf(".Domain_%d.com", domain_num));
     std::string gurl("www" + domain_name);
     for (int cookie_num = 0; cookie_num < 50; cookie_num++) {
       std::string cookie_line(base::StringPrintf("Cookie_%d=1; Path=/",
                                                  cookie_num));
       AddCookieToList(gurl, cookie_line,
                       base::Time::FromInternalValue(time_tick++),
                       &initial_cookies);
     }
   }
 
   store->SetLoadExpectation(true, initial_cookies);
 
-  scoped_refptr<CookieMonster> cm(new CookieMonster(store, NULL));
+  scoped_refptr<CookieMonster> cm(new CookieMonster(store.get(), NULL));
 
   // Import will happen on first access.
   GURL gurl("www.google.com");
   CookieOptions options;
   PerfTimeLogger timer("Cookie_monster_import_from_store");
-  getCookiesCallback.GetCookies(cm, gurl);
+  getCookiesCallback.GetCookies(cm.get(), gurl);
   timer.Done();
 
   // Just confirm keys were set as expected.
   EXPECT_EQ("domain_1.com", cm->GetKey("www.Domain_1.com"));
 }
 
 TEST_F(CookieMonsterTest, TestGetKey) {
   scoped_refptr<CookieMonster> cm(new CookieMonster(NULL, NULL));
   PerfTimeLogger timer("Cookie_monster_get_key");
   for (int i = 0; i < kNumCookies; i++)
@@ skipping 47 matching lines @@
   for (int ci = 0; ci < static_cast<int>(ARRAYSIZE_UNSAFE(test_cases)); ++ci) {
     const TestCase& test_case(test_cases[ci]);
     scoped_refptr<CookieMonster> cm(
         CreateMonsterFromStoreForGC(
             test_case.num_cookies, test_case.num_old_cookies,
             CookieMonster::kSafeFromGlobalPurgeDays * 2));
 
     GURL gurl("http://google.com");
     std::string cookie_line("z=3");
     // Trigger the Garbage collection we're allowed.
-    setCookieCallback.SetCookie(cm, gurl, cookie_line);
+    setCookieCallback.SetCookie(cm.get(), gurl, cookie_line);
 
     PerfTimeLogger timer((std::string("GC_") + test_case.name).c_str());
     for (int i = 0; i < kNumCookies; i++)
-      setCookieCallback.SetCookie(cm, gurl, cookie_line);
+      setCookieCallback.SetCookie(cm.get(), gurl, cookie_line);
     timer.Done();
   }
 }
 
 }  // namespace