Expose whether a frame is focused to the browser process

content/public/browser/render_frame_host.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 
 #include <string>
 
 #include "base/callback_forward.h"
@@ skipping 81 matching lines @@
   // the result object on the page, navigating assistive technology to that
   // result.
   virtual void ActivateFindInPageResultForAccessibility(int request_id) = 0;
 
   // Temporary until we get rid of RenderViewHost.
   virtual RenderViewHost* GetRenderViewHost() = 0;
 
   // 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;
   RenderFrameHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_