MacViews: Implement Tab Dragging

chrome/test/base/interactive_test_utils.h

Index: chrome/test/base/interactive_test_utils.h
diff --git a/chrome/test/base/interactive_test_utils.h b/chrome/test/base/interactive_test_utils.h
index f0ccc770f2a8cfae8ca6437cdfc83364527badf7..cd92d2c2af122dd459ecb9e573aa7f6f6eea8746 100644
--- a/chrome/test/base/interactive_test_utils.h
+++ b/chrome/test/base/interactive_test_utils.h
@@ -21,6 +21,18 @@ class View;
 
 namespace ui_test_utils {
 
+#if defined(OS_MACOSX)
+// Moves the mouse to the |from| position, presses left mouse button,
+// then moves mouse to the |to| position, waits the |delay|, releases left mouse
+// button.
+// |steps| indicates number of intermediate points that are interpolated between
+// |from| and |to|.
+void DragAndDrop(const gfx::Point& from,
+                 const gfx::Point& to,
+                 base::TimeDelta delay = base::TimeDelta(),

[tapted, 2016/03/10 11:51:19]

can the delay argument be removed? It might help while debugging the test, but
if people are tempted to use it, it could lead to test flakiness.

[themblsha, 2016/03/10 17:18:58]

I thought that this delay would be useful for testing
tab_strip_drag_controller.mm as it has kTearDuration, and if we don't wait for
enough time the precise window bounds comparisons will fail.

+                 unsigned int steps = 1);

[tapted, 2016/03/10 11:51:19]

just int

[themblsha, 2016/03/10 17:18:58]

Done.

+#endif  // OS_MACOSX
+
 // Brings the native window for |browser| to the foreground. Returns true on
 // success.
 bool BringBrowserWindowToFront(const Browser* browser) WARN_UNUSED_RESULT;