commit 958b5d3ddc11c53cf8676be0d13fd09d34f42967
author sorin <sorin@chromium.org> Sat Jan 09 02:00:20 2016
committer Commit bot <commit-bot@chromium.org> Sat Jan 09 02:01:07 2016
tree 91cf3c5703e5a98c8a48dba1176ed9ade972aeac
parent 801f0f304e8627cb5f69d35a7af87969c9326630

Change const Configurator& with scoped_refptr<Configurator>.

This is a mechanical refactoring to replace C++ references to Configurator
with refcounted pointers. This change preceeds exposing a public interface
to send an uninstall ping through update_client.

Configurator is a refcounted type to begin with but some old code is still
using it as a C++ reference.

BUG=501641

Review URL: https://codereview.chromium.org/1571983002

Cr-Commit-Position: refs/heads/master@{#368506}

components/update_client/request_sender.h

diff --git a/components/update_client/request_sender.h b/components/update_client/request_sender.h
index 5c05350d..7b9414354 100644
--- a/components/update_client/request_sender.h
+++ b/components/update_client/request_sender.h
@@ -11,6 +11,7 @@
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/thread_checker.h"
 #include "net/url_request/url_fetcher_delegate.h"
@@ -34,7 +35,7 @@
   typedef base::Callback<void(const net::URLFetcher* source)>
       RequestSenderCallback;
 
-  explicit RequestSender(const Configurator& config);
+  explicit RequestSender(const scoped_refptr<Configurator>& config);
   ~RequestSender() override;
 
   void Send(const std::string& request_string,
@@ -47,7 +48,7 @@
   // Overrides for URLFetcherDelegate.
   void OnURLFetchComplete(const net::URLFetcher* source) override;
 
-  const Configurator& config_;
+  const scoped_refptr<Configurator> config_;
   std::vector<GURL> urls_;
   std::vector<GURL>::const_iterator cur_url_;
   scoped_ptr<net::URLFetcher> url_fetcher_;