Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5851)

Unified Diff: chrome/browser/task_manager/guest_resource_provider.cc

Issue 16431010: Refactor RenderProcessHost to use IPC::Listener instead of RenderWidgetHost (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase on top of hash_pair move. Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/task_manager/guest_resource_provider.cc
diff --git a/chrome/browser/task_manager/guest_resource_provider.cc b/chrome/browser/task_manager/guest_resource_provider.cc
index 1d7e016efd54200d9a0670ee5b1f21c1bacf52b2..5dcedf1a1bb6579a079b579ee42a4982e6a90112 100644
--- a/chrome/browser/task_manager/guest_resource_provider.cc
+++ b/chrome/browser/task_manager/guest_resource_provider.cc
@@ -132,19 +132,12 @@ void GuestResourceProvider::StartUpdating() {
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 (size_t i = 0; i < widgets.size(); ++i) {
+ if (widgets[i]->IsRenderView()) {
+ RenderViewHost* rvh = RenderViewHost::From(widgets[i]);
+ if (rvh->IsSubframe())
+ Add(rvh);
}
}

Powered by Google App Engine
This is Rietveld 408576698