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

Unified Diff: content/browser/web_contents/web_contents_view_mac.mm

Issue 2005953002: [mac][views] Notify web_contents focused. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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/web_contents/web_contents_view_mac.mm
diff --git a/content/browser/web_contents/web_contents_view_mac.mm b/content/browser/web_contents/web_contents_view_mac.mm
index b87c8b96365db41b59054e359dbcec885d3f729c..365f89e011dafbe68b82584828ae8998d5a711c0 100644
--- a/content/browser/web_contents/web_contents_view_mac.mm
+++ b/content/browser/web_contents/web_contents_view_mac.mm
@@ -228,6 +228,7 @@ void WebContentsViewMac::UpdateDragCursor(WebDragOperation operation) {
void WebContentsViewMac::GotFocus() {
// This is only used in the views FocusManager stuff but it bleeds through
tapted 2016/06/01 11:34:47 this comment isn't needed any longer
Avi (use Gerrit) 2016/06/01 13:43:38 Agreed.
yamaxim 2016/06/01 14:23:55 Deleted
// all subclasses. http://crbug.com/21875
+ web_contents_->NotifyWebContentsFocused();
}
// This is called when the renderer asks us to take focus back (i.e., it has

Powered by Google App Engine
This is Rietveld 408576698