Commit 7e446bb7 authored by tommi@chromium.org's avatar tommi@chromium.org

Remove the RecordWithSlowSink test. The test has problems running on the...

Remove the RecordWithSlowSink test.  The test has problems running on the build bots and after discussing its benefits, we decided that we don't need to test this level of tolerance and that the Record test is enough.

BUG=107546
Review URL: http://codereview.chromium.org/8957006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114628 0039d316-1c4b-4281-b951-d872f2087c98
parent b0cfc1cf
...@@ -190,34 +190,3 @@ TEST(AudioInputTest, Record) { ...@@ -190,34 +190,3 @@ TEST(AudioInputTest, Record) {
ais->Stop(); ais->Stop();
ais->Close(); ais->Close();
} }
// Test a recording sequence with delays in the audio callback.
// TODO(joth): See bug 107546. This fails on slow bots. Once fixed, remove the
// CHROME_HEADLESS check below.
TEST(AudioInputTest, RecordWithSlowSink) {
scoped_refptr<AudioManager> audio_man(AudioManager::Create());
if (!CanRunAudioTests(audio_man.get()))
return;
scoped_ptr<base::Environment> env(base::Environment::Create());
if (env->HasVar("CHROME_HEADLESS"))
return;
MessageLoop message_loop(MessageLoop::TYPE_DEFAULT);
AudioInputStream* ais = CreateTestAudioInputStream(audio_man.get());
EXPECT_TRUE(ais->Open());
// We should normally get a callback every 50ms, and a 20ms delay inside each
// callback should not change this sequence.
TestInputCallbackBlocking test_callback(kSamplesPerPacket * 4, 0, 20);
ais->Start(&test_callback);
// Verify at least 500ms worth of audio was recorded, after giving sufficient
// extra time.
message_loop.PostDelayedTask(FROM_HERE, MessageLoop::QuitClosure(), 590);
message_loop.Run();
EXPECT_GE(test_callback.callback_count(), 10);
EXPECT_FALSE(test_callback.had_error());
ais->Stop();
ais->Close();
}
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment