Do not set the mouse event handler if Widget::SetCapture() fails

ui/views/widget/widget_interactive_uitest.cc

 // Copyright (c) 2013 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 "base/basictypes.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
@@ skipping 875 matching lines @@
   TestCapture(false);
 }
 
 #if !defined(OS_CHROMEOS)
 // See description in TestCapture(). Creates DesktopNativeWidget.
 TEST_F(WidgetCaptureTest, CaptureDesktopNativeWidget) {
   TestCapture(true);
 }
 #endif
 
+// Test that no state is set if capture fails.
+TEST_F(WidgetCaptureTest, FailedCaptureRequestIsNoop) {
+  Widget widget;
+  Widget::InitParams params =
+      CreateParams(Widget::InitParams::TYPE_WINDOW_FRAMELESS);
+  params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  params.bounds = gfx::Rect(400, 400);
+  widget.Init(params);
+
+  MouseView* mouse_view1 = new MouseView;
+  MouseView* mouse_view2 = new MouseView;
+  View* contents_view = new View;
+  contents_view->AddChildView(mouse_view1);
+  contents_view->AddChildView(mouse_view2);
+  widget.SetContentsView(contents_view);
+
+  mouse_view1->SetBounds(0, 0, 200, 400);
+  mouse_view2->SetBounds(200, 0, 200, 400);
+
+  // Setting capture should fail because |widget| is not visible.
+  widget.SetCapture(mouse_view1);
+  EXPECT_FALSE(widget.HasCapture());
+
+  widget.Show();
+  ui::MouseEvent mouse_press_event(ui::ET_MOUSE_PRESSED, gfx::Point(300, 10),
+      gfx::Point(300, 10), ui::EF_NONE, ui::EF_NONE);
+  ui::EventDispatchDetails details = widget.GetNativeWindow()->GetHost()->
+      event_processor()->OnEventFromSource(&mouse_press_event);
+  ASSERT_FALSE(details.dispatcher_destroyed);
+  EXPECT_FALSE(mouse_view1->pressed());
+  EXPECT_TRUE(mouse_view2->pressed());
+}
+
 #if !defined(OS_CHROMEOS)
 // Test that a synthetic mouse exit is sent to the widget which was handling
 // mouse events when a different widget grabs capture.
 // TODO(pkotwicz): Make test pass on CrOS.
 TEST_F(WidgetCaptureTest, MouseExitOnCaptureGrab) {
   Widget widget1;
   Widget::InitParams params1 =
       CreateParams(Widget::InitParams::TYPE_WINDOW_FRAMELESS);
   params1.native_widget = CreateNativeWidget(true, &widget1);
   params1.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
@@ skipping 139 matching lines @@
   ui::EventDispatchDetails details = widget1.GetNativeWindow()->
       GetHost()->event_processor()->OnEventFromSource(&mouse_event);
   ASSERT_FALSE(details.dispatcher_destroyed);
   EXPECT_TRUE(widget1.GetAndClearGotMouseEvent());
   EXPECT_FALSE(widget2.GetAndClearGotMouseEvent());
 }
 #endif
 
 }  // namespace test
 }  // namespace views