Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(368)

Unified Diff: content/browser/frame_host/render_frame_host_delegate.h

Issue 2571583008: Support tracking focused element for OOPIFs. (Closed)
Patch Set: Fixed some compile errors Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/frame_host/render_frame_host_delegate.h
diff --git a/content/browser/frame_host/render_frame_host_delegate.h b/content/browser/frame_host/render_frame_host_delegate.h
index 6f3497ca3439fc0dba77c337296323adaf3a2629..20a5565593102094ea433a247e3f93dbda5654d5 100644
--- a/content/browser/frame_host/render_frame_host_delegate.h
+++ b/content/browser/frame_host/render_frame_host_delegate.h
@@ -36,11 +36,16 @@ class GeolocationServiceContext;
class WakeLockServiceContext;
}
+namespace gfx {
+class Rect;
+}
+
namespace content {
class FrameTreeNode;
class InterstitialPage;
class PageState;
class RenderFrameHost;
+class RenderFrameHostImpl;
class ScreenOrientationProvider;
class WebContents;
struct AXEventNotificationDetails;
@@ -212,6 +217,13 @@ class CONTENT_EXPORT RenderFrameHostDelegate {
virtual std::unique_ptr<WebUIImpl> CreateWebUIForRenderFrameHost(
const GURL& url);
+ // Called by |frame| to notify that it has received an update on focused
+ // element. |bounds_in_frame_widget| is the rectangle containing the element
ncarter (slow) 2016/12/16 19:17:20 |bounds_in_frame_widget| or |bounds_in_root_view|?
EhsanK 2016/12/20 16:39:10 Sorry. |boudns_in_root_view|. Thanks! Done.
+ // focused and is with respect to frame's RenderWidgetHost's coordinate space.
+ virtual void OnFocusedElementChangedInFrame(
+ RenderFrameHostImpl* frame,
+ const gfx::Rect& bounds_in_root_view) {}
+
protected:
virtual ~RenderFrameHostDelegate() {}
};

Powered by Google App Engine
This is Rietveld 408576698