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

Unified Diff: chrome/test/automation/automation_proxy.cc

Issue 10388175: Remove all the unnused automation IPCs. These were used by UI tests that have been converted to bro… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 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 | « chrome/test/automation/automation_proxy.h ('k') | chrome/test/automation/browser_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/automation/automation_proxy.cc
===================================================================
--- chrome/test/automation/automation_proxy.cc (revision 137438)
+++ chrome/test/automation/automation_proxy.cc (working copy)
@@ -262,42 +262,6 @@
return wait_success;
}
-bool AutomationProxy::GetShowingAppModalDialog(bool* showing_app_modal_dialog,
- ui::DialogButton* button) {
- if (!showing_app_modal_dialog || !button) {
- NOTREACHED();
- return false;
- }
-
- int button_int = 0;
-
- if (!Send(new AutomationMsg_ShowingAppModalDialog(
- showing_app_modal_dialog, &button_int))) {
- return false;
- }
-
- *button = static_cast<ui::DialogButton>(button_int);
- return true;
-}
-
-bool AutomationProxy::ClickAppModalDialogButton(ui::DialogButton button) {
- bool succeeded = false;
-
- if (!Send(new AutomationMsg_ClickAppModalDialogButton(
- button, &succeeded))) {
- return false;
- }
-
- return succeeded;
-}
-
-bool AutomationProxy::WaitForAppModalDialog() {
- bool wait_success = false;
- if (!Send(new AutomationMsg_WaitForAppModalDialogToBeShown(&wait_success)))
- return false;
- return wait_success;
-}
-
bool AutomationProxy::IsURLDisplayed(GURL url) {
int window_count;
if (!GetBrowserWindowCount(&window_count))
@@ -339,13 +303,6 @@
return Send(new AutomationMsg_SetFilteredInet(enabled));
}
-int AutomationProxy::GetFilteredInetHitCount() {
- int hit_count;
- if (!Send(new AutomationMsg_GetFilteredInetHitCount(&hit_count)))
- return -1;
- return hit_count;
-}
-
bool AutomationProxy::SendProxyConfig(const std::string& new_proxy_config) {
return Send(new AutomationMsg_SetProxyConfig(new_proxy_config));
}
@@ -386,14 +343,6 @@
return ProxyObjectFromHandle<BrowserProxy>(handle);
}
-bool AutomationProxy::GetBrowserLocale(string16* locale) {
- DCHECK(locale != NULL);
- if (!Send(new AutomationMsg_GetBrowserLocale(locale)))
- return false;
-
- return !locale->empty();
-}
-
scoped_refptr<BrowserProxy> AutomationProxy::FindTabbedBrowserWindow() {
int handle = 0;
if (!Send(new AutomationMsg_FindTabbedBrowserWindow(&handle)))
@@ -402,14 +351,6 @@
return ProxyObjectFromHandle<BrowserProxy>(handle);
}
-scoped_refptr<BrowserProxy> AutomationProxy::GetLastActiveBrowserWindow() {
- int handle = 0;
- if (!Send(new AutomationMsg_LastActiveBrowserWindow(&handle)))
- return NULL;
-
- return ProxyObjectFromHandle<BrowserProxy>(handle);
-}
-
IPC::SyncChannel* AutomationProxy::channel() {
return channel_.get();
}
@@ -509,18 +450,6 @@
tracker_->put_channel(NULL);
}
-#if defined(OS_CHROMEOS)
-bool AutomationProxy::LoginWithUserAndPass(const std::string& username,
- const std::string& password) {
- bool success;
- bool sent = Send(new AutomationMsg_LoginWithUserAndPass(username,
- password,
- &success));
- // If message sending unsuccessful or test failed, return false.
- return sent && success;
-}
-#endif
-
bool AutomationProxy::BeginTracing(const std::string& categories) {
bool result = false;
bool send_success = Send(new AutomationMsg_BeginTracing(categories,
« no previous file with comments | « chrome/test/automation/automation_proxy.h ('k') | chrome/test/automation/browser_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698