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

Side by Side Diff: webrtc/audio/audio_send_stream_unittest.cc

Issue 2979833002: Add a histogram metric tracking for how long audio RTP packets are sent (Closed)
Patch Set: Created 3 years, 5 months 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
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 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 .Times(1); 226 .Times(1);
227 } else { 227 } else {
228 EXPECT_CALL(*channel_proxy_, RegisterSenderCongestionControlObjects( 228 EXPECT_CALL(*channel_proxy_, RegisterSenderCongestionControlObjects(
229 &fake_transport_, Eq(nullptr))) 229 &fake_transport_, Eq(nullptr)))
230 .Times(1); 230 .Times(1);
231 } 231 }
232 EXPECT_CALL(*channel_proxy_, SetBitrate(_, _)) 232 EXPECT_CALL(*channel_proxy_, SetBitrate(_, _))
233 .Times(1); 233 .Times(1);
234 EXPECT_CALL(*channel_proxy_, ResetSenderCongestionControlObjects()) 234 EXPECT_CALL(*channel_proxy_, ResetSenderCongestionControlObjects())
235 .Times(1); 235 .Times(1);
236 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1); 236 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(_)).Times(1);
ossu 2017/07/14 11:13:15 This is odd. Why would the test want to set the ex
saza WebRTC 2017/07/17 14:27:29 Done.
237 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1); 237 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1);
238 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1); 238 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1);
239 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull())) 239 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull()))
240 .Times(1); // Destructor resets the event log 240 .Times(1); // Destructor resets the event log
241 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1); 241 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1);
242 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(testing::IsNull())) 242 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(testing::IsNull()))
243 .Times(1); // Destructor resets the rtt stats. 243 .Times(1); // Destructor resets the rtt stats.
244 } 244 }
245 245
246 void SetupMockForSetupSendCodec(bool expect_set_encoder_call) { 246 void SetupMockForSetupSendCodec(bool expect_set_encoder_call) {
(...skipping 316 matching lines...) Expand 10 before | Expand all | Expand 10 after
563 EXPECT_CALL(*helper.channel_proxy(), RegisterSenderCongestionControlObjects( 563 EXPECT_CALL(*helper.channel_proxy(), RegisterSenderCongestionControlObjects(
564 helper.transport(), Ne(nullptr))) 564 helper.transport(), Ne(nullptr)))
565 .Times(1); 565 .Times(1);
566 } 566 }
567 send_stream.Reconfigure(new_config); 567 send_stream.Reconfigure(new_config);
568 } 568 }
569 569
570 570
571 } // namespace test 571 } // namespace test
572 } // namespace webrtc 572 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698