Don't track mouse events in HWNDMessageHandler when they are forwarded by the LegacyRenderWidgetHost

ui/views/win/hwnd_message_handler.cc

 // Copyright (c) 2012 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 "ui/views/win/hwnd_message_handler.h"
 
 #include <dwmapi.h>
 #include <oleacc.h>
 #include <shellapi.h>
 #include <wtsapi32.h>
@@ skipping 770 matching lines @@
     fwi.dwFlags = FLASHW_STOP;
   }
   FlashWindowEx(&fwi);
 }
 
 void HWNDMessageHandler::ClearNativeFocus() {
   ::SetFocus(hwnd());
 }
 
 void HWNDMessageHandler::SetCapture() {
-  DCHECK(!HasCapture());

[sky, 2014/02/12 14:36:31]

What did the call stack look like when you hit this?

   ::SetCapture(hwnd());
 }
 
 void HWNDMessageHandler::ReleaseCapture() {
   if (HasCapture())
     ::ReleaseCapture();
 }
 
 bool HWNDMessageHandler::HasCapture() const {
   return ::GetCapture() == hwnd();
@@ skipping 487 matching lines @@
   return handled;
 }
 
 void HWNDMessageHandler::OnCancelMode() {
   delegate_->HandleCancelMode();
   // Need default handling, otherwise capture and other things aren't canceled.
   SetMsgHandled(FALSE);
 }
 
 void HWNDMessageHandler::OnCaptureChanged(HWND window) {
-  delegate_->HandleCaptureLost();
+  if (!HasCapture())
+    delegate_->HandleCaptureLost();
 }
 
 void HWNDMessageHandler::OnClose() {
   delegate_->HandleClose();
 }
 
 void HWNDMessageHandler::OnCommand(UINT notification_code,
                                    int command,
                                    HWND window) {
   // If the notification code is > 1 it means it is control specific and we
@@ skipping 308 matching lines @@
       }
     }
   } else if (message == WM_NCRBUTTONDOWN &&
       (w_param == HTCAPTION || w_param == HTSYSMENU)) {
     is_right_mouse_pressed_on_caption_ = true;
     // We SetCapture() to ensure we only show the menu when the button
     // down and up are both on the caption. Note: this causes the button up to
     // be WM_RBUTTONUP instead of WM_NCRBUTTONUP.
     SetCapture();
   }
-
   MSG msg = { hwnd(), message, w_param, l_param, GetMessageTime(),
               { CR_GET_X_LPARAM(l_param), CR_GET_Y_LPARAM(l_param) } };
   ui::MouseEvent event(msg);
   if (!touch_ids_.empty() || ui::IsMouseEventFromTouch(message))
     event.set_flags(event.flags() | ui::EF_FROM_TOUCH);
 
   if (!(event.flags() & ui::EF_IS_NON_CLIENT))
     delegate_->HandleTooltipMouseMove(message, w_param, l_param);
 
   if (event.type() == ui::ET_MOUSE_MOVED && !HasCapture()) {
@@ skipping 747 matching lines @@
   SetMsgHandled(FALSE);
 }
 
 void HWNDMessageHandler::HandleTouchEvents(const TouchEvents& touch_events) {
   base::WeakPtr<HWNDMessageHandler> ref(weak_factory_.GetWeakPtr());
   for (size_t i = 0; i < touch_events.size() && ref; ++i)
     delegate_->HandleTouchEvent(touch_events[i]);
 }
 
 }  // namespace views