Ensure that windowed plugins get focus during WM_MOUSEACTIVATE. This fixes...

chrome/browser/render_widget_host_view_win.cc

 // Copyright (c) 2006-2008 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.
 
 #include "chrome/browser/render_widget_host_view_win.h"
 
 #include "base/command_line.h"
 #include "base/gfx/gdi_util.h"
 #include "base/gfx/rect.h"
 #include "base/histogram.h"
@@ skipping 761 matching lines @@
   if (!handled_by_webcontents) {
     render_widget_host_->ForwardWheelEvent(
         WebMouseWheelEvent(m_hWnd, message, wparam, lparam));
   }
   handled = TRUE;
   return 0;
 }
 
 LRESULT RenderWidgetHostViewWin::OnMouseActivate(UINT, WPARAM, LPARAM,
                                                  BOOL& handled) {
-  // We handle WM_MOUSEACTIVATE to set focus to the underlying plugin
-  // child window. This is to ensure that keyboard events are received
-  // by the plugin. The correct way to fix this would be send over
-  // an event to the renderer which would then eventually send over
-  // a setFocus call to the plugin widget. This would ensure that
-  // the renderer (webkit) knows about the plugin widget receiving
-  // focus.
-  // TODO(iyengar) Do the right thing as per the above comment.
-  POINT cursor_pos = {0};
-  ::GetCursorPos(&cursor_pos);
-  MapWindowPoints(m_hWnd, &cursor_pos, 1);
-  HWND child_window = ::RealChildWindowFromPoint(m_hWnd, cursor_pos);
-  if (::IsWindow(child_window)) {
-    ::SetFocus(child_window);
-    return MA_NOACTIVATE;
-  } else {
-    handled = FALSE;
-    return MA_ACTIVATE;
+  HWND focus_window = GetFocus();
+  if (!::IsWindow(focus_window) || !IsChild(focus_window)) {
+    // We handle WM_MOUSEACTIVATE to set focus to the underlying plugin
+    // child window. This is to ensure that keyboard events are received
+    // by the plugin. The correct way to fix this would be send over
+    // an event to the renderer which would then eventually send over
+    // a setFocus call to the plugin widget. This would ensure that
+    // the renderer (webkit) knows about the plugin widget receiving
+    // focus.
+    // TODO(iyengar) Do the right thing as per the above comment.
+    POINT cursor_pos = {0};
+    ::GetCursorPos(&cursor_pos);
+    ::ScreenToClient(m_hWnd, &cursor_pos);
+    HWND child_window = ::RealChildWindowFromPoint(m_hWnd, cursor_pos);
+    if (::IsWindow(child_window)) {
+      ::SetFocus(child_window);
+      return MA_NOACTIVATE;
+    }
   }
+  handled = FALSE;
+  return MA_ACTIVATE;
 }
 
 LRESULT RenderWidgetHostViewWin::OnGetObject(UINT message, WPARAM wparam,
                                              LPARAM lparam, BOOL& handled) {
   LRESULT reference_result = static_cast<LRESULT>(0L);
   // TODO(jcampan): http://b/issue?id=1432077 Disabling accessibility in the
   // renderer is a temporary work-around until that bug is fixed.
   if (!renderer_accessible_)
     return reference_result;
 
@@ skipping 96 matching lines @@
     ::DestroyWindow(tooltip_hwnd_);
   tooltip_hwnd_ = NULL;
 }
 
 void RenderWidgetHostViewWin::ShutdownHost() {
   shutdown_factory_.RevokeAll();
   render_widget_host_->Shutdown();
   // Do not touch any members at this point, |this| has been deleted.
 }