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

Side by Side Diff: webrtc/modules/audio_device/android/audio_device_unittest.cc

Issue 2517383006: Re-enables AudioDeviceTest.StartStopPlayout on Android (Closed)
Patch Set: Created 4 years 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 798 matching lines...) Expand 10 before | Expand all | Expand 10 after
809 const int default_volume = GetSpeakerVolume(); 809 const int default_volume = GetSpeakerVolume();
810 const int max_volume = GetMaxSpeakerVolume(); 810 const int max_volume = GetMaxSpeakerVolume();
811 EXPECT_EQ(0, audio_device()->SetSpeakerVolume(max_volume)); 811 EXPECT_EQ(0, audio_device()->SetSpeakerVolume(max_volume));
812 int new_volume = GetSpeakerVolume(); 812 int new_volume = GetSpeakerVolume();
813 EXPECT_EQ(new_volume, max_volume); 813 EXPECT_EQ(new_volume, max_volume);
814 EXPECT_EQ(0, audio_device()->SetSpeakerVolume(default_volume)); 814 EXPECT_EQ(0, audio_device()->SetSpeakerVolume(default_volume));
815 } 815 }
816 816
817 // Tests that playout can be initiated, started and stopped. No audio callback 817 // Tests that playout can be initiated, started and stopped. No audio callback
818 // is registered in this test. 818 // is registered in this test.
819 // Flaky on our trybots makes this test unusable. 819 TEST_F(AudioDeviceTest, StartStopPlayout) {
820 // https://code.google.com/p/webrtc/issues/detail?id=5046
821 TEST_F(AudioDeviceTest, DISABLED_StartStopPlayout) {
822 StartPlayout(); 820 StartPlayout();
823 StopPlayout(); 821 StopPlayout();
824 StartPlayout(); 822 StartPlayout();
825 StopPlayout(); 823 StopPlayout();
826 } 824 }
827 825
828 // Tests that recording can be initiated, started and stopped. No audio callback 826 // Tests that recording can be initiated, started and stopped. No audio callback
829 // is registered in this test. 827 // is registered in this test.
830 TEST_F(AudioDeviceTest, StartStopRecording) { 828 TEST_F(AudioDeviceTest, StartStopRecording) {
831 StartRecording(); 829 StartRecording();
(...skipping 184 matching lines...) Expand 10 before | Expand all | Expand 10 after
1016 StopPlayout(); 1014 StopPlayout();
1017 StopRecording(); 1015 StopRecording();
1018 // Verify that the correct number of transmitted impulses are detected. 1016 // Verify that the correct number of transmitted impulses are detected.
1019 EXPECT_EQ(latency_audio_stream->num_latency_values(), 1017 EXPECT_EQ(latency_audio_stream->num_latency_values(),
1020 static_cast<size_t>( 1018 static_cast<size_t>(
1021 kImpulseFrequencyInHz * kMeasureLatencyTimeInSec - 1)); 1019 kImpulseFrequencyInHz * kMeasureLatencyTimeInSec - 1));
1022 latency_audio_stream->PrintResults(); 1020 latency_audio_stream->PrintResults();
1023 } 1021 }
1024 1022
1025 } // namespace webrtc 1023 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698