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

Unified Diff: chrome/test/automated_ui_tests/automated_ui_test_base.cc

Issue 211033: Automated ui test porting + cleanup:... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: all green\! Created 11 years, 3 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
Index: chrome/test/automated_ui_tests/automated_ui_test_base.cc
===================================================================
--- chrome/test/automated_ui_tests/automated_ui_test_base.cc (revision 26702)
+++ chrome/test/automated_ui_tests/automated_ui_test_base.cc (working copy)
@@ -2,6 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include "base/gfx/point.h"
+#include "base/gfx/rect.h"
#include "chrome/app/chrome_dll_resource.h"
#include "chrome/browser/view_ids.h"
#include "chrome/test/automated_ui_tests/automated_ui_test_base.h"
@@ -9,11 +11,8 @@
#include "chrome/test/automation/tab_proxy.h"
#include "chrome/test/automation/window_proxy.h"
#include "chrome/test/ui/ui_test.h"
+#include "views/event.h"
-#if defined(TOOLKIT_VIEWS)
-#include "views/view.h"
-#endif
-
AutomatedUITestBase::AutomatedUITestBase() {}
AutomatedUITestBase::~AutomatedUITestBase() {}
@@ -84,7 +83,6 @@
return RunCommand(IDC_DUPLICATE_TAB);
}
-#if defined(OS_WIN)
bool AutomatedUITestBase::DragTabOut() {
BrowserProxy* browser = active_browser();
if (browser == NULL) {
@@ -141,13 +139,13 @@
return false;
}
- // Click on the center of the tab, and drag it downwads.
- POINT start;
- POINT end;
- start.x = dragged_tab_bounds.x() + dragged_tab_bounds.width()/2;
- start.y = dragged_tab_bounds.y() + dragged_tab_bounds.height()/2;
- end.x = start.x;
- end.y = start.y + 3*urlbar_bounds.height();
+ // Click on the center of the tab, and drag it downwards.
+ gfx::Point start;
+ gfx::Point end;
+ start.set_x(dragged_tab_bounds.x() + dragged_tab_bounds.width() / 2);
+ start.set_y(dragged_tab_bounds.y() + dragged_tab_bounds.height() / 2);
+ end.set_x(start.x());
+ end.set_y(start.y() + 3 * urlbar_bounds.height());
if (!browser->SimulateDragWithTimeout(start, end,
views::Event::EF_LEFT_BUTTON_DOWN,
@@ -203,8 +201,8 @@
}
// Click on the center of the tab, and drag it to the left or the right.
- POINT dragged_tab_point(dragged_tab_bounds.CenterPoint().ToPOINT());
- POINT destination_point(dragged_tab_point);
+ gfx::Point dragged_tab_point = dragged_tab_bounds.CenterPoint();
+ gfx::Point destination_point = dragged_tab_point;
int new_tab_index;
if (drag_right) {
@@ -213,14 +211,14 @@
return false;
}
new_tab_index = tab_index + 1;
- destination_point.x += 2 * dragged_tab_bounds.width() / 3;
+ destination_point.Offset(2 * dragged_tab_bounds.width() / 3, 0);
} else {
if (tab_index <= 0) {
LogInfoMessage("cant_drag_to_left");
return false;
}
new_tab_index = tab_index - 1;
- destination_point.x -= 2 * dragged_tab_bounds.width() / 3;
+ destination_point.Offset(-2 * dragged_tab_bounds.width() / 3, 0);
}
if (!browser->SimulateDragWithTimeout(dragged_tab_point,
@@ -239,7 +237,6 @@
return true;
}
-#endif
bool AutomatedUITestBase::FindInPage() {
if (!RunCommandAsync(IDC_FIND))

Powered by Google App Engine
This is Rietveld 408576698