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

Unified Diff: content/browser/renderer_host/render_widget_host_impl.h

Issue 2451143003: <webview>: Correctly shift focus between WebContents. (Closed)
Patch Set: Address comments from alexmos and lfg. Created 4 years, 2 months 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/renderer_host/render_widget_host_impl.h
diff --git a/content/browser/renderer_host/render_widget_host_impl.h b/content/browser/renderer_host/render_widget_host_impl.h
index eb69fce58933f696fe3b2e2f7293b19d713581e4..2d39c14c1e87c6c4a9544085758a2b3263223875 100644
--- a/content/browser/renderer_host/render_widget_host_impl.h
+++ b/content/browser/renderer_host/render_widget_host_impl.h
@@ -250,6 +250,10 @@ class CONTENT_EXPORT RenderWidgetHostImpl : public RenderWidgetHost,
// but it failed, thus HasFocus() returns false.
bool is_focused() const { return is_focused_; }
+ // Support for focus tracking on multi-WebContents cases. Users other than
alexmos 2016/10/31 18:57:57 Let's explain more carefully what this does. I.e.
avallee 2016/11/07 19:18:44 Done.
+ // WebContens and RenderWidgetHost should use Focus()/Blur().
+ void SetPageFocus(bool focused);
+
// Called to notify the RenderWidget that it has lost the mouse lock.
void LostMouseLock();

Powered by Google App Engine
This is Rietveld 408576698