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

Unified Diff: chrome/test/automation/browser_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/browser_proxy.h ('k') | chrome/test/automation/tab_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/automation/browser_proxy.cc
===================================================================
--- chrome/test/automation/browser_proxy.cc (revision 137438)
+++ chrome/test/automation/browser_proxy.cc (working copy)
@@ -70,17 +70,6 @@
return append_tab_response >= 0;
}
-bool BrowserProxy::AppendBackgroundTab(const GURL& tab_url) {
- if (!is_valid())
- return false;
-
- int append_tab_response = -1;
-
- sender_->Send(new AutomationMsg_AppendBackgroundTab(handle_, tab_url,
- &append_tab_response));
- return append_tab_response >= 0;
-}
-
bool BrowserProxy::GetActiveTabIndex(int* active_tab_index) const {
if (!is_valid())
return false;
@@ -175,21 +164,6 @@
return true;
}
-bool BrowserProxy::IsApplication(bool* is_application) {
- DCHECK(is_application);
-
- if (!is_valid())
- return false;
-
- bool success = false;
- if (!sender_->Send(new AutomationMsg_IsBrowserInApplicationMode(
- handle_, is_application, &success))) {
- return false;
- }
-
- return success;
-}
-
bool BrowserProxy::ApplyAccelerator(int id) {
return RunCommandAsync(id);
}
@@ -248,13 +222,6 @@
// This message expects no response.
}
-bool BrowserProxy::GetFindWindowLocation(int* x, int* y) {
- if (!is_valid() || !x || !y)
- return false;
-
- return sender_->Send(new AutomationMsg_FindWindowLocation(handle_, x, y));
-}
-
bool BrowserProxy::IsFindWindowFullyVisible(bool* is_visible) {
if (!is_valid())
return false;
@@ -418,72 +385,6 @@
is_visible));
}
-bool BrowserProxy::SetIntPreference(const std::string& name, int value) {
- if (!is_valid())
- return false;
-
- bool result = false;
-
- sender_->Send(new AutomationMsg_SetIntPreference(handle_, name, value,
- &result));
- return result;
-}
-
-bool BrowserProxy::SetStringPreference(const std::string& name,
- const std::string& value) {
- if (!is_valid())
- return false;
-
- bool result = false;
-
- sender_->Send(new AutomationMsg_SetStringPreference(handle_, name, value,
- &result));
- return result;
-}
-
-bool BrowserProxy::GetBooleanPreference(const std::string& name,
- bool* value) {
- if (!is_valid())
- return false;
-
- bool result = false;
-
- sender_->Send(new AutomationMsg_GetBooleanPreference(handle_, name, value,
- &result));
- return result;
-}
-
-bool BrowserProxy::SetBooleanPreference(const std::string& name,
- bool value) {
- if (!is_valid())
- return false;
-
- bool result = false;
-
- sender_->Send(new AutomationMsg_SetBooleanPreference(handle_, name,
- value, &result));
- return result;
-}
-
-bool BrowserProxy::SetDefaultContentSetting(ContentSettingsType content_type,
- ContentSetting setting) {
- return SetContentSetting(std::string(), content_type, setting);
-}
-
-bool BrowserProxy::SetContentSetting(const std::string& host,
- ContentSettingsType content_type,
- ContentSetting setting) {
- if (!is_valid())
- return false;
-
- bool result = false;
-
- sender_->Send(new AutomationMsg_SetContentSetting(handle_, host,
- content_type, setting,
- &result));
- return result;
-}
-
bool BrowserProxy::TerminateSession() {
if (!is_valid())
return false;
@@ -520,39 +421,6 @@
return result;
}
-bool BrowserProxy::IsFullscreen(bool* is_fullscreen) {
- DCHECK(is_fullscreen);
-
- if (!is_valid())
- return false;
-
- return sender_->Send(new AutomationMsg_IsFullscreen(handle_, is_fullscreen));
-}
-
-bool BrowserProxy::IsFullscreenBubbleVisible(bool* is_visible) {
- DCHECK(is_visible);
-
- if (!is_valid())
- return false;
-
- return sender_->Send(new AutomationMsg_IsFullscreenBubbleVisible(handle_,
- is_visible));
-}
-
-bool BrowserProxy::ShutdownSessionService() {
- bool did_shutdown = false;
- bool succeeded = sender_->Send(
- new AutomationMsg_ShutdownSessionService(handle_, &did_shutdown));
-
- if (!succeeded) {
- DLOG(ERROR) <<
- "ShutdownSessionService did not complete in a timely fashion";
- return false;
- }
-
- return did_shutdown;
-}
-
bool BrowserProxy::SendJSONRequest(const std::string& request,
int timeout_ms,
std::string* response) {
« no previous file with comments | « chrome/test/automation/browser_proxy.h ('k') | chrome/test/automation/tab_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698