Provide a way for RenderProcessHosts to register themselves in the global host map.

content/browser/renderer_host/render_process_host_impl.cc

 // Copyright (c) 2011 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.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 
 #include <algorithm>
@@ skipping 231 matching lines @@
     return false;
 
   if (ChildProcessSecurityPolicy::GetInstance()->HasWebUIBindings(
           host->GetID()) !=
       content::WebUIFactory::Get()->HasWebUIScheme(site_url))
     return false;
 
   return content::GetContentClient()->browser()->IsSuitableHost(host, site_url);
 }
 
-}  // namespace
-
 // the global list of all renderer processes
 base::LazyInstance<
     IDMap<content::RenderProcessHost>,
     base::LeakyLazyInstanceTraits<IDMap<content::RenderProcessHost> > >
         g_all_hosts = LAZY_INSTANCE_INITIALIZER;
 
+}  // namespace
+
 // static
 bool g_run_renderer_in_process_ = false;
 
 // static
 void content::RenderProcessHost::SetMaxRendererProcessCountForTest(
     size_t count) {
   max_renderer_count_override = count;
 }
 
 RenderProcessHostImpl::RenderProcessHostImpl(
@@ skipping 46 matching lines @@
       base::PLATFORM_FILE_WRITE_ATTRIBUTES | base::PLATFORM_FILE_ENUMERATE);
   // This is so that we can rename the old sandbox out of the way so that we
   // know we've taken care of it.
   ChildProcessSecurityPolicy::GetInstance()->GrantPermissionsForFile(
       GetID(), browser_context->GetPath().Append(
           fileapi::SandboxMountPointProvider::kRenamedOldFileSystemDirectory),
       base::PLATFORM_FILE_CREATE | base::PLATFORM_FILE_CREATE_ALWAYS |
       base::PLATFORM_FILE_WRITE);
 
   CHECK(!content::ExitedMainMessageLoop());
-  g_all_hosts.Get().AddWithID(this, GetID());
+  RegisterHost(GetID(), this);
   g_all_hosts.Get().set_check_on_null_data(true);
   // Initialize |child_process_activity_time_| to a reasonable value.
   mark_child_process_activity_time();
   // Note: When we create the RenderProcessHostImpl, it's technically
   //       backgrounded, because it has no visible listeners.  But the process
   //       doesn't actually exist yet, so we'll Background it later, after
   //       creation.
 }
 
 RenderProcessHostImpl::~RenderProcessHostImpl() {
   ChildProcessSecurityPolicy::GetInstance()->Remove(GetID());
 
   // We may have some unsent messages at this point, but that's OK.
   channel_.reset();
   while (!queued_messages_.empty()) {
     delete queued_messages_.front();
     queued_messages_.pop();
   }
 
   ClearTransportDIBCache();
-  if (g_all_hosts.Get().Lookup(GetID()))
-    g_all_hosts.Get().Remove(GetID());
+  UnregisterHost(GetID());
 }
 
 void RenderProcessHostImpl::EnableSendQueue() {
   is_initialized_ = false;
 }
 
 bool RenderProcessHostImpl::Init(bool is_accessibility_enabled) {
   // calling Init() more than once does nothing, this makes it more convenient
   // for the view host which may not be sure in some cases
   if (channel_.get())
@@ skipping 738 matching lines @@
   if (listeners_.size() == count)
     return FastShutdownIfPossible();
   return false;
 }
 
 bool RenderProcessHostImpl::FastShutdownStarted() const {
   return fast_shutdown_started_;
 }
 
 // static
+void RenderProcessHostImpl::RegisterHost(int host_id,
+                                         content::RenderProcessHost* host) {
+  g_all_hosts.Get().AddWithID(host, host_id);
+}
+
+void RenderProcessHostImpl::UnregisterHost(int host_id) {
+  if (g_all_hosts.Get().Lookup(host_id))
+    g_all_hosts.Get().Remove(host_id);
+}
+
+// static
 bool content::RenderProcessHost::run_renderer_in_process() {
   return g_run_renderer_in_process_;
 }
 
 void content::RenderProcessHost::set_run_renderer_in_process(bool value) {
   g_run_renderer_in_process_ = value;
 }
 
 content::RenderProcessHost::iterator
     content::RenderProcessHost::AllHostsIterator() {
@@ skipping 177 matching lines @@
 void RenderProcessHostImpl::OnRevealFolderInOS(const FilePath& path) {
   // Only honor the request if appropriate persmissions are granted.
   if (ChildProcessSecurityPolicy::GetInstance()->CanReadFile(GetID(), path))
     content::GetContentClient()->browser()->OpenItem(path);
 }
 
 void RenderProcessHostImpl::OnSavedPageAsMHTML(int job_id, int64 data_size) {
   content::GetContentClient()->browser()->GetMHTMLGenerationManager()->
       MHTMLGenerated(job_id, data_size);
 }