Make RenderFrameHostImpl::GetRenderWidgetHost() always return an object

content/browser/accessibility/dump_accessibility_browsertest_base.cc

Index: content/browser/accessibility/dump_accessibility_browsertest_base.cc
diff --git a/content/browser/accessibility/dump_accessibility_browsertest_base.cc b/content/browser/accessibility/dump_accessibility_browsertest_base.cc
index 0c09681752226a2b0bb249d9d5c6e58acba5a8d1..f1e007fbc4136f23f228f1c8ae578b87f00fefdb 100644
--- a/content/browser/accessibility/dump_accessibility_browsertest_base.cc
+++ b/content/browser/accessibility/dump_accessibility_browsertest_base.cc
@@ -269,10 +269,8 @@ void DumpAccessibilityTestBase::RunTestForPlatform(
     RenderFrameHostImpl* current_frame_host = node->current_frame_host();
     if (!current_frame_host)
       continue;
-    RenderWidgetHostImpl* rwh = current_frame_host->GetRenderWidgetHost();
-    if (!rwh)
-      continue;
-    RenderWidgetHostViewBase* rwhv = rwh->GetView();
+    RenderWidgetHostViewBase* rwhv =
+        static_cast<RenderWidgetHostViewBase*>(current_frame_host->GetView());

[Charlie Reis, 2016/05/27 22:50:28]

Sanity check: Before, this code skipped frames that weren't local frame roots,
since those had null widgets.  That seemed to work out well, since we would only
do the surface check below once per RWHV.  Now we'll do it once per frame, which
might be several times per RWHV.

That doesn't seem like a big problem to me (since the subsequent waits should
return right away), but maybe it's worth a comment in case someone later wants
to simplify this to iterate over the RWHVs instead of the frames.

[kenrb, 2016/05/30 19:39:53]

Good point, fixed it to match the old behavior.

     if (rwhv && rwhv->IsChildFrameForTesting()) {
       SurfaceHitTestReadyNotifier notifier(
           static_cast<RenderWidgetHostViewChildFrame*>(rwhv));