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

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

Issue 193118: Revert "First part of automated_ui_tests improvements." (Closed)
Patch Set: 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
diff --git a/chrome/test/automated_ui_tests/automated_ui_test_base.cc b/chrome/test/automated_ui_tests/automated_ui_test_base.cc
index 5f2ef954b30f259456a51ce768f6a9253bd4b25f..bf3983168d98dfec89580711c6c41ad2a3bb7899 100644
--- a/chrome/test/automated_ui_tests/automated_ui_test_base.cc
+++ b/chrome/test/automated_ui_tests/automated_ui_test_base.cc
@@ -241,13 +241,6 @@ bool AutomatedUITestBase::DragActiveTab(bool drag_right) {
}
#endif
-bool AutomatedUITestBase::FindInPage() {
- if (!RunCommandAsync(IDC_FIND))
- return false;
-
- return WaitForFindWindowVisibilityChange(active_browser(), true);
-}
-
bool AutomatedUITestBase::ForwardButton() {
return RunCommand(IDC_FORWARD);
}
@@ -256,10 +249,6 @@ bool AutomatedUITestBase::GoOffTheRecord() {
return RunCommand(IDC_NEW_INCOGNITO_WINDOW);
}
-bool AutomatedUITestBase::Home() {
- return RunCommand(IDC_HOME);
-}
-
bool AutomatedUITestBase::OpenAndActivateNewBrowserWindow(
scoped_refptr<BrowserProxy>* previous_browser) {
if (!automation()->OpenNewBrowserWindow(true /* SW_SHOWNORMAL */)) {
@@ -328,41 +317,6 @@ bool AutomatedUITestBase::RestoreTab() {
return RunCommand(IDC_RESTORE_TAB);
}
-bool AutomatedUITestBase::SelectNextTab() {
- return RunCommand(IDC_SELECT_NEXT_TAB);
-}
-
-bool AutomatedUITestBase::SelectPreviousTab() {
- return RunCommand(IDC_SELECT_PREVIOUS_TAB);
-}
-
-bool AutomatedUITestBase::ShowBookmarkBar() {
- bool is_visible;
- bool is_animating;
- if (!active_browser()->GetBookmarkBarVisibility(&is_visible,
- &is_animating)) {
- return false;
- }
-
- if (is_visible) {
- // If the bar is visible, then issuing the command again will toggle it.
- return true;
- }
-
- if (!RunCommandAsync(IDC_SHOW_BOOKMARK_BAR))
- return false;
-
- return WaitForBookmarkBarVisibilityChange(active_browser(), true);
-}
-
-bool AutomatedUITestBase::ShowDownloads() {
- return RunCommand(IDC_SHOW_DOWNLOADS);
-}
-
-bool AutomatedUITestBase::ShowHistory() {
- return RunCommand(IDC_SHOW_HISTORY);
-}
-
bool AutomatedUITestBase::RunCommandAsync(int browser_command) {
BrowserProxy* browser = active_browser();
if (NULL == browser) {
« no previous file with comments | « chrome/test/automated_ui_tests/automated_ui_test_base.h ('k') | chrome/test/automated_ui_tests/automated_ui_test_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698