Refactor RenderProcessHost to use IPC::Listener instead of RenderWidgetHost

chrome/browser/task_manager/guest_resource_provider.cc

 // Copyright 2013 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/task_manager/guest_resource_provider.h"
 
 #include "base/string16.h"
 #include "chrome/browser/favicon/favicon_tab_helper.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/task_manager/renderer_resource.h"
@@ skipping 114 matching lines @@
   }
 
   return NULL;
 }
 
 void GuestResourceProvider::StartUpdating() {
   DCHECK(!updating_);
   updating_ = true;
 
   // Add all the existing guest WebContents.
-  for (RenderProcessHost::iterator i(
-           RenderProcessHost::AllHostsIterator());
-       !i.IsAtEnd(); i.Advance()) {
-    RenderProcessHost::RenderWidgetHostsIterator iter =
-        i.GetCurrentValue()->GetRenderWidgetHostsIterator();
-    for (; !iter.IsAtEnd(); iter.Advance()) {
-      const RenderWidgetHost* widget = iter.GetCurrentValue();
-      if (widget->IsRenderView()) {
-        RenderViewHost* rvh =
-            RenderViewHost::From(const_cast<RenderWidgetHost*>(widget));
-        if (rvh->IsSubframe())
-          Add(rvh);
-      }
+  RenderWidgetHost::List widgets = RenderWidgetHost::GetRenderWidgetHosts();
+  for (RenderWidgetHost::List::const_iterator it = widgets.begin();

[jam, 2013/06/12 19:59:59]

ditto

[nasko, 2013/06/12 21:18:59]

Done.

+      it != widgets.end(); ++it) {
+    const RenderWidgetHost* widget = *it;
+    if (widget->IsRenderView()) {
+      RenderViewHost* rvh =
+          RenderViewHost::From(const_cast<RenderWidgetHost*>(widget));
+      if (rvh->IsSubframe())
+        Add(rvh);
     }
   }
 
   // Then we register for notifications to get new guests.
   registrar_.Add(this, content::NOTIFICATION_WEB_CONTENTS_CONNECTED,
       content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, content::NOTIFICATION_WEB_CONTENTS_DISCONNECTED,
       content::NotificationService::AllBrowserContextsAndSources());
 }
 
@@ skipping 46 matching lines @@
       break;
     case content::NOTIFICATION_WEB_CONTENTS_DISCONNECTED:
       Remove(web_contents->GetRenderViewHost());
       break;
     default:
       NOTREACHED() << "Unexpected notification.";
   }
 }
 
 }  // namespace task_manager