Make ComponentInstaller refcounted.

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index c64b40eec5939be7b98de2f62569cac0c4c2de9c..6a719e8896e98c82fbdf22008a54d7de2c9e89cf 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -878,7 +878,7 @@ BrowserProcessImpl::component_updater() {
 }
 
 CRLSetFetcher* BrowserProcessImpl::crl_set_fetcher() {
-  if (!crl_set_fetcher_.get())
+  if (!crl_set_fetcher_)

[Lei Zhang, 2015/02/04 22:21:52]

Can we keep the .get() for
https://code.google.com/p/chromium/issues/detail?id=110610 ?

[Bernhard Bauer, 2015/02/04 22:50:55]

I thought this is already fixed (see https://codereview.chromium.org/759503002),
so we're using the bool conversion here?

[Lei Zhang, 2015/02/04 23:39:57]

Err, yes. n/m.

     crl_set_fetcher_ = new CRLSetFetcher();
   return crl_set_fetcher_.get();
 }
@@ -886,13 +886,13 @@ CRLSetFetcher* BrowserProcessImpl::crl_set_fetcher() {
 component_updater::PnaclComponentInstaller*
 BrowserProcessImpl::pnacl_component_installer() {
 #if !defined(DISABLE_NACL)
-  if (!pnacl_component_installer_.get()) {
-    pnacl_component_installer_.reset(
-        new component_updater::PnaclComponentInstaller());
+  if (!pnacl_component_installer_) {
+    pnacl_component_installer_ =
+        new component_updater::PnaclComponentInstaller();
   }
   return pnacl_component_installer_.get();
 #else
-  return NULL;
+  return nullptr;
 #endif
 }