Send mouse events to the renderer before changing mouse capture, so that the ...

chrome/browser/renderer_host/render_widget_host_view_win.cc

Index: chrome/browser/renderer_host/render_widget_host_view_win.cc
===================================================================
--- chrome/browser/renderer_host/render_widget_host_view_win.cc	(revision 40968)
+++ chrome/browser/renderer_host/render_widget_host_view_win.cc	(working copy)
@@ -1557,6 +1557,11 @@
 
   WebMouseEvent event(
       WebInputEventFactory::mouseEvent(m_hWnd, message, wparam, lparam));
+
+  // Send the event to the renderer before changing mouse capture, so that the
+  // capturelost event arrives after mouseup.
+  render_widget_host_->ForwardMouseEvent(event);

[darin (slow to review), 2010/03/10 00:14:06]

LGTM

I think you should make a similar change to WebWidgetHost::MouseEvent.

[jam, 2010/03/10 01:51:19]

good idea, thanks.  will add it and commit.

+
   switch (event.type) {
     case WebInputEvent::MouseMove:
       TrackMouseLeave(true);
@@ -1573,8 +1578,6 @@
       break;
   }
 
-  render_widget_host_->ForwardMouseEvent(event);
-
   if (activatable_ && event.type == WebInputEvent::MouseDown) {
     // This is a temporary workaround for bug 765011 to get focus when the
     // mouse is clicked. This happens after the mouse down event is sent to