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

Side by Side Diff: webrtc/media/engine/webrtcvideoengine2_unittest.cc

Issue 2430603003: Implement qpSum stat for video send ssrc stats. (Closed)
Patch Set: Change qp_sum to rtc::Optional<uint64_t>. Created 4 years, 2 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) 2004 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2004 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 2887 matching lines...) Expand 10 before | Expand all | Expand 10 after
2898 FakeVideoSendStream* stream = AddSendStream(); 2898 FakeVideoSendStream* stream = AddSendStream();
2899 webrtc::VideoSendStream::Stats stats; 2899 webrtc::VideoSendStream::Stats stats;
2900 stats.frames_encoded = 13; 2900 stats.frames_encoded = 13;
2901 stream->SetStats(stats); 2901 stream->SetStats(stats);
2902 2902
2903 cricket::VideoMediaInfo info; 2903 cricket::VideoMediaInfo info;
2904 ASSERT_TRUE(channel_->GetStats(&info)); 2904 ASSERT_TRUE(channel_->GetStats(&info));
2905 EXPECT_EQ(stats.frames_encoded, info.senders[0].frames_encoded); 2905 EXPECT_EQ(stats.frames_encoded, info.senders[0].frames_encoded);
2906 } 2906 }
2907 2907
2908 TEST_F(WebRtcVideoChannel2Test, GetStatsReportsQpSum) {
2909 FakeVideoSendStream* stream = AddSendStream();
2910 webrtc::VideoSendStream::Stats stats;
2911 stats.qp_sum = rtc::Optional<uint64_t>(13);
2912 stream->SetStats(stats);
2913
2914 cricket::VideoMediaInfo info;
2915 ASSERT_TRUE(channel_->GetStats(&info));
2916 EXPECT_EQ(stats.qp_sum, info.senders[0].qp_sum);
2917 }
2918
2908 TEST_F(WebRtcVideoChannel2Test, GetStatsReportsUpperResolution) { 2919 TEST_F(WebRtcVideoChannel2Test, GetStatsReportsUpperResolution) {
2909 FakeVideoSendStream* stream = AddSendStream(); 2920 FakeVideoSendStream* stream = AddSendStream();
2910 webrtc::VideoSendStream::Stats stats; 2921 webrtc::VideoSendStream::Stats stats;
2911 stats.substreams[17].width = 123; 2922 stats.substreams[17].width = 123;
2912 stats.substreams[17].height = 40; 2923 stats.substreams[17].height = 40;
2913 stats.substreams[42].width = 80; 2924 stats.substreams[42].width = 80;
2914 stats.substreams[42].height = 31; 2925 stats.substreams[42].height = 31;
2915 stats.substreams[11].width = 20; 2926 stats.substreams[11].width = 20;
2916 stats.substreams[11].height = 90; 2927 stats.substreams[11].height = 90;
2917 stream->SetStats(stats); 2928 stream->SetStats(stats);
(...skipping 951 matching lines...) Expand 10 before | Expand all | Expand 10 after
3869 } 3880 }
3870 3881
3871 // Test that we normalize send codec format size in simulcast. 3882 // Test that we normalize send codec format size in simulcast.
3872 TEST_F(WebRtcVideoChannel2SimulcastTest, SetSendCodecsWithOddSizeInSimulcast) { 3883 TEST_F(WebRtcVideoChannel2SimulcastTest, SetSendCodecsWithOddSizeInSimulcast) {
3873 cricket::VideoCodec codec(kVp8Codec270p); 3884 cricket::VideoCodec codec(kVp8Codec270p);
3874 codec.width += 1; 3885 codec.width += 1;
3875 codec.height += 1; 3886 codec.height += 1;
3876 VerifySimulcastSettings(codec, 2, 2); 3887 VerifySimulcastSettings(codec, 2, 2);
3877 } 3888 }
3878 } // namespace cricket 3889 } // namespace cricket
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698