summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--base/test/launcher/unit_test_launcher.cc6
-rw-r--r--base/test/test_switches.cc1
-rw-r--r--base/test/test_switches.h1
-rw-r--r--base/test/test_timeouts.cc9
-rw-r--r--base/test/test_timeouts.h10
-rw-r--r--media/base/demuxer_perftest.cc35
6 files changed, 20 insertions, 42 deletions
diff --git a/base/test/launcher/unit_test_launcher.cc b/base/test/launcher/unit_test_launcher.cc
index 57552e0..0cbae2f 100644
--- a/base/test/launcher/unit_test_launcher.cc
+++ b/base/test/launcher/unit_test_launcher.cc
@@ -186,7 +186,7 @@ class UnitTestLauncherDelegate : public TestLauncherDelegate {
test_launcher->LaunchChildGTestProcess(
cmd_line,
std::string(),
- TestTimeouts::test_launcher_unit_timeout(),
+ TestTimeouts::test_launcher_timeout(),
use_job_objects_ ? TestLauncher::USE_JOB_OBJECTS : 0,
Bind(&UnitTestLauncherDelegate::SerialGTestCallback,
Unretained(this),
@@ -218,7 +218,7 @@ class UnitTestLauncherDelegate : public TestLauncherDelegate {
// Note: do NOT parse child's stdout to do that, it's known to be
// unreliable (e.g. buffering issues can mix up the output).
base::TimeDelta timeout =
- test_names.size() * TestTimeouts::test_launcher_unit_timeout();
+ test_names.size() * TestTimeouts::test_launcher_timeout();
GTestCallbackState callback_state;
callback_state.test_launcher = test_launcher;
@@ -345,7 +345,7 @@ class UnitTestLauncherDelegate : public TestLauncherDelegate {
// For consistent handling of tests independent of order and other
// factors, mark them as timing out.
if (test_result.elapsed_time >
- TestTimeouts::test_launcher_unit_timeout()) {
+ TestTimeouts::test_launcher_timeout()) {
test_result.status = TestResult::TEST_TIMEOUT;
}
}
diff --git a/base/test/test_switches.cc b/base/test/test_switches.cc
index b23538c..c970fd2 100644
--- a/base/test/test_switches.cc
+++ b/base/test/test_switches.cc
@@ -50,7 +50,6 @@ const char switches::kTestLauncherTotalShards[] =
// Time (in milliseconds) that the tests should wait before timing out.
const char switches::kTestLauncherTimeout[] = "test-launcher-timeout";
-const char switches::kTestLauncherUnitTimeout[] = "test-launcher-unit-timeout";
// TODO(phajdan.jr): Clean up the switch names.
const char switches::kTestTinyTimeout[] = "test-tiny-timeout";
const char switches::kUiTestActionTimeout[] = "ui-test-action-timeout";
diff --git a/base/test/test_switches.h b/base/test/test_switches.h
index b7d0860..c228cf0 100644
--- a/base/test/test_switches.h
+++ b/base/test/test_switches.h
@@ -21,7 +21,6 @@ extern const char kTestLauncherPrintTestStdio[];
extern const char kTestLauncherShardIndex[];
extern const char kTestLauncherTotalShards[];
extern const char kTestLauncherTimeout[];
-extern const char kTestLauncherUnitTimeout[];
extern const char kTestTinyTimeout[];
extern const char kUiTestActionTimeout[];
extern const char kUiTestActionMaxTimeout[];
diff --git a/base/test/test_timeouts.cc b/base/test/test_timeouts.cc
index 92bcf9b..7392450 100644
--- a/base/test/test_timeouts.cc
+++ b/base/test/test_timeouts.cc
@@ -72,7 +72,6 @@ int TestTimeouts::action_max_timeout_ms_ = 45000;
int TestTimeouts::action_max_timeout_ms_ = 30000;
#endif // NDEBUG
-int TestTimeouts::test_launcher_unit_timeout_ms_ = 5000;
int TestTimeouts::test_launcher_timeout_ms_ = 45000;
// static
@@ -99,16 +98,12 @@ void TestTimeouts::Initialize() {
&action_max_timeout_ms_);
// Test launcher timeout is independent from anything above action timeout.
- InitializeTimeout(switches::kTestLauncherUnitTimeout, action_timeout_ms_,
- &test_launcher_unit_timeout_ms_);
- InitializeTimeout(switches::kTestLauncherTimeout,
- test_launcher_unit_timeout_ms_,
+ InitializeTimeout(switches::kTestLauncherTimeout, action_timeout_ms_,
&test_launcher_timeout_ms_);
// The timeout values should be increasing in the right order.
CHECK(tiny_timeout_ms_ <= action_timeout_ms_);
CHECK(action_timeout_ms_ <= action_max_timeout_ms_);
- CHECK(action_timeout_ms_ <= test_launcher_unit_timeout_ms_);
- CHECK(test_launcher_unit_timeout_ms_ <= test_launcher_timeout_ms_);
+ CHECK(action_timeout_ms_ <= test_launcher_timeout_ms_);
}
diff --git a/base/test/test_timeouts.h b/base/test/test_timeouts.h
index 5f79264..2819e4a 100644
--- a/base/test/test_timeouts.h
+++ b/base/test/test_timeouts.h
@@ -38,14 +38,7 @@ class TestTimeouts {
return base::TimeDelta::FromMilliseconds(action_max_timeout_ms_);
}
- // Timeout for a single unit test launched using built-in test launcher.
- // Do not use outside of the test launcher.
- static base::TimeDelta test_launcher_unit_timeout() {
- DCHECK(initialized_);
- return base::TimeDelta::FromMilliseconds(test_launcher_unit_timeout_ms_);
- }
-
- // Timeout for a single test launched using built-in test launcher.
+ // Timeout for a single test launched used built-in test launcher.
// Do not use outside of the test launcher.
static base::TimeDelta test_launcher_timeout() {
DCHECK(initialized_);
@@ -58,7 +51,6 @@ class TestTimeouts {
static int tiny_timeout_ms_;
static int action_timeout_ms_;
static int action_max_timeout_ms_;
- static int test_launcher_unit_timeout_ms_;
static int test_launcher_timeout_ms_;
DISALLOW_IMPLICIT_CONSTRUCTORS(TestTimeouts);
diff --git a/media/base/demuxer_perftest.cc b/media/base/demuxer_perftest.cc
index 144b053..ad67363 100644
--- a/media/base/demuxer_perftest.cc
+++ b/media/base/demuxer_perftest.cc
@@ -207,33 +207,26 @@ static void RunDemuxerBenchmark(const std::string& filename) {
true);
}
-class DemuxerPerfTest : public testing::TestWithParam<const char*> {
-};
-
-const char* kDemuxerBenchmarks[] = {
- "bear.ogv",
- "bear-640x360.webm",
- "sfx_s16le.wav",
+#if defined(OS_WIN)
+// http://crbug.com/399002
+#define MAYBE_Demuxer DISABLED_Demuxer
+#else
+#define MAYBE_Demuxer Demuxer
+#endif
+TEST(DemuxerPerfTest, MAYBE_Demuxer) {
+ RunDemuxerBenchmark("bear.ogv");
+ RunDemuxerBenchmark("bear-640x360.webm");
+ RunDemuxerBenchmark("sfx_s16le.wav");
#if defined(USE_PROPRIETARY_CODECS)
- "bear-1280x720.mp4",
- "sfx.mp3",
+ RunDemuxerBenchmark("bear-1280x720.mp4");
+ RunDemuxerBenchmark("sfx.mp3");
#endif
#if defined(OS_CHROMEOS)
- "bear.flac",
+ RunDemuxerBenchmark("bear.flac");
#endif
#if defined(USE_PROPRIETARY_CODECS) && defined(OS_CHROMEOS)
- "bear.avi",
+ RunDemuxerBenchmark("bear.avi");
#endif
-};
-
-TEST_P(DemuxerPerfTest, Demuxer) {
- RunDemuxerBenchmark(GetParam());
}
-#if !defined(OS_WIN)
-// http://crbug.com/399002
-INSTANTIATE_TEST_CASE_P(, DemuxerPerfTest,
- testing::ValuesIn(kDemuxerBenchmarks));
-#endif // !defined(OS_WIN)
-
} // namespace media