Convert implicit scoped_refptr constructor calls to explicit ones, part 2

chrome/browser/memory_purger.cc

 // Copyright (c) 2009 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 "chrome/browser/memory_purger.h"
 
 #include <set>
 
 #include "base/thread.h"
 #include "chrome/browser/browser_list.h"
@@ skipping 19 matching lines @@
 // This is a small helper class used to ensure that the objects we want to use
 // on multiple threads are properly refed, so they don't get deleted out from
 // under us.
 class PurgeMemoryIOHelper
     : public base::RefCountedThreadSafe<PurgeMemoryIOHelper> {
  public:
   explicit PurgeMemoryIOHelper(SafeBrowsingService* safe_browsing_service)
       : safe_browsing_service_(safe_browsing_service) {
   }
 
-  void AddRequestContextGetter(URLRequestContextGetter* request_context_getter);
+  void AddRequestContextGetter(
+      scoped_refptr<URLRequestContextGetter> request_context_getter);
 
   void PurgeMemoryOnIOThread();
 
  private:
   typedef scoped_refptr<URLRequestContextGetter> RequestContextGetter;
   typedef std::set<RequestContextGetter> RequestContextGetters;
 
   RequestContextGetters request_context_getters_;
   scoped_refptr<SafeBrowsingService> safe_browsing_service_;
 
   DISALLOW_COPY_AND_ASSIGN(PurgeMemoryIOHelper);
 };
 
 void PurgeMemoryIOHelper::AddRequestContextGetter(
-    URLRequestContextGetter* request_context_getter) {
-  if (!request_context_getters_.count(request_context_getter)) {
-    request_context_getters_.insert(
-        RequestContextGetter(request_context_getter));
-  }
+    scoped_refptr<URLRequestContextGetter> request_context_getter) {
+  request_context_getters_.insert(request_context_getter);
 }
 
 void PurgeMemoryIOHelper::PurgeMemoryOnIOThread() {
   // Ask ProxyServices to purge any memory they can (generally garbage in the
   // wrapped ProxyResolver's JS engine).
   for (RequestContextGetters::const_iterator i(
            request_context_getters_.begin());
        i != request_context_getters_.end(); ++i)
     (*i)->GetURLRequestContext()->proxy_service()->PurgeMemory();
 
@@ skipping 27 matching lines @@
 
   // Per-profile cleanup.
   scoped_refptr<PurgeMemoryIOHelper> purge_memory_io_helper(
       new PurgeMemoryIOHelper(g_browser_process->resource_dispatcher_host()->
           safe_browsing_service()));
   ProfileManager* profile_manager = g_browser_process->profile_manager();
   for (ProfileManager::iterator i(profile_manager->begin());
        i != profile_manager->end(); ++i) {
     Profile* profile = *i;
     purge_memory_io_helper->AddRequestContextGetter(
-        profile->GetRequestContext());
+        make_scoped_refptr(profile->GetRequestContext()));
 
     // NOTE: Some objects below may be duplicates across profiles.  We could
     // conceivably put all these in sets and then iterate over the sets.
 
     // Unload all history backends (freeing memory used to cache sqlite).
     // Spinning up the history service is expensive, so we avoid doing it if it
     // hasn't been done already.
     HistoryService* history_service =
         profile->GetHistoryServiceWithoutCreating();
     if (history_service)
@@ skipping 39 matching lines @@
   for (RenderProcessHost::iterator i(RenderProcessHost::AllHostsIterator());
        !i.IsAtEnd(); i.Advance())
     PurgeRendererForHost(i.GetCurrentValue());
 }
 
 // static
 void MemoryPurger::PurgeRendererForHost(RenderProcessHost* host) {
   // Direct the renderer to free everything it can.
   host->Send(new ViewMsg_PurgeMemory());
 }