Refactor RenderProcessHost to use IPC::Listener instead of RenderWidgetHost

content/browser/accessibility/browser_accessibility_state_impl.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 "content/browser/accessibility/browser_accessibility_state_impl.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/timer.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
@@ skipping 110 matching lines @@
 #if !defined(OS_WIN)
 void BrowserAccessibilityStateImpl::UpdatePlatformSpecificHistograms() {
 }
 #endif
 
 void BrowserAccessibilityStateImpl::SetAccessibilityMode(
     AccessibilityMode mode) {
   if (accessibility_mode_ == mode)
     return;
   accessibility_mode_ = mode;
-  for (RenderProcessHost::iterator it(RenderProcessHost::AllHostsIterator());
-       !it.IsAtEnd(); it.Advance()) {
-    RenderProcessHost* render_process_host = it.GetCurrentValue();
-    DCHECK(render_process_host);
 
+  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;
     // Ignore processes that don't have a connection, such as crashed tabs.
-    if (!render_process_host->HasConnection())
+    if (!widget->GetProcess()->HasConnection())
+      continue;
+    if (!widget->IsRenderView())
       continue;
 
-    for (RenderProcessHost::RenderWidgetHostsIterator rwit(
-             render_process_host->GetRenderWidgetHostsIterator());
-         !rwit.IsAtEnd();
-         rwit.Advance()) {
-      RenderWidgetHost* rwh = const_cast<RenderWidgetHost*>(
-          rwit.GetCurrentValue());
-      DCHECK(rwh);
-      if (!rwh || !rwh->IsRenderView())
-        continue;
-      RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
-      rwhi->SetAccessibilityMode(mode);
-    }
+    RenderWidgetHostImpl* rwhi =
+        RenderWidgetHostImpl::From(const_cast<RenderWidgetHost*>(widget));

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

you'd get rid of the const cast after switching to size_t iterating

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

Done.

+    rwhi->SetAccessibilityMode(mode);
   }
 }
 
 }  // namespace content