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

Unified Diff: chrome/test/ui/ui_test.h

Issue 5139001: Revert 66350 - Add named testing interface. This allows you to connect to a p... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 1 month 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/ui/named_interface_uitest.cc ('k') | chrome/test/ui/ui_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/ui/ui_test.h
===================================================================
--- chrome/test/ui/ui_test.h (revision 66367)
+++ chrome/test/ui/ui_test.h (working copy)
@@ -38,7 +38,6 @@
class DictionaryValue;
class FilePath;
class GURL;
-class ProxyLauncher;
class ScopedTempDir;
class TabProxy;
@@ -69,21 +68,14 @@
public:
// ********* Utility functions *********
- // Launches the browser only.
- void LaunchBrowser();
-
- // Launches the browser and IPC testing connection in server mode.
+ // Launches the browser and IPC testing server.
void LaunchBrowserAndServer();
- // Launches the IPC testing connection in client mode,
- // which then attempts to connect to a browser.
- void ConnectToRunningBrowser();
-
// Only for pyauto.
void set_command_execution_timeout_ms(int timeout);
- // Overridable so that derived classes can provide their own ProxyLauncher.
- virtual ProxyLauncher* CreateProxyLauncher();
+ // Overridable so that derived classes can provide their own AutomationProxy.
+ virtual AutomationProxy* CreateAutomationProxy(int execution_timeout);
// Closes the browser and IPC testing server.
void CloseBrowserAndServer();
@@ -110,7 +102,7 @@
// Terminates the browser, simulates end of session.
void TerminateBrowser();
- // Tells the browser to navigate to the given URL in the active tab
+ // Tells the browser to navigato to the givne URL in the active tab
// of the first app window.
// Does not wait for the navigation to complete to return.
void NavigateToURLAsync(const GURL& url);
@@ -369,8 +361,8 @@
protected:
AutomationProxy* automation() {
- EXPECT_TRUE(automation_proxy_.get());
- return automation_proxy_.get();
+ EXPECT_TRUE(server_.get());
+ return server_.get();
}
virtual bool ShouldFilterInet() {
@@ -420,7 +412,6 @@
// id on the command line? Default is
// true.
bool enable_file_cookies_; // Enable file cookies, default is true.
- scoped_ptr<ProxyLauncher> launcher_; // Launches browser and AutomationProxy.
ProfileType profile_type_; // Are we using a profile with a
// complex theme?
FilePath websocket_pid_file_; // PID file for websocket server.
@@ -428,8 +419,6 @@
// the browser. Used in ShutdownTest.
private:
- void WaitForBrowserLaunch();
-
bool LaunchBrowserHelper(const CommandLine& arguments,
bool wait,
base::ProcessHandle* process);
@@ -461,7 +450,7 @@
static std::string js_flags_; // Flags passed to the JS engine.
static std::string log_level_; // Logging level.
- scoped_ptr<AutomationProxy> automation_proxy_;
+ scoped_ptr<AutomationProxy> server_;
std::string ui_test_name_;
@@ -479,7 +468,7 @@
virtual void SetUp();
virtual void TearDown();
- virtual ProxyLauncher* CreateProxyLauncher();
+ virtual AutomationProxy* CreateAutomationProxy(int execution_timeout);
// Synchronously launches local http server normally used to run LayoutTests.
void StartHttpServer(const FilePath& root_directory);
« no previous file with comments | « chrome/test/ui/named_interface_uitest.cc ('k') | chrome/test/ui/ui_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698