Expose whether a frame is focused to the browser process

content/browser/frame_host/render_frame_host_impl.h

Index: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index e9649758eed0a210b94fe3583e3a8e901f5476be..6aed951db7e8561672fd77e50db67f49718496c6 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -134,9 +134,9 @@ class CONTENT_EXPORT RenderFrameHostImpl
   void ExecuteJavaScript(const base::string16& javascript,
                          const JavaScriptResultCallback& callback) override;
   void ExecuteJavaScriptForTests(const base::string16& javascript) override;
+  void ActivateFindInPageResultForAccessibility(int request_id) override;
   RenderViewHost* GetRenderViewHost() override;
   ServiceRegistry* GetServiceRegistry() override;
-  void ActivateFindInPageResultForAccessibility(int request_id) override;
 
   // IPC::Sender
   bool Send(IPC::Message* msg) override;
@@ -405,6 +405,11 @@ class CONTENT_EXPORT RenderFrameHostImpl
   // this instance and its associated render frame.
   void InvalidateMojoConnection();
 
+  // Returns whether the frame is focused. A frame is considered focused when it
+  // is the parent chain of the focused frame within the frame tree and its

[kenrb, 2015/01/23 18:40:12]

nit: "it is *in* the parent chain"

More generally, I find this comment hard to follow. I think the distinction
between focused frame and focused RenderWidgetHost is too subtle for this to be
easy to parse.

[mlamouri (slow - plz ping), 2015/01/26 14:02:18]

I re-worded a bit.

+  // associated RenderWidgetHost is also focused.
+  bool IsFocused();
+
  protected:
   friend class RenderFrameHostFactory;