Index: media/cast/audio_sender/audio_sender_unittest.cc |
diff --git a/media/cast/audio_sender/audio_sender_unittest.cc b/media/cast/audio_sender/audio_sender_unittest.cc |
index f7e5c0a91479ad3497b9fb701c1c6227f361bf51..7c9b154a3c190a6cadb58f8f1826abbb88c10902 100644 |
--- a/media/cast/audio_sender/audio_sender_unittest.cc |
+++ b/media/cast/audio_sender/audio_sender_unittest.cc |
@@ -33,6 +33,12 @@ class TestPacketSender : public transport::PacketSender { |
if (Rtcp::IsRtcpPacket(&packet->data[0], packet->data.size())) { |
++number_of_rtcp_packets_; |
} else { |
+ // Check that at least one RTCP packet was sent before the first RTP |
+ // packet. This confirms that the receiver will have the necessary lip |
+ // sync info before it has to calculate the playout time of the first |
+ // frame. |
+ if (number_of_rtp_packets_ == 0) |
+ EXPECT_LE(1, number_of_rtcp_packets_); |
++number_of_rtp_packets_; |
} |
return true; |
@@ -88,7 +94,7 @@ class AudioSenderTest : public ::testing::Test { |
virtual ~AudioSenderTest() {} |
static void UpdateCastTransportStatus(transport::CastTransportStatus status) { |
- EXPECT_EQ(status, transport::TRANSPORT_AUDIO_INITIALIZED); |
+ EXPECT_EQ(transport::TRANSPORT_AUDIO_INITIALIZED, status); |
} |
base::SimpleTestTickClock* testing_clock_; // Owned by CastEnvironment. |
@@ -110,9 +116,8 @@ TEST_F(AudioSenderTest, Encode20ms) { |
audio_sender_->InsertAudio(bus.Pass(), testing_clock_->NowTicks()); |
task_runner_->RunTasks(); |
- EXPECT_GE( |
- transport_.number_of_rtp_packets() + transport_.number_of_rtcp_packets(), |
- 1); |
+ EXPECT_LE(1, transport_.number_of_rtp_packets()); |
+ EXPECT_LE(1, transport_.number_of_rtcp_packets()); |
} |
TEST_F(AudioSenderTest, RtcpTimer) { |
@@ -131,8 +136,8 @@ TEST_F(AudioSenderTest, RtcpTimer) { |
base::TimeDelta::FromMilliseconds(1 + kDefaultRtcpIntervalMs * 3 / 2); |
testing_clock_->Advance(max_rtcp_timeout); |
task_runner_->RunTasks(); |
- EXPECT_GE(transport_.number_of_rtp_packets(), 1); |
- EXPECT_EQ(transport_.number_of_rtcp_packets(), 1); |
+ EXPECT_LE(1, transport_.number_of_rtp_packets()); |
+ EXPECT_LE(1, transport_.number_of_rtcp_packets()); |
} |
} // namespace cast |