Index: chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc |
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc b/chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc |
index dbceb13bbeb6f9df748a7ca4a28296c98d56f5de..df1908ec538dd85dbcec156fbce5c377ad97800c 100644 |
--- a/chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc |
+++ b/chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc |
@@ -206,7 +206,7 @@ |
}; |
// TODO(erg): Fix bookmark DND tests on linux_aura. crbug.com/163931 |
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
+#if defined(OS_LINUX) && defined(USE_AURA) |
#define MAYBE(x) DISABLED_##x |
#else |
#define MAYBE(x) x |
@@ -485,7 +485,7 @@ |
} |
}; |
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
+#if defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(USE_AURA) |
// TODO(erg): linux_aura bringup: http://crbug.com/163931 |
#define MAYBE_HideOnDesktopClick DISABLED_HideOnDesktopClick |
#else |
@@ -1234,7 +1234,7 @@ |
BookmarkContextMenuNotificationObserver observer_; |
}; |
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
+#if defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(USE_AURA) |
// TODO(erg): linux_aura bringup: http://crbug.com/163931 |
#define MAYBE_CloseMenuAfterClosingContextMenu \ |
DISABLED_CloseMenuAfterClosingContextMenu |
@@ -1335,7 +1335,7 @@ |
} |
}; |
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
+#if defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(USE_AURA) |
// TODO(erg): linux_aura bringup: http://crbug.com/163931 |
#define MAYBE_CloseWithModalDialog DISABLED_CloseWithModalDialog |
#else |
@@ -1672,7 +1672,7 @@ |
BookmarkContextMenuNotificationObserver observer_; |
}; |
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
+#if defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(USE_AURA) |
// TODO(erg): linux_aura bringup: http://crbug.com/163931 |
#define MAYBE_ContextMenus3 DISABLED_ContextMenus3 |
#elif defined(USE_OZONE) |
@@ -1997,74 +1997,3 @@ |
}; |
VIEW_TEST(BookmarkBarViewTest21, ContextMenusForEmptyFolder) |
- |
-// Test that closing the source browser window while dragging a bookmark does |
-// not cause a crash. |
-class BookmarkBarViewTest22 : public BookmarkBarViewEventTestBase { |
- protected: |
- void DoTestOnMessageLoop() override { |
- // Move the mouse to the first folder on the bookmark bar and press the |
- // mouse. |
- views::LabelButton* button = GetBookmarkButton(0); |
- ui_test_utils::MoveMouseToCenterAndPress(button, ui_controls::LEFT, |
- ui_controls::DOWN | ui_controls::UP, |
- CreateEventTask(this, &BookmarkBarViewTest22::Step2)); |
- } |
- |
- private: |
- void Step2() { |
- // Menu should be showing. |
- views::MenuItemView* menu = bb_view_->GetMenu(); |
- ASSERT_TRUE(menu != NULL); |
- ASSERT_TRUE(menu->GetSubmenu()->IsShowing()); |
- |
- views::MenuItemView* child_menu = |
- menu->GetSubmenu()->GetMenuItemAt(0); |
- ASSERT_TRUE(child_menu != NULL); |
- |
- // Move mouse to center of menu and press button. |
- ui_test_utils::MoveMouseToCenterAndPress(child_menu, ui_controls::LEFT, |
- ui_controls::DOWN, |
- CreateEventTask(this, &BookmarkBarViewTest22::Step3)); |
- } |
- |
- void Step3() { |
- views::MenuItemView* target_menu = |
- bb_view_->GetMenu()->GetSubmenu()->GetMenuItemAt(1); |
- gfx::Point loc(1, target_menu->height() - 1); |
- views::View::ConvertPointToScreen(target_menu, &loc); |
- |
- // Start a drag. |
- ui_controls::SendMouseMoveNotifyWhenDone(loc.x() + 10, loc.y(), |
- CreateEventTask(this, &BookmarkBarViewTest22::Step4)); |
- |
- // See comment above this method as to why we do this. |
- ScheduleMouseMoveInBackground(loc.x(), loc.y()); |
- } |
- |
- void Step4() { |
- window_->Close(); |
- window_ = NULL; |
- |
-#if defined(OS_CHROMEOS) |
- ui_controls::SendMouseEventsNotifyWhenDone( |
- ui_controls::LEFT, ui_controls::UP, |
- CreateEventTask(this, &BookmarkBarViewTest22::Done)); |
-#else |
- // There are no widgets to send the mouse release to. |
- Done(); |
-#endif |
- } |
-}; |
- |
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
-// TODO(pkotwicz): Enable on Desktop Linux once crbug.com/438365 is fixed. |
-#define MAYBE_CloseSourceBrowserDuringDrag DISABLED_CloseSourceBrowserDuringDrag |
-#elif defined(OS_WIN) |
-// This test times out on Windows. TODO(pkotwicz): Find out why. |
-#define MAYBE_CloseSourceBrowserDuringDrag DISABLED_CloseSourceBrowserDuringDrag |
-#else |
-#define MAYBE_CloseSourceBrowserDuringDrag CloseSourceBrowserDuringDrag |
-#endif |
- |
-VIEW_TEST(BookmarkBarViewTest22, MAYBE_CloseSourceBrowserDuringDrag) |