Add RenderFrameObserver::FocusedNodeChanged

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 7258dbb97d8338997e601ae19c18429750a3aa09..de1617a29ef6eea3c13f45f6d17bbfba4950cd20 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -703,8 +703,8 @@ void RenderFrameImpl::PepperTextInputTypeChanged(
     return;
 
   GetRenderWidget()->UpdateTextInputType();
-  if (renderer_accessibility())
-    renderer_accessibility()->FocusedNodeChanged(WebNode());
+
+  FocusedNodeChangedForAccessibility(WebNode());
 }
 
 void RenderFrameImpl::PepperCaretPositionChanged(
@@ -3741,8 +3741,12 @@ void RenderFrameImpl::HandleWebAccessibilityEvent(
 }
 
 void RenderFrameImpl::FocusedNodeChanged(const WebNode& node) {
-  if (renderer_accessibility_)
-    renderer_accessibility_->FocusedNodeChanged(node);
+  FOR_EACH_OBSERVER(RenderFrameObserver, observers_, FocusedNodeChanged(node));

[nasko, 2015/01/20 05:09:51]

Why not call FocusNodeChangedForAccessibility from here?

[dmazzoni, 2015/01/20 07:41:46]

The accessibility tree can span multiple frames, so the focused node changed
notification needs to be sent to the main frame, which is the one that owns the
accessibility tree.

It'd be better if we didn't need FocusNodeChangedForAccessibility at all, I'll
try to get rid of it soon (we should be able to do everything via
WebViewClient::postAccessibilityEvent).

+}
+
+void RenderFrameImpl::FocusedNodeChangedForAccessibility(const WebNode& node) {
+  if (renderer_accessibility())
+    renderer_accessibility()->AccessibilityFocusedNodeChanged(node);
 }
 
 // PlzNavigate