diff options
-rw-r--r-- | base/base.gyp | 4 | ||||
-rw-r--r-- | base/test/test_suite.cc | 3 | ||||
-rw-r--r-- | base/test/test_switches.cc | 17 | ||||
-rw-r--r-- | base/test/test_switches.h | 22 | ||||
-rw-r--r-- | base/test/test_timeouts.cc (renamed from chrome/test/test_timeouts.cc) | 4 | ||||
-rw-r--r-- | base/test/test_timeouts.h (renamed from chrome/test/test_timeouts.h) | 6 | ||||
-rw-r--r-- | chrome/browser/autofill/autofill_download_unittest.cc | 3 | ||||
-rw-r--r-- | chrome/browser/cocoa/cocoa_test_helper.mm | 2 | ||||
-rw-r--r-- | chrome/browser/debugger/devtools_remote_listen_socket_unittest.cc | 2 | ||||
-rw-r--r-- | chrome/browser/importer/firefox_importer_unittest_utils_mac.cc | 2 | ||||
-rw-r--r-- | chrome/chrome_tests.gypi | 2 | ||||
-rw-r--r-- | chrome/test/live_sync/profile_sync_service_test_harness.cc | 2 | ||||
-rw-r--r-- | chrome/test/mini_installer_test/mini_installer_test_util.cc | 2 | ||||
-rw-r--r-- | chrome/test/mini_installer_test/run_all_unittests.cc | 2 | ||||
-rw-r--r-- | chrome/test/test_switches.cc | 12 | ||||
-rw-r--r-- | chrome/test/test_switches.h | 7 | ||||
-rw-r--r-- | chrome/test/ui/ui_test.h | 2 | ||||
-rw-r--r-- | chrome/test/unit/chrome_test_suite.cc | 3 |
18 files changed, 59 insertions, 38 deletions
diff --git a/base/base.gyp b/base/base.gyp index b919d6e..f24279a 100644 --- a/base/base.gyp +++ b/base/base.gyp @@ -261,6 +261,10 @@ 'test/test_file_util_win.cc', 'test/test_suite.cc', 'test/test_suite.h', + 'test/test_switches.cc', + 'test/test_switches.h', + 'test/test_timeouts.cc', + 'test/test_timeouts.h', ], }, { diff --git a/base/test/test_suite.cc b/base/test/test_suite.cc index 6df1fb9..a08bbd9 100644 --- a/base/test/test_suite.cc +++ b/base/test/test_suite.cc @@ -19,6 +19,7 @@ #include "base/scoped_nsautorelease_pool.h" #include "base/scoped_ptr.h" #include "base/test/multiprocess_test.h" +#include "base/test/test_timeouts.h" #include "base/time.h" #include "testing/gtest/include/gtest/gtest.h" #include "testing/multiprocess_func_list.h" @@ -208,6 +209,8 @@ void TestSuite::Initialize() { #endif // defined(USE_NSS) CatchMaybeTests(); + + TestTimeouts::Initialize(); } void TestSuite::Shutdown() { diff --git a/base/test/test_switches.cc b/base/test/test_switches.cc new file mode 100644 index 0000000..1dfd63f --- /dev/null +++ b/base/test/test_switches.cc @@ -0,0 +1,17 @@ +// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +#include "base/test/test_switches.h" + +// Time (in milliseconds) for which we wait for live operations to complete. +const char switches::kLiveOperationTimeout[] = "live-operation-timeout"; + +// Time (in milliseconds) that the tests should wait before timing out. +// TODO(phajdan.jr): Clean up the switch names. +const char switches::kTestLargeTimeout[] = "test-large-timeout"; +const char switches::kUiTestActionTimeout[] = "ui-test-action-timeout"; +const char switches::kUiTestActionMaxTimeout[] = "ui-test-action-max-timeout"; +const char switches::kUiTestCommandExecutionTimeout[] = "ui-test-timeout"; +const char switches::kUiTestTerminateTimeout[] = "ui-test-terminate-timeout"; +const char switches::kUiTestTimeout[] = "test-timeout"; diff --git a/base/test/test_switches.h b/base/test/test_switches.h new file mode 100644 index 0000000..2194596 --- /dev/null +++ b/base/test/test_switches.h @@ -0,0 +1,22 @@ +// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +#ifndef BASE_TEST_TEST_SWITCHES_H_ +#define BASE_TEST_TEST_SWITCHES_H_ + +namespace switches { + +// All switches in alphabetical order. The switches should be documented +// alongside the definition of their values in the .cc file. +extern const char kLiveOperationTimeout[]; +extern const char kTestLargeTimeout[]; +extern const char kUiTestActionTimeout[]; +extern const char kUiTestActionMaxTimeout[]; +extern const char kUiTestCommandExecutionTimeout[]; +extern const char kUiTestTerminateTimeout[]; +extern const char kUiTestTimeout[]; + +} // namespace switches + +#endif // BASE_TEST_TEST_SWITCHES_H_ diff --git a/chrome/test/test_timeouts.cc b/base/test/test_timeouts.cc index ee096fb..7ee1b70 100644 --- a/chrome/test/test_timeouts.cc +++ b/base/test/test_timeouts.cc @@ -2,12 +2,12 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -#include "chrome/test/test_timeouts.h" +#include "base/test/test_timeouts.h" #include "base/command_line.h" #include "base/logging.h" #include "base/string_number_conversions.h" -#include "chrome/test/test_switches.h" +#include "base/test/test_switches.h" namespace { diff --git a/chrome/test/test_timeouts.h b/base/test/test_timeouts.h index dc9084b..d49ca2c 100644 --- a/chrome/test/test_timeouts.h +++ b/base/test/test_timeouts.h @@ -2,8 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -#ifndef CHROME_TEST_TEST_TIMEOUTS_H_ -#define CHROME_TEST_TEST_TIMEOUTS_H_ +#ifndef BASE_TEST_TEST_TIMEOUTS_H_ +#define BASE_TEST_TEST_TIMEOUTS_H_ #include "base/basictypes.h" @@ -63,4 +63,4 @@ class TestTimeouts { DISALLOW_IMPLICIT_CONSTRUCTORS(TestTimeouts); }; -#endif // CHROME_TEST_TEST_TIMEOUTS_H_ +#endif // BASE_TEST_TEST_TIMEOUTS_H_ diff --git a/chrome/browser/autofill/autofill_download_unittest.cc b/chrome/browser/autofill/autofill_download_unittest.cc index 572fc1c..deb9b49 100644 --- a/chrome/browser/autofill/autofill_download_unittest.cc +++ b/chrome/browser/autofill/autofill_download_unittest.cc @@ -5,10 +5,10 @@ #include <list> #include "base/string_util.h" +#include "base/test/test_timeouts.h" #include "base/utf_string_conversions.h" #include "chrome/browser/autofill/autofill_download.h" #include "chrome/common/net/test_url_fetcher_factory.h" -#include "chrome/test/test_timeouts.h" #include "chrome/test/testing_profile.h" #include "net/url_request/url_request_status.h" #include "testing/gtest/include/gtest/gtest.h" @@ -295,4 +295,3 @@ TEST(AutoFillDownloadTest, QueryAndUploadTest) { } } // namespace - diff --git a/chrome/browser/cocoa/cocoa_test_helper.mm b/chrome/browser/cocoa/cocoa_test_helper.mm index 25ee845..46029aa 100644 --- a/chrome/browser/cocoa/cocoa_test_helper.mm +++ b/chrome/browser/cocoa/cocoa_test_helper.mm @@ -5,8 +5,8 @@ #import "chrome/browser/cocoa/cocoa_test_helper.h" #include "base/logging.h" +#include "base/test/test_timeouts.h" #import "chrome/browser/chrome_browser_application_mac.h" -#include "chrome/test/test_timeouts.h" @implementation CocoaTestHelperWindow diff --git a/chrome/browser/debugger/devtools_remote_listen_socket_unittest.cc b/chrome/browser/debugger/devtools_remote_listen_socket_unittest.cc index a6c3583..a3eb3ef 100644 --- a/chrome/browser/debugger/devtools_remote_listen_socket_unittest.cc +++ b/chrome/browser/debugger/devtools_remote_listen_socket_unittest.cc @@ -10,7 +10,7 @@ #endif #include "base/eintr_wrapper.h" -#include "chrome/test/test_timeouts.h" +#include "base/test/test_timeouts.h" #include "net/base/net_util.h" #include "testing/platform_test.h" diff --git a/chrome/browser/importer/firefox_importer_unittest_utils_mac.cc b/chrome/browser/importer/firefox_importer_unittest_utils_mac.cc index cf93d9e..4d21435 100644 --- a/chrome/browser/importer/firefox_importer_unittest_utils_mac.cc +++ b/chrome/browser/importer/firefox_importer_unittest_utils_mac.cc @@ -9,8 +9,8 @@ #include "base/debug_on_start.h" #include "base/file_path.h" #include "base/message_loop.h" +#include "base/test/test_timeouts.h" #include "chrome/browser/importer/firefox_importer_utils.h" -#include "chrome/test/test_timeouts.h" #include "ipc/ipc_channel.h" #include "ipc/ipc_descriptors.h" #include "ipc/ipc_message.h" diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi index 297cf85..25756db 100644 --- a/chrome/chrome_tests.gypi +++ b/chrome/chrome_tests.gypi @@ -119,8 +119,6 @@ 'test/test_location_bar.h', 'test/test_switches.cc', 'test/test_switches.h', - 'test/test_timeouts.cc', - 'test/test_timeouts.h', 'test/testing_pref_service.cc', 'test/testing_pref_service.h', 'test/testing_profile.cc', diff --git a/chrome/test/live_sync/profile_sync_service_test_harness.cc b/chrome/test/live_sync/profile_sync_service_test_harness.cc index dbf3ef1..d814541 100644 --- a/chrome/test/live_sync/profile_sync_service_test_harness.cc +++ b/chrome/test/live_sync/profile_sync_service_test_harness.cc @@ -3,6 +3,7 @@ // found in the LICENSE file. #include "base/message_loop.h" +#include "base/test/test_timeouts.h" #include "chrome/browser/browser.h" #include "chrome/browser/defaults.h" #include "chrome/browser/prefs/pref_service.h" @@ -15,7 +16,6 @@ #include "chrome/common/net/gaia/google_service_auth_error.h" #include "chrome/common/pref_names.h" #include "chrome/test/live_sync/profile_sync_service_test_harness.h" -#include "chrome/test/test_timeouts.h" #include "chrome/test/ui_test_utils.h" #include "testing/gtest/include/gtest/gtest.h" diff --git a/chrome/test/mini_installer_test/mini_installer_test_util.cc b/chrome/test/mini_installer_test/mini_installer_test_util.cc index e86b3e1..697f6d0 100644 --- a/chrome/test/mini_installer_test/mini_installer_test_util.cc +++ b/chrome/test/mini_installer_test/mini_installer_test_util.cc @@ -9,11 +9,11 @@ #include "base/platform_thread.h" #include "base/process_util.h" #include "base/string_util.h" +#include "base/test/test_timeouts.h" #include "base/time.h" #include "base/utf_string_conversions.h" #include "chrome/installer/util/logging_installer.h" #include "chrome/test/mini_installer_test/mini_installer_test_constants.h" -#include "chrome/test/test_timeouts.h" #include "testing/gtest/include/gtest/gtest.h" // Change current directory so that chrome.dll from current folder diff --git a/chrome/test/mini_installer_test/run_all_unittests.cc b/chrome/test/mini_installer_test/run_all_unittests.cc index 13229d2..f8f1d42 100644 --- a/chrome/test/mini_installer_test/run_all_unittests.cc +++ b/chrome/test/mini_installer_test/run_all_unittests.cc @@ -6,10 +6,10 @@ #include "base/file_util.h" #include "base/process_util.h" #include "base/test/test_suite.h" +#include "base/test/test_timeouts.h" #include "chrome/common/chrome_switches.h" #include "chrome/installer/util/util_constants.h" #include "chrome/test/mini_installer_test/mini_installer_test_constants.h" -#include "chrome/test/test_timeouts.h" #include "chrome_mini_installer.h" void BackUpProfile(bool chrome_frame) { diff --git a/chrome/test/test_switches.cc b/chrome/test/test_switches.cc index fdfadb3..aa93da4 100644 --- a/chrome/test/test_switches.cc +++ b/chrome/test/test_switches.cc @@ -10,17 +10,5 @@ const char switches::kExtraChromeFlags[] = "extra-chrome-flags"; // Enable displaying error dialogs (for debugging). const char switches::kEnableErrorDialogs[] = "enable-errdialogs"; -// Time (in milliseconds) for which we wait for live operations to complete. -const char switches::kLiveOperationTimeout[] = "live-operation-timeout"; - // Number of iterations that page cycler tests will run. const char switches::kPageCyclerIterations[] = "page-cycler-iterations"; - -// Time (in milliseconds) that the ui_tests should wait before timing out. -// TODO(phajdan.jr): Clean up the switch names. -const char switches::kTestLargeTimeout[] = "test-large-timeout"; -const char switches::kUiTestActionTimeout[] = "ui-test-action-timeout"; -const char switches::kUiTestActionMaxTimeout[] = "ui-test-action-max-timeout"; -const char switches::kUiTestCommandExecutionTimeout[] = "ui-test-timeout"; -const char switches::kUiTestTerminateTimeout[] = "ui-test-terminate-timeout"; -const char switches::kUiTestTimeout[] = "test-timeout"; diff --git a/chrome/test/test_switches.h b/chrome/test/test_switches.h index 07824fe..68613d4 100644 --- a/chrome/test/test_switches.h +++ b/chrome/test/test_switches.h @@ -11,14 +11,7 @@ namespace switches { // alongside the definition of their values in the .cc file. extern const char kExtraChromeFlags[]; extern const char kEnableErrorDialogs[]; -extern const char kLiveOperationTimeout[]; extern const char kPageCyclerIterations[]; -extern const char kTestLargeTimeout[]; -extern const char kUiTestActionTimeout[]; -extern const char kUiTestActionMaxTimeout[]; -extern const char kUiTestCommandExecutionTimeout[]; -extern const char kUiTestTerminateTimeout[]; -extern const char kUiTestTimeout[]; } // namespace switches diff --git a/chrome/test/ui/ui_test.h b/chrome/test/ui/ui_test.h index 70825fb..b17c740 100644 --- a/chrome/test/ui/ui_test.h +++ b/chrome/test/ui/ui_test.h @@ -24,13 +24,13 @@ #include "base/message_loop.h" #include "base/process.h" #include "base/scoped_ptr.h" +#include "base/test/test_timeouts.h" #include "base/time.h" #include "build/build_config.h" // TODO(evanm): we should be able to just forward-declare // AutomationProxy here, but many files that #include this one don't // themselves #include automation_proxy.h. #include "chrome/test/automation/automation_proxy.h" -#include "chrome/test/test_timeouts.h" #include "testing/platform_test.h" class AutomationProxy; diff --git a/chrome/test/unit/chrome_test_suite.cc b/chrome/test/unit/chrome_test_suite.cc index 5f8d911..d9b9c65 100644 --- a/chrome/test/unit/chrome_test_suite.cc +++ b/chrome/test/unit/chrome_test_suite.cc @@ -16,7 +16,6 @@ #include "chrome/common/chrome_paths.h" #include "chrome/common/chrome_switches.h" #include "chrome/common/url_constants.h" -#include "chrome/test/test_timeouts.h" #include "chrome/test/testing_browser_process.h" #include "net/base/net_errors.h" @@ -98,8 +97,6 @@ void ChromeTestSuite::Initialize() { app::RegisterPathProvider(); g_browser_process = new TestingBrowserProcess; - TestTimeouts::Initialize(); - if (!browser_dir_.empty()) { PathService::Override(base::DIR_EXE, browser_dir_); PathService::Override(base::DIR_MODULE, browser_dir_); |