Enable WidgetCaptureTest.CaptureDesktopNativeWidget on desktop Linux

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 864 matching lines @@
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WidgetCaptureTest);
 };
 
 // See description in TestCapture().
 TEST_F(WidgetCaptureTest, Capture) {
   TestCapture(false);
 }
 
-#if !defined(OS_LINUX)
+#if !defined(OS_CHROMEOS)
 // See description in TestCapture(). Creates DesktopNativeWidget.
 TEST_F(WidgetCaptureTest, CaptureDesktopNativeWidget) {
   TestCapture(true);
 }
 #endif
 
 #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.
@@ skipping 27 matching lines @@
   EXPECT_EQ(0, mouse_view1->ExitedCalls());
 
   widget2.SetCapture(NULL);
   EXPECT_EQ(0, mouse_view1->EnteredCalls());
   // Grabbing native capture on Windows generates a ui::ET_MOUSE_EXITED event
   // in addition to the one generated by Chrome.
   EXPECT_LT(0, mouse_view1->ExitedCalls());
 }
 #endif
 
-#if !defined(OS_CHROMEOS)
+#if defined(OS_WIN)
 namespace {
 
-// Used to veirfy OnMouseEvent() has been invoked.
+// Used to verify OnMouseEvent() has been invoked.
 class MouseEventTrackingWidget : public Widget {
  public:
   MouseEventTrackingWidget() : got_mouse_event_(false) {}
   virtual ~MouseEventTrackingWidget() {}
 
   bool GetAndClearGotMouseEvent() {
     bool value = got_mouse_event_;
     got_mouse_event_ = false;
     return value;
   }
 
   // Widget:
   virtual void OnMouseEvent(ui::MouseEvent* event) OVERRIDE {
     got_mouse_event_ = true;
     Widget::OnMouseEvent(event);
   }
 
  private:
   bool got_mouse_event_;
 
   DISALLOW_COPY_AND_ASSIGN(MouseEventTrackingWidget);
 };
 
 }  // namespace
 
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS)
-// TODO(erg): linux_aura bringup: http://crbug.com/163931
-#define MAYBE_MouseEventDispatchedToRightWindow \
-  DISABLED_MouseEventDispatchedToRightWindow
-#else
-#define MAYBE_MouseEventDispatchedToRightWindow \
-  MouseEventDispatchedToRightWindow
-#endif
-
 // Verifies if a mouse event is received on a widget that doesn't have capture
-// it is correctly processed by the widget that doesn't have capture.
-TEST_F(WidgetCaptureTest, MAYBE_MouseEventDispatchedToRightWindow) {
+// on Windows that it is correctly processed by the widget that doesn't have
+// capture. This behavior is not desired on OSes other than Windows.

[pkotwicz, 2014/07/10 19:22:28]

We do not want this behavior on Desktop Linux. Notice the capture related code
in DesktopWindowTreeHostX11::DispatchMouseEvent()

+TEST_F(WidgetCaptureTest, MouseEventDispatchedToRightWindow) {
   MouseEventTrackingWidget widget1;
   Widget::InitParams params1 =
       CreateParams(views::Widget::InitParams::TYPE_WINDOW);
   params1.native_widget = new DesktopNativeWidgetAura(&widget1);
   params1.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   widget1.Init(params1);
   widget1.Show();
 
   MouseEventTrackingWidget widget2;
   Widget::InitParams params2 =
@@ skipping 17 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