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

Unified Diff: chrome/test/test_launcher/out_of_proc_test_runner.cc

Issue 197045: Implement a way to run each interactive UI test isolated (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
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
« no previous file with comments | « chrome/test/test_launcher/in_proc_test_runner.cc ('k') | chrome/test/test_launcher/run_all_unittests.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/test_launcher/out_of_proc_test_runner.cc
===================================================================
--- chrome/test/test_launcher/out_of_proc_test_runner.cc (revision 0)
+++ chrome/test/test_launcher/out_of_proc_test_runner.cc (working copy)
@@ -8,23 +8,22 @@
#include "base/logging.h"
#include "base/process_util.h"
-#include "chrome/test/browser/browser_test_runner.h"
+#include "chrome/test/test_launcher/test_runner.h"
#include "chrome/test/unit/chrome_test_suite.h"
-// This version of the browser test launcher forks a new process for each test
-// it runs.
+// This version of the test launcher forks a new process for each test it runs.
namespace {
const wchar_t* const kGTestListTestsFlag = L"gtest_list_tests";
const wchar_t* const kChildProcessFlag = L"child";
-class OutOfProcBrowserTestRunner : public browser_tests::BrowserTestRunner {
+class OutOfProcTestRunner : public tests::TestRunner {
public:
- OutOfProcBrowserTestRunner() {
+ OutOfProcTestRunner() {
}
- virtual ~OutOfProcBrowserTestRunner() {
+ virtual ~OutOfProcTestRunner() {
}
bool Init() {
@@ -55,20 +54,19 @@
}
private:
- DISALLOW_COPY_AND_ASSIGN(OutOfProcBrowserTestRunner);
+ DISALLOW_COPY_AND_ASSIGN(OutOfProcTestRunner);
};
-class OutOfProcBrowserTestRunnerFactory
- : public browser_tests::BrowserTestRunnerFactory {
+class OutOfProcTestRunnerFactory : public tests::TestRunnerFactory {
public:
- OutOfProcBrowserTestRunnerFactory() { }
+ OutOfProcTestRunnerFactory() { }
- virtual browser_tests::BrowserTestRunner* CreateBrowserTestRunner() const {
- return new OutOfProcBrowserTestRunner();
+ virtual tests::TestRunner* CreateTestRunner() const {
+ return new OutOfProcTestRunner();
}
private:
- DISALLOW_COPY_AND_ASSIGN(OutOfProcBrowserTestRunnerFactory);
+ DISALLOW_COPY_AND_ASSIGN(OutOfProcTestRunnerFactory);
};
} // namespace
@@ -83,6 +81,6 @@
if (command_line->HasSwitch(kGTestListTestsFlag))
return ChromeTestSuite(argc, argv).Run();
- OutOfProcBrowserTestRunnerFactory test_runner_factory;
- return browser_tests::RunTests(test_runner_factory) ? 0 : 1;
+ OutOfProcTestRunnerFactory test_runner_factory;
+ return tests::RunTests(test_runner_factory) ? 0 : 1;
}
« no previous file with comments | « chrome/test/test_launcher/in_proc_test_runner.cc ('k') | chrome/test/test_launcher/run_all_unittests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698