MacViews: Disable tests in MenuViewDragAndDropTest.

chrome/browser/ui/views/menu_view_drag_and_drop_test.cc

 // Copyright 2014 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/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/views/menu_test_base.h"
 #include "chrome/test/base/interactive_test_utils.h"
 #include "ui/base/dragdrop/drag_drop_types.h"
 #include "ui/base/dragdrop/os_exchange_data.h"
 #include "ui/views/controls/menu/menu_controller.h"
 #include "ui/views/controls/menu/menu_item_view.h"
 #include "ui/views/controls/menu/menu_runner.h"
 #include "ui/views/controls/menu/submenu_view.h"
 #include "ui/views/view.h"
 
 namespace {
 
-// Borrowed from chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc,
-// since these are also disabled on Linux for drag and drop.
-// TODO(erg): Fix DND tests on linux_aura. crbug.com/163931
-#if defined(OS_LINUX) && defined(USE_AURA)
-#define MAYBE(x) DISABLED_##x
-#else
-#define MAYBE(x) x
-#endif
-
 const char kTestNestedDragData[] = "test_nested_drag_data";
 const char kTestTopLevelDragData[] = "test_top_level_drag_data";
 
 // A simple view which can be dragged.
 class TestDragView : public views::View {
  public:
   TestDragView();
   ~TestDragView() override;
 
  private:
@@ skipping 304 matching lines @@
   EXPECT_FALSE(target_view()->dropped());
   EXPECT_FALSE(asked_to_close());
   EXPECT_FALSE(menu()->GetSubmenu()->IsShowing());
 
   Done();
 }
 
 // Test that an in-menu (i.e., entirely implemented in the menu code) closes the
 // menu automatically once the drag is complete, and does not ask the delegate
 // to stay open.
-#if !defined(OS_WIN)  // flaky http://crbug.com/401226
-VIEW_TEST(MenuViewDragAndDropTestTestInMenuDrag, MAYBE(TestInMenuDrag))
-#endif
+// Disabled on all platforms for being flaky. Tracked in:
+// TODO(erg): Fix DND tests on linux_aura. http://crbug.com/163931.
+// Windows: http://crbug.com/401226.
+// TODO(tapted): De-flake and run on Mac. http://crbug.com/449058.
+VIEW_TEST(MenuViewDragAndDropTestTestInMenuDrag, DISABLED_TestInMenuDrag)
 
 class MenuViewDragAndDropTestNestedDrag : public MenuViewDragAndDropTest {
  public:
   MenuViewDragAndDropTestNestedDrag() {}
   ~MenuViewDragAndDropTestNestedDrag() override {}
 
  private:
   // MenuViewDragAndDropTest:
   void DoTestWithMenuOpen() override;
 
@@ skipping 77 matching lines @@
 
   // Clean up.
   menu()->GetSubmenu()->Close();
 
   Done();
 }
 
 // Test that a nested drag (i.e. one via a child view, and not entirely
 // implemented in menu code) will consult the delegate before closing the view
 // after the drag.
-#if !defined(OS_WIN)  // http://crbug.com/401226
+// Disabled on all platforms for being flaky. Tracked in:
+// TODO(erg): Fix DND tests on linux_aura. http://crbug.com/163931.
+// Windows: http://crbug.com/401226.
+// TODO(tapted): De-flake and run on Mac. http://crbug.com/449058.
 VIEW_TEST(MenuViewDragAndDropTestNestedDrag,
-          MAYBE(MenuViewDragAndDropNestedDrag))
-#endif
+          DISABLED_MenuViewDragAndDropNestedDrag)
 
 class MenuViewDragAndDropForDropStayOpen : public MenuViewDragAndDropTest {
  public:
   MenuViewDragAndDropForDropStayOpen() {}
   ~MenuViewDragAndDropForDropStayOpen() override {}
 
  private:
   // MenuViewDragAndDropTest:
   int GetMenuRunnerFlags() override;
   void DoTestWithMenuOpen() override;
@@ skipping 44 matching lines @@
   views::MenuController* controller = menu()->GetMenuController();
   ASSERT_TRUE(controller);
   EXPECT_TRUE(controller->IsCancelAllTimerRunningForTest());
 
   Done();
 }
 
 // Test that if a menu is opened for a drop handled entirely by menu code, the
 // menu will try to close if it does not receive any drag updates.
 VIEW_TEST(MenuViewDragAndDropForDropCancel, MenuViewCancelsForOwnDrag)