Index: talk/media/webrtc/webrtcvoiceengine_unittest.cc |
diff --git a/talk/media/webrtc/webrtcvoiceengine_unittest.cc b/talk/media/webrtc/webrtcvoiceengine_unittest.cc |
index fc373db916a361b9b8ffa19938f2e5a2ef0e9291..82b2152a1c5ddf3aa03b0a610f571bf11aa13056 100644 |
--- a/talk/media/webrtc/webrtcvoiceengine_unittest.cc |
+++ b/talk/media/webrtc/webrtcvoiceengine_unittest.cc |
@@ -59,8 +59,8 @@ static const cricket::AudioCodec* const kAudioCodecs[] = { |
&kCn8000Codec, &kCn16000Codec, &kTelephoneEventCodec, |
}; |
const char kRingbackTone[] = "RIFF____WAVE____ABCD1234"; |
-static uint32 kSsrc1 = 0x99; |
-static uint32 kSsrc2 = 0x98; |
+static uint32_t kSsrc1 = 0x99; |
+static uint32_t kSsrc2 = 0x98; |
class FakeVoEWrapper : public cricket::VoEWrapper { |
public: |
@@ -101,7 +101,7 @@ class WebRtcVoiceEngineTestFake : public testing::Test { |
channel->SignalMediaError.connect( |
this, &ChannelErrorListener::OnVoiceChannelError); |
} |
- void OnVoiceChannelError(uint32 ssrc, |
+ void OnVoiceChannelError(uint32_t ssrc, |
cricket::VoiceMediaChannel::Error error) { |
ssrc_ = ssrc; |
error_ = error; |
@@ -110,15 +110,13 @@ class WebRtcVoiceEngineTestFake : public testing::Test { |
ssrc_ = 0; |
error_ = cricket::VoiceMediaChannel::ERROR_NONE; |
} |
- uint32 ssrc() const { |
- return ssrc_; |
- } |
+ uint32_t ssrc() const { return ssrc_; } |
cricket::VoiceMediaChannel::Error error() const { |
return error_; |
} |
private: |
- uint32 ssrc_; |
+ uint32_t ssrc_; |
cricket::VoiceMediaChannel::Error error_; |
}; |
@@ -151,7 +149,7 @@ class WebRtcVoiceEngineTestFake : public testing::Test { |
EXPECT_TRUE(SetupEngine()); |
// Remove stream added in Setup, which is corresponding to default channel. |
int default_channel_num = voe_.GetLastChannel(); |
- uint32 default_send_ssrc = 0u; |
+ uint32_t default_send_ssrc = 0u; |
EXPECT_EQ(0, voe_.GetLocalSSRC(default_channel_num, default_send_ssrc)); |
EXPECT_EQ(kSsrc1, default_send_ssrc); |
EXPECT_TRUE(channel_->RemoveSendStream(default_send_ssrc)); |
@@ -168,7 +166,7 @@ class WebRtcVoiceEngineTestFake : public testing::Test { |
engine_.Terminate(); |
} |
- void TestInsertDtmf(uint32 ssrc, bool caller) { |
+ void TestInsertDtmf(uint32_t ssrc, bool caller) { |
EXPECT_TRUE(engine_.Init(rtc::Thread::Current())); |
channel_ = engine_.CreateChannel(&call_, cricket::AudioOptions()); |
EXPECT_TRUE(channel_ != nullptr); |
@@ -1896,7 +1894,7 @@ TEST_F(WebRtcVoiceEngineTestFake, SendAndPlayout) { |
TEST_F(WebRtcVoiceEngineTestFake, CreateAndDeleteMultipleSendStreams) { |
SetupForMultiSendStream(); |
- static const uint32 kSsrcs4[] = {1, 2, 3, 4}; |
+ static const uint32_t kSsrcs4[] = {1, 2, 3, 4}; |
// Set the global state for sending. |
EXPECT_TRUE(channel_->SetSend(cricket::SEND_MICROPHONE)); |
@@ -1931,7 +1929,7 @@ TEST_F(WebRtcVoiceEngineTestFake, CreateAndDeleteMultipleSendStreams) { |
TEST_F(WebRtcVoiceEngineTestFake, SetSendCodecsWithMultipleSendStreams) { |
SetupForMultiSendStream(); |
- static const uint32 kSsrcs4[] = {1, 2, 3, 4}; |
+ static const uint32_t kSsrcs4[] = {1, 2, 3, 4}; |
// Create send streams. |
for (unsigned int i = 0; i < ARRAY_SIZE(kSsrcs4); ++i) { |
EXPECT_TRUE(channel_->AddSendStream( |
@@ -1970,7 +1968,7 @@ TEST_F(WebRtcVoiceEngineTestFake, SetSendCodecsWithMultipleSendStreams) { |
TEST_F(WebRtcVoiceEngineTestFake, SetSendWithMultipleSendStreams) { |
SetupForMultiSendStream(); |
- static const uint32 kSsrcs4[] = {1, 2, 3, 4}; |
+ static const uint32_t kSsrcs4[] = {1, 2, 3, 4}; |
// Create the send channels and they should be a SEND_NOTHING date. |
for (unsigned int i = 0; i < ARRAY_SIZE(kSsrcs4); ++i) { |
EXPECT_TRUE(channel_->AddSendStream( |
@@ -2000,7 +1998,7 @@ TEST_F(WebRtcVoiceEngineTestFake, SetSendWithMultipleSendStreams) { |
TEST_F(WebRtcVoiceEngineTestFake, GetStatsWithMultipleSendStreams) { |
SetupForMultiSendStream(); |
- static const uint32 kSsrcs4[] = {1, 2, 3, 4}; |
+ static const uint32_t kSsrcs4[] = {1, 2, 3, 4}; |
// Create send streams. |
for (unsigned int i = 0; i < ARRAY_SIZE(kSsrcs4); ++i) { |
EXPECT_TRUE(channel_->AddSendStream( |
@@ -2436,7 +2434,7 @@ TEST_F(WebRtcVoiceEngineTestFake, RecvWithMultipleStreams) { |
char packets[4][sizeof(kPcmuFrame)]; |
for (size_t i = 0; i < ARRAY_SIZE(packets); ++i) { |
memcpy(packets[i], kPcmuFrame, sizeof(kPcmuFrame)); |
- rtc::SetBE32(packets[i] + 8, static_cast<uint32>(i)); |
+ rtc::SetBE32(packets[i] + 8, static_cast<uint32_t>(i)); |
} |
EXPECT_TRUE(voe_.CheckNoPacket(channel_num1)); |
EXPECT_TRUE(voe_.CheckNoPacket(channel_num2)); |
@@ -3147,7 +3145,7 @@ TEST_F(WebRtcVoiceEngineTestFake, SetOutputScaling) { |
} |
TEST_F(WebRtcVoiceEngineTestFake, SetsSyncGroupFromSyncLabel) { |
- const uint32 kAudioSsrc = 123; |
+ const uint32_t kAudioSsrc = 123; |
const std::string kSyncLabel = "AvSyncLabel"; |
EXPECT_TRUE(SetupEngine()); |
@@ -3171,21 +3169,21 @@ TEST_F(WebRtcVoiceEngineTestFake, SetsSyncGroupFromSyncLabel) { |
TEST_F(WebRtcVoiceEngineTestFake, CanChangeCombinedBweOption) { |
// Test that changing the combined_audio_video_bwe option results in the |
// expected state changes on an associated Call. |
- std::vector<uint32> ssrcs; |
+ std::vector<uint32_t> ssrcs; |
ssrcs.push_back(223); |
ssrcs.push_back(224); |
EXPECT_TRUE(SetupEngine()); |
cricket::WebRtcVoiceMediaChannel* media_channel = |
static_cast<cricket::WebRtcVoiceMediaChannel*>(channel_); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
EXPECT_TRUE(media_channel->AddRecvStream( |
cricket::StreamParams::CreateLegacy(ssrc))); |
} |
EXPECT_EQ(2, call_.GetAudioReceiveStreams().size()); |
// Combined BWE should be disabled. |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
const auto* s = call_.GetAudioReceiveStream(ssrc); |
EXPECT_NE(nullptr, s); |
EXPECT_EQ(false, s->GetConfig().combined_audio_video_bwe); |
@@ -3194,7 +3192,7 @@ TEST_F(WebRtcVoiceEngineTestFake, CanChangeCombinedBweOption) { |
// Enable combined BWE option - now it should be set up. |
send_parameters_.options.combined_audio_video_bwe.Set(true); |
EXPECT_TRUE(media_channel->SetSendParameters(send_parameters_)); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
const auto* s = call_.GetAudioReceiveStream(ssrc); |
EXPECT_NE(nullptr, s); |
EXPECT_EQ(true, s->GetConfig().combined_audio_video_bwe); |
@@ -3203,7 +3201,7 @@ TEST_F(WebRtcVoiceEngineTestFake, CanChangeCombinedBweOption) { |
// Disable combined BWE option - should be disabled again. |
send_parameters_.options.combined_audio_video_bwe.Set(false); |
EXPECT_TRUE(media_channel->SetSendParameters(send_parameters_)); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
const auto* s = call_.GetAudioReceiveStream(ssrc); |
EXPECT_NE(nullptr, s); |
EXPECT_EQ(false, s->GetConfig().combined_audio_video_bwe); |
@@ -3221,7 +3219,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfigureCombinedBweForNewRecvStreams) { |
send_parameters_.options.combined_audio_video_bwe.Set(true); |
EXPECT_TRUE(media_channel->SetSendParameters(send_parameters_)); |
- static const uint32 kSsrcs[] = {1, 2, 3, 4}; |
+ static const uint32_t kSsrcs[] = {1, 2, 3, 4}; |
for (unsigned int i = 0; i < ARRAY_SIZE(kSsrcs); ++i) { |
EXPECT_TRUE(media_channel->AddRecvStream( |
cricket::StreamParams::CreateLegacy(kSsrcs[i]))); |
@@ -3233,7 +3231,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfigureCombinedBweForNewRecvStreams) { |
TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) { |
// Test that setting the header extensions results in the expected state |
// changes on an associated Call. |
- std::vector<uint32> ssrcs; |
+ std::vector<uint32_t> ssrcs; |
ssrcs.push_back(223); |
ssrcs.push_back(224); |
@@ -3242,14 +3240,14 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) { |
static_cast<cricket::WebRtcVoiceMediaChannel*>(channel_); |
send_parameters_.options.combined_audio_video_bwe.Set(true); |
EXPECT_TRUE(media_channel->SetSendParameters(send_parameters_)); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
EXPECT_TRUE(media_channel->AddRecvStream( |
cricket::StreamParams::CreateLegacy(ssrc))); |
} |
// Combined BWE should be set up, but with no configured extensions. |
EXPECT_EQ(2, call_.GetAudioReceiveStreams().size()); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
const auto* s = call_.GetAudioReceiveStream(ssrc); |
EXPECT_NE(nullptr, s); |
EXPECT_EQ(0, s->GetConfig().rtp.extensions.size()); |
@@ -3261,7 +3259,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) { |
recv_parameters.extensions = e_exts; |
channel_->SetRecvParameters(recv_parameters); |
EXPECT_EQ(2, call_.GetAudioReceiveStreams().size()); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
const auto* s = call_.GetAudioReceiveStream(ssrc); |
EXPECT_NE(nullptr, s); |
const auto& s_exts = s->GetConfig().rtp.extensions; |
@@ -3277,7 +3275,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) { |
// Disable receive extensions. |
channel_->SetRecvParameters(cricket::AudioRecvParameters()); |
- for (uint32 ssrc : ssrcs) { |
+ for (uint32_t ssrc : ssrcs) { |
const auto* s = call_.GetAudioReceiveStream(ssrc); |
EXPECT_NE(nullptr, s); |
EXPECT_EQ(0, s->GetConfig().rtp.extensions.size()); |
@@ -3286,7 +3284,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) { |
TEST_F(WebRtcVoiceEngineTestFake, DeliverAudioPacket_Call) { |
// Test that packets are forwarded to the Call when configured accordingly. |
- const uint32 kAudioSsrc = 1; |
+ const uint32_t kAudioSsrc = 1; |
rtc::Buffer kPcmuPacket(kPcmuFrame, sizeof(kPcmuFrame)); |
static const unsigned char kRtcp[] = { |
0x80, 0xc9, 0x00, 0x01, 0x00, 0x00, 0x00, 0x02, |