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

Unified Diff: webrtc/media/base/codec_unittest.cc

Issue 2437453003: Remove VideoCodec resolution validation. (Closed)
Patch Set: Remove broken tests. 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/media/base/codec.cc ('k') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/base/codec_unittest.cc
diff --git a/webrtc/media/base/codec_unittest.cc b/webrtc/media/base/codec_unittest.cc
index ce159ef94a61e5e53143f3f41f555be90cb3b68c..11717b743ad3d90826615c5f2b50e94d3beea44a 100644
--- a/webrtc/media/base/codec_unittest.cc
+++ b/webrtc/media/base/codec_unittest.cc
@@ -348,16 +348,6 @@ TEST(CodecTest, TestValidateCodecFormat) {
too_high_payload_type.id = 128;
EXPECT_FALSE(too_high_payload_type.ValidateCodecFormat());
- // Reject zero-width codecs.
- VideoCodec zero_width = codec;
- zero_width.width = 0;
- EXPECT_FALSE(zero_width.ValidateCodecFormat());
-
- // Reject zero-height codecs.
- VideoCodec zero_height = codec;
- zero_height.height = 0;
- EXPECT_FALSE(zero_height.ValidateCodecFormat());
-
// Accept non-video codecs with zero dimensions.
VideoCodec zero_width_rtx_codec = VideoCodec::CreateRtxCodec(96, 120);
zero_width_rtx_codec.width = 0;
« no previous file with comments | « webrtc/media/base/codec.cc ('k') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698