[mac][views] Notify web_contents focused.

content/browser/web_contents/web_contents_view_mac.mm

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..15319e6ebacbdc10762b766abca229446bd0276f 100644
--- a/content/browser/web_contents/web_contents_view_mac.mm
+++ b/content/browser/web_contents/web_contents_view_mac.mm
@@ -183,6 +183,7 @@ void WebContentsViewMac::Focus() {
   if (![window isVisible])
     return;
   [window makeKeyAndOrderFront:nil];
+  GotFocus();

[Avi (use Gerrit), 2016/05/30 16:13:33]

Should this be necessary?

RenderViewHostImpl::RenderWidgetGotFocus() calls it, so shouldn't that work for
us too on the Mac?

[yamaxim, 2016/05/30 16:57:21]

It's not called for mac_views because WebContentsViewMac::Focus() differs from
WebContentsViewAura::Focus() that call rwhv->Focus

[tapted, 2016/05/31 12:01:50]

So I think we want this to be triggered by -[RenderWidgetHostViewCocoa
becomeFirstResponder].

That does:

- (BOOL)becomeFirstResponder {
  if (!renderWidgetHostView_->render_widget_host_)
    return NO;

  renderWidgetHostView_->render_widget_host_->Focus();
  / * etc */


I think we just need to change that call to Focus() to be GotFocus() instead.
Since the first thing RenderWidgetHostImpl::GotFocus() does is to call Focus()

 }
 
 void WebContentsViewMac::SetInitialFocus() {
@@ -228,6 +229,7 @@ void WebContentsViewMac::UpdateDragCursor(WebDragOperation operation) {
 void WebContentsViewMac::GotFocus() {
   // This is only used in the views FocusManager stuff but it bleeds through
   // 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