Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1362)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc

Issue 301613002: Enable Tab Dragging Tests with Linux and Ash (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update Defines and Disabled Tests Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc
diff --git a/chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc b/chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc
index 71d49f436cbde5b34340bd5b817d0e7108d7972e..7200734c647310f560e3f990709c36eeb7aed0c1 100644
--- a/chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc
+++ b/chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc
@@ -4,6 +4,8 @@
#include "chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.h"
+#include <algorithm>
jonross 2014/05/27 17:44:27 Cpplint
+
#include "ash/wm/window_state.h"
#include "base/bind.h"
#include "base/callback.h"
@@ -46,9 +48,10 @@
#include "ash/display/display_controller.h"
#include "ash/display/display_manager.h"
#include "ash/shell.h"
+#include "ash/shell_init_params.h"
#include "ash/test/cursor_manager_test_api.h"
+#include "ash/test/test_shell_delegate.h"
#include "ash/wm/coordinate_conversion.h"
-#include "ash/wm/window_state.h"
jonross 2014/05/27 17:44:27 cpplint
#include "ash/wm/window_util.h"
#include "chrome/browser/ui/views/frame/immersive_mode_controller.h"
#include "ui/aura/client/screen_position_client.h"
@@ -334,7 +337,7 @@ IN_PROC_BROWSER_TEST_F(TabDragControllerTest, GestureEndShouldEndDragTest) {
ui::GestureEventDetails(ui::ET_GESTURE_BEGIN, 0.0f, 0.0f), 0);
tab_strip->MaybeStartDrag(tab1, gesture_begin,
tab_strip->GetSelectionModel());
- //tab_strip->tab_at(1)->OnGestureEvent(&gesture_begin);
+ // tab_strip->tab_at(1)->OnGestureEvent(&gesture_begin);
jonross 2014/05/27 17:44:27 cpplint
pkotwicz 2014/05/28 00:32:03 Can't we just get rid of this line?
jonross 2014/05/28 14:55:57 Done.
EXPECT_TRUE(TabDragController::IsActive());
ui::GestureEvent gesture_end(ui::ET_GESTURE_END, tab_1_center.x(),
@@ -354,7 +357,7 @@ class DetachToBrowserTabDragControllerTest
DetachToBrowserTabDragControllerTest() {}
virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
if (!docked_windows_enabled()) {
CommandLine::ForCurrentProcess()->AppendSwitch(
ash::switches::kAshDisableDockedWindows);
@@ -363,12 +366,26 @@ class DetachToBrowserTabDragControllerTest
}
virtual void SetUpOnMainThread() OVERRIDE {
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
pkotwicz 2014/05/28 00:32:03 Can this new code be removed?
jonross 2014/05/28 14:55:57 With this now restricted to chrome os, yes. Done.
+ if (!ash::Shell::HasInstance()) {
+ ash::ShellInitParams params;
+ // Owned by ash::Shell
+ params.delegate = new ash::test::TestShellDelegate;
+ params.context_factory = ui::ContextFactory::GetInstance();
+ ash::Shell::CreateInstance(params);
+ }
event_generator_.reset(new aura::test::EventGenerator(
ash::Shell::GetPrimaryRootWindow()));
#endif
}
+ virtual void TearDownOnMainThread() OVERRIDE {
+#if defined(OS_CHROMEOS)
+ event_generator_.reset();
+ ash::Shell::DeleteInstance();
+#endif
+ }
+
InputSource input_source() const {
return strstr(GetParam(), "mouse") ?
INPUT_SOURCE_MOUSE : INPUT_SOURCE_TOUCH;
@@ -382,7 +399,7 @@ class DetachToBrowserTabDragControllerTest
void SetEventGeneratorRootWindow(const gfx::Point& point) {
if (input_source() == INPUT_SOURCE_MOUSE)
return;
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_.reset(new aura::test::EventGenerator(
new ScreenEventGeneratorDelegate(ash::wm::GetRootWindowAt(point))));
#endif
@@ -396,7 +413,7 @@ class DetachToBrowserTabDragControllerTest
ui_test_utils::SendMouseEventsSync(
ui_controls::LEFT, ui_controls::DOWN);
}
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_->set_current_location(location);
event_generator_->PressTouch();
#else
@@ -408,7 +425,7 @@ class DetachToBrowserTabDragControllerTest
bool PressInput2() {
// Second touch input is only used for touch sequence tests.
EXPECT_EQ(INPUT_SOURCE_TOUCH, input_source());
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_->set_current_location(
event_generator_->current_location());
event_generator_->PressTouchId(1);
@@ -421,7 +438,7 @@ class DetachToBrowserTabDragControllerTest
bool DragInputTo(const gfx::Point& location) {
if (input_source() == INPUT_SOURCE_MOUSE)
return ui_test_utils::SendMouseMoveSync(location);
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_->MoveTouch(location);
#else
NOTREACHED();
@@ -432,7 +449,7 @@ class DetachToBrowserTabDragControllerTest
bool DragInputToAsync(const gfx::Point& location) {
if (input_source() == INPUT_SOURCE_MOUSE)
return ui_controls::SendMouseMove(location.x(), location.y());
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_->MoveTouch(location);
#else
NOTREACHED();
@@ -445,7 +462,7 @@ class DetachToBrowserTabDragControllerTest
const base::Closure& task) {
if (input_source() == INPUT_SOURCE_MOUSE)
return ui_controls::SendMouseMoveNotifyWhenDone(x, y, task);
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
base::MessageLoop::current()->PostTask(FROM_HERE, task);
event_generator_->MoveTouch(gfx::Point(x, y));
#else
@@ -460,7 +477,7 @@ class DetachToBrowserTabDragControllerTest
base::TimeDelta delay) {
if (input_source() == INPUT_SOURCE_MOUSE)
return ui_controls::SendMouseMoveNotifyWhenDone(x, y, task);
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
base::MessageLoop::current()->PostDelayedTask(FROM_HERE, task, delay);
event_generator_->MoveTouch(gfx::Point(x, y));
#else
@@ -474,7 +491,7 @@ class DetachToBrowserTabDragControllerTest
const base::Closure& task) {
if (input_source() == INPUT_SOURCE_MOUSE)
return ui_controls::SendMouseMoveNotifyWhenDone(x, y, task);
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
base::MessageLoop::current()->PostTask(FROM_HERE, task);
event_generator_->MoveTouchId(gfx::Point(x, y), 1);
#else
@@ -488,7 +505,7 @@ class DetachToBrowserTabDragControllerTest
return ui_test_utils::SendMouseEventsSync(
ui_controls::LEFT, ui_controls::UP);
}
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_->ReleaseTouch();
#else
NOTREACHED();
@@ -501,7 +518,7 @@ class DetachToBrowserTabDragControllerTest
return ui_test_utils::SendMouseEventsSync(
ui_controls::LEFT, ui_controls::UP);
}
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
event_generator_->ReleaseTouchId(1);
#else
NOTREACHED();
@@ -536,7 +553,7 @@ class DetachToBrowserTabDragControllerTest
Browser* browser() const { return InProcessBrowserTest::browser(); }
private:
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
scoped_ptr<aura::test::EventGenerator> event_generator_;
#endif
@@ -651,7 +668,7 @@ void DetachToOwnWindowStep2(DetachToBrowserTabDragControllerTest* test) {
ASSERT_TRUE(test->ReleaseInput());
}
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
+#if defined(OS_CHROMEOS)
bool IsWindowPositionManaged(aura::Window* window) {
return ash::wm::GetWindowState(window)->window_position_managed();
}
@@ -918,7 +935,7 @@ void DeleteSourceDetachedStep2(WebContents* tab,
} // namespace
-#if defined(OS_CHROMEOS)
+#if defined(OS_CHROMEOS) || defined(OS_LINUX)
// TODO(sky,sad): Disabled as it fails due to resize locks with a real
// compositor. crbug.com/331924
#define MAYBE_DeleteSourceDetached DISABLED_DeleteSourceDetached
@@ -968,7 +985,7 @@ void PressEscapeWhileDetachedStep2(const BrowserList* browser_list) {
} // namespace
-#if defined(OS_CHROMEOS)
+#if defined(OS_CHROMEOS) || defined(OS_LINUX)
// TODO(sky,sad): Disabled as it fails due to resize locks with a real
// compositor. crbug.com/331924
#define MAYBE_PressEscapeWhileDetached DISABLED_PressEscapeWhileDetached
@@ -1341,7 +1358,7 @@ void CancelOnNewTabWhenDraggingStep2(
} // namespace
-#if defined(OS_CHROMEOS)
+#if defined(OS_CHROMEOS) || defined(OS_LINUX)
// TODO(sky,sad): Disabled as it fails due to resize locks with a real
// compositor. crbug.com/331924
#define MAYBE_CancelOnNewTabWhenDragging DISABLED_CancelOnNewTabWhenDragging
@@ -1376,8 +1393,9 @@ IN_PROC_BROWSER_TEST_P(DetachToBrowserTabDragControllerTest,
}
}
-#if defined(USE_ASH) && !defined(OS_WIN) // TODO(win_ash)
-
+#if defined(OS_CHROMEOS)
+// TODO(sky,sad): A number of tests below are disabled as they fail due to
+// resize locks with a real compositor. crbug.com/331924
namespace {
void DragInMaximizedWindowStep2(DetachToBrowserTabDragControllerTest* test,
@@ -1404,16 +1422,9 @@ void DragInMaximizedWindowStep2(DetachToBrowserTabDragControllerTest* test,
} // namespace
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DragInMaximizedWindow DISABLED_DragInMaximizedWindow
-#else
-#define MAYBE_DragInMaximizedWindow DragInMaximizedWindow
-#endif
// Creates a browser with two tabs, maximizes it, drags the tab out.
IN_PROC_BROWSER_TEST_P(DetachToBrowserTabDragControllerTest,
- MAYBE_DragInMaximizedWindow) {
+ DISABLED_DragInMaximizedWindow) {
AddTabAndResetBrowser(browser());
browser()->window()->Maximize();
@@ -1497,18 +1508,9 @@ void DragSingleTabToSeparateWindowInSecondDisplayStep2(
} // namespace
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DragSingleTabToSeparateWindowInSecondDisplay \
- DISABLED_DragSingleTabToSeparateWindowInSecondDisplay
-#else
-#define MAYBE_DragSingleTabToSeparateWindowInSecondDisplay \
- DragSingleTabToSeparateWindowInSecondDisplay
-#endif
// Drags from browser to a second display and releases input.
IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
- MAYBE_DragSingleTabToSeparateWindowInSecondDisplay) {
+ DISABLED_DragSingleTabToSeparateWindowInSecondDisplay) {
// Add another tab.
AddTabAndResetBrowser(browser());
TabStrip* tab_strip = GetTabStripForBrowser(browser());
@@ -1579,17 +1581,9 @@ void DragTabToWindowInSeparateDisplayStep2(
} // namespace
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DragTabToWindowInSeparateDisplay \
- DISABLED_DragTabToWindowInSeparateDisplay
-#else
-#define MAYBE_DragTabToWindowInSeparateDisplay DragTabToWindowInSeparateDisplay
-#endif
// Drags from browser to another browser on a second display and releases input.
IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
- MAYBE_DragTabToWindowInSeparateDisplay) {
+ DISABLED_DragTabToWindowInSeparateDisplay) {
// Add another tab.
AddTabAndResetBrowser(browser());
TabStrip* tab_strip = GetTabStripForBrowser(browser());
@@ -1637,17 +1631,9 @@ IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
EXPECT_FALSE(browser2->window()->IsMaximized());
}
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DragTabToWindowOnSecondDisplay \
- DISABLED_DragTabToWindowOnSecondDisplay
-#else
-#define MAYBE_DragTabToWindowOnSecondDisplay DragTabToWindowOnSecondDisplay
-#endif
// Drags from browser to another browser on a second display and releases input.
IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
- MAYBE_DragTabToWindowOnSecondDisplay) {
+ DISABLED_DragTabToWindowOnSecondDisplay) {
// Add another tab.
AddTabAndResetBrowser(browser());
TabStrip* tab_strip = GetTabStripForBrowser(browser());
@@ -1707,19 +1693,10 @@ IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
EXPECT_FALSE(browser2->window()->IsMaximized());
}
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DragMaxTabToNonMaxWindowInSeparateDisplay \
- DISABLED_DragMaxTabToNonMaxWindowInSeparateDisplay
-#else
-#define MAYBE_DragMaxTabToNonMaxWindowInSeparateDisplay \
- DragMaxTabToNonMaxWindowInSeparateDisplay
-#endif
// Drags from a maximized browser to another non-maximized browser on a second
// display and releases input.
IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
- MAYBE_DragMaxTabToNonMaxWindowInSeparateDisplay) {
+ DISABLED_DragMaxTabToNonMaxWindowInSeparateDisplay) {
// Add another tab.
AddTabAndResetBrowser(browser());
browser()->window()->Maximize();
@@ -1732,7 +1709,7 @@ IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
aura::Window* second_root = roots[1];
gfx::Rect work_area = gfx::Screen::GetNativeScreen()->GetDisplayNearestWindow(
second_root).work_area();
- work_area.Inset(20,20,20,60);
+ work_area.Inset(20, 20, 20, 60);
jonross 2014/05/27 17:44:27 cpplint
Browser::CreateParams params(browser()->profile(),
browser()->host_desktop_type());
params.initial_show_state = ui::SHOW_STATE_NORMAL;
@@ -1783,17 +1760,10 @@ IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
EXPECT_FALSE(browser2->window()->IsMaximized());
}
-// Immersive fullscreen is Ash only. However, Windows Ash does not support
-// multiple displays.
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DragTabToImmersiveBrowserOnSeparateDisplay \
- DISABLED_DragTabToImmersiveBrowserOnSeparateDisplay
// Drags from a restored browser to an immersive fullscreen browser on a
// second display and releases input.
IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
- MAYBE_DragTabToImmersiveBrowserOnSeparateDisplay) {
+ DISABLED_DragTabToImmersiveBrowserOnSeparateDisplay) {
// Add another tab.
AddTabAndResetBrowser(browser());
TabStrip* tab_strip = GetTabStripForBrowser(browser());
@@ -1866,7 +1836,6 @@ IN_PROC_BROWSER_TEST_P(DetachToBrowserInSeparateDisplayTabDragControllerTest,
EXPECT_FALSE(immersive_controller2->IsRevealed());
EXPECT_TRUE(tab_strip2->IsImmersiveStyle());
}
-#endif // OS_CHROMEOS
// Subclass of DetachToBrowserTabDragControllerTest that
// creates multiple displays with different device scale factors.
@@ -1946,17 +1915,10 @@ void CursorDeviceScaleFactorStep(
} // namespace
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_CursorDeviceScaleFactor DISABLED_CursorDeviceScaleFactor
-#else
-#define MAYBE_CursorDeviceScaleFactor CursorDeviceScaleFactor
-#endif
// Verifies cursor's device scale factor is updated when a tab is moved across
// displays with different device scale factors (http://crbug.com/154183).
IN_PROC_BROWSER_TEST_P(DifferentDeviceScaleFactorDisplayTabDragControllerTest,
- MAYBE_CursorDeviceScaleFactor) {
+ DISABLED_CursorDeviceScaleFactor) {
// Add another tab.
AddTabAndResetBrowser(browser());
TabStrip* tab_strip = GetTabStripForBrowser(browser());
@@ -2047,19 +2009,10 @@ void CancelDragTabToWindowInSeparateDisplayStep2(
} // namespace
-#if defined(OS_CHROMEOS) || defined(OS_LINUX)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_CancelDragTabToWindowIn2ndDisplay \
- DISABLED_CancelDragTabToWindowIn2ndDisplay
-#else
-#define MAYBE_CancelDragTabToWindowIn2ndDisplay \
- CancelDragTabToWindowIn2ndDisplay
-#endif
// Drags from browser to a second display and releases input.
IN_PROC_BROWSER_TEST_F(
DetachToBrowserInSeparateDisplayAndCancelTabDragControllerTest,
- MAYBE_CancelDragTabToWindowIn2ndDisplay) {
+ DISABLED_CancelDragTabToWindowIn2ndDisplay) {
// Add another tab.
AddTabAndResetBrowser(browser());
TabStrip* tab_strip = GetTabStripForBrowser(browser());
@@ -2094,19 +2047,10 @@ IN_PROC_BROWSER_TEST_F(
ui_controls::LEFT, ui_controls::UP));
}
-#if defined(OS_CHROMEOS) || defined(OS_LINUX)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_CancelDragTabToWindowIn1stDisplay \
- DISABLED_CancelDragTabToWindowIn1stDisplay
-#else
-#define MAYBE_CancelDragTabToWindowIn1stDisplay \
- CancelDragTabToWindowIn1stDisplay
-#endif
// Drags from browser from a second display to primary and releases input.
IN_PROC_BROWSER_TEST_F(
DetachToBrowserInSeparateDisplayAndCancelTabDragControllerTest,
- MAYBE_CancelDragTabToWindowIn1stDisplay) {
+ DISABLED_CancelDragTabToWindowIn1stDisplay) {
aura::Window::Windows roots = ash::Shell::GetAllRootWindows();
ASSERT_EQ(2u, roots.size());
@@ -2162,16 +2106,9 @@ void PressSecondFingerWhileDetachedStep2(
} // namespace
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_PressSecondFingerWhileDetached DISABLED_PressSecondFingerWhileDetached
-#else
-#define MAYBE_PressSecondFingerWhileDetached PressSecondFingerWhileDetached
-#endif
// Detaches a tab and while detached presses a second finger.
IN_PROC_BROWSER_TEST_P(DetachToBrowserTabDragControllerTestTouch,
- MAYBE_PressSecondFingerWhileDetached) {
+ DISABLED_PressSecondFingerWhileDetached) {
gfx::Rect bounds(browser()->window()->GetBounds());
// Add another tab.
AddTabAndResetBrowser(browser());
@@ -2234,18 +2171,9 @@ void DetachToDockedWindowNextStep(
} // namespace
-#if defined(OS_CHROMEOS)
-// TODO(sky,sad): Disabled as it fails due to resize locks with a real
-// compositor. crbug.com/331924
-#define MAYBE_DetachToDockedWindowFromMaximizedWindow \
- DISABLED_DetachToDockedWindowFromMaximizedWindow
-#else
-#define MAYBE_DetachToDockedWindowFromMaximizedWindow \
- DetachToDockedWindowFromMaximizedWindow
-#endif
// Drags from browser to separate window, docks that window and releases mouse.
IN_PROC_BROWSER_TEST_P(DetachToDockedTabDragControllerTest,
- MAYBE_DetachToDockedWindowFromMaximizedWindow) {
+ DISABLED_DetachToDockedWindowFromMaximizedWindow) {
// TODO(sky,sad): Disabled as it fails due to resize locks with a real
// compositor. crbug.com/331924
if (docked_windows_enabled()) {
@@ -2315,8 +2243,7 @@ IN_PROC_BROWSER_TEST_P(DetachToDockedTabDragControllerTest,
}
}
-
-#endif
+#endif // OS_CHROMEOS
#if defined(USE_ASH) && defined(OS_CHROMEOS) // TODO(win_ash,linux_ash)
INSTANTIATE_TEST_CASE_P(TabDragging,
@@ -2334,7 +2261,7 @@ INSTANTIATE_TEST_CASE_P(TabDragging,
INSTANTIATE_TEST_CASE_P(TabDragging,
DetachToBrowserTabDragControllerTestTouch,
::testing::Values("touch", "touch docked"));
-#elif defined(USE_ASH) && !defined(OS_LINUX) // TODO(linux_ash)
+#elif defined(USE_ASH) // TODO(linux_ash)
pkotwicz 2014/05/28 00:32:03 Remove TODO
jonross 2014/05/28 14:55:57 Done.
INSTANTIATE_TEST_CASE_P(TabDragging,
DetachToBrowserTabDragControllerTest,
::testing::Values("mouse"));
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698