Temporary fix for installer/visitedlink crash (crbug.com/171475)

chrome/browser/visitedlink/visitedlink_event_listener.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 "chrome/browser/visitedlink/visitedlink_event_listener.h"
 
 #include "base/shared_memory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/visitedlink/visitedlink_master_factory.h"
 #include "chrome/common/render_messages.h"
@@ skipping 98 matching lines @@
 };
 
 VisitedLinkEventListener::VisitedLinkEventListener(Profile* profile)
     : profile_(profile) {
   registrar_.Add(this, content::NOTIFICATION_RENDERER_PROCESS_CREATED,
                  content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, content::NOTIFICATION_RENDERER_PROCESS_TERMINATED,
                  content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, content::NOTIFICATION_RENDER_WIDGET_VISIBILITY_CHANGED,
                  content::NotificationService::AllBrowserContextsAndSources());
+  for (content::RenderProcessHost::iterator i(
+         content::RenderProcessHost::AllHostsIterator());
+       !i.IsAtEnd(); i.Advance()) {
+    content::RenderProcessHost* process = i.GetCurrentValue();
+    Profile* rph_profile =
+        Profile::FromBrowserContext(process->GetBrowserContext());
+    if (!profile_->IsSameProfile(rph_profile))
+      continue;
+    updaters_[process->GetID()] =

[brettw, 2013/02/13 21:57:24]

Below in the NOTIFICATION_RENDERER_PROCESS_CREATED we also send the table. Here
we don't. Why don't we need to send the RPH's the table in this case?

[Cait (Slow), 2013/02/14 16:58:13]

We do need to send the table here. This was a mistake on my part. I looked into
sending the table and it looks like this will require changing the init process
quite a bit (the table is not ready yet when the Listener is created).

As it stands, this code isn't doing much, so I've removed it. 

+          make_linked_ptr(new VisitedLinkUpdater(process->GetID()));
+  }
 }
 
 VisitedLinkEventListener::~VisitedLinkEventListener() {
   if (!pending_visited_links_.empty())
     pending_visited_links_.clear();
 }
 
 void VisitedLinkEventListener::NewTable(base::SharedMemory* table_memory) {
   if (!table_memory)
     return;
@@ skipping 84 matching lines @@
       int child_id = widget->GetProcess()->GetID();
       if (updaters_.count(child_id))
         updaters_[child_id]->Update();
       break;
     }
     default:
       NOTREACHED();
       break;
   }
 }