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

Unified Diff: webrtc/video/send_statistics_proxy_unittest.cc

Issue 1900193004: Remove VCMPacketizationCallback (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fixed bug. Created 4 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/video/send_statistics_proxy_unittest.cc
diff --git a/webrtc/video/send_statistics_proxy_unittest.cc b/webrtc/video/send_statistics_proxy_unittest.cc
index eb77a278877050ff8804c8e97ee5f168665a2875..7c2fc534e0dbd65c309550c038a6c7ee1ebb9524 100644
--- a/webrtc/video/send_statistics_proxy_unittest.cc
+++ b/webrtc/video/send_statistics_proxy_unittest.cc
@@ -147,11 +147,13 @@ TEST_F(SendStatisticsProxyTest, EncodedBitrateAndFramerate) {
int media_bitrate_bps = 500;
int encode_fps = 29;
- statistics_proxy_->OnOutgoingRate(encode_fps, media_bitrate_bps);
+ statistics_proxy_->OnEncoderStatsUpdate(encode_fps, media_bitrate_bps,
+ "a name");
pbos-webrtc 2016/05/02 00:16:06 "encoder name" or something descriptive
perkj_webrtc 2016/05/02 12:02:12 Done.
VideoSendStream::Stats stats = statistics_proxy_->GetStats();
EXPECT_EQ(media_bitrate_bps, stats.media_bitrate_bps);
EXPECT_EQ(encode_fps, stats.encode_frame_rate);
+ EXPECT_EQ("a name", stats.encoder_implementation_name);
}
TEST_F(SendStatisticsProxyTest, Suspended) {

Powered by Google App Engine
This is Rietveld 408576698