Expose whether a frame is focused to the browser process

content/public/browser/render_frame_host.h

Index: content/public/browser/render_frame_host.h
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 43ae82b1e8e6366c935b312f441de0f92be8ac82..9351b1e42e9644ff0fb2888a1c8af3042afa12a3 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -99,6 +99,11 @@ class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
   // Returns the ServiceRegistry for this frame.
   virtual ServiceRegistry* GetServiceRegistry() = 0;
 
+  // Returns whether the frame is focused. A frame is considered focused when it
+  // is the focused frame with in its frame tree and its associated

[Charlie Reis, 2015/01/22 17:46:04]

nit: within

[mlamouri (slow - plz ping), 2015/01/22 18:09:40]

Done.

+  // RenderWidgetHost is also focused.
+  virtual bool IsFocused() = 0;

[Charlie Reis, 2015/01/22 17:46:04]

Sanity check: Do you have a CL ready to go that uses this outside content/?  I
remembered that we generally don't introduce public methods unless they have a
caller outside content:

http://www.chromium.org/developers/content-module/content-api
"only expose methods in the public API that embedders need. If a method is only
used by other code in content, it belongs in foo_impl.h and not foo.h."

[mlamouri (slow - plz ping), 2015/01/22 18:09:40]

Moved to RFHI and updated the tests.

+
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameHostImpl;