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

Unified Diff: content/renderer/media/media_recorder_handler_unittest.cc

Issue 1920483002: MediaRecorderHandler and WebmMuxer: add support for H264. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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: content/renderer/media/media_recorder_handler_unittest.cc
diff --git a/content/renderer/media/media_recorder_handler_unittest.cc b/content/renderer/media/media_recorder_handler_unittest.cc
index 5bc99aedc203172726f6290026851a7834b68ed2..e4ba9a4dcf194ed59b12af14cff8876b60b26992 100644
--- a/content/renderer/media/media_recorder_handler_unittest.cc
+++ b/content/renderer/media/media_recorder_handler_unittest.cc
@@ -59,6 +59,9 @@ struct MediaRecorderTestParams {
static const MediaRecorderTestParams kMediaRecorderTestParams[] = {
{true, false, "video/webm", "vp8"},
{true, false, "video/webm", "vp9"},
+#if BUILDFLAG(RTC_USE_H264)
+ {true, false, "video/webm", "h264"},
+#endif
{false, true, "video/webm", "vp8"}};
class MediaRecorderHandlerTest : public TestWithParam<MediaRecorderTestParams>,
@@ -156,6 +159,9 @@ TEST_F(MediaRecorderHandlerTest, CanSupportMimeType) {
const WebString example_good_codecs_3(base::UTF8ToUTF16("VP9,opus"));
EXPECT_TRUE(media_recorder_handler_->canSupportMimeType(
mime_type_video, example_good_codecs_3));
+ const WebString example_good_codecs_4(base::UTF8ToUTF16("H264"));
+ EXPECT_TRUE(media_recorder_handler_->canSupportMimeType(
+ mime_type_video, example_good_codecs_4));
const WebString example_unsupported_codecs_1(base::UTF8ToUTF16("daala"));
EXPECT_FALSE(media_recorder_handler_->canSupportMimeType(
@@ -164,12 +170,12 @@ TEST_F(MediaRecorderHandlerTest, CanSupportMimeType) {
const WebString mime_type_audio(base::UTF8ToUTF16("audio/webm"));
EXPECT_TRUE(media_recorder_handler_->canSupportMimeType(
mime_type_audio, WebString()));
- const WebString example_good_codecs_4(base::UTF8ToUTF16("opus"));
- EXPECT_TRUE(media_recorder_handler_->canSupportMimeType(
- mime_type_audio, example_good_codecs_4));
- const WebString example_good_codecs_5(base::UTF8ToUTF16("OpUs"));
+ const WebString example_good_codecs_5(base::UTF8ToUTF16("opus"));
EXPECT_TRUE(media_recorder_handler_->canSupportMimeType(
mime_type_audio, example_good_codecs_5));
+ const WebString example_good_codecs_6(base::UTF8ToUTF16("OpUs"));
+ EXPECT_TRUE(media_recorder_handler_->canSupportMimeType(
+ mime_type_audio, example_good_codecs_6));
const WebString example_unsupported_codecs_2(base::UTF8ToUTF16("vorbis"));
EXPECT_FALSE(media_recorder_handler_->canSupportMimeType(
@@ -221,7 +227,7 @@ TEST_P(MediaRecorderHandlerTest, EncodeVideoFrames) {
const scoped_refptr<media::VideoFrame> video_frame =
media::VideoFrame::CreateBlackFrame(gfx::Size(160, 80));
- const size_t kEncodedSizeThreshold = 16;
+ const size_t kEncodedSizeThreshold = 13;
{
base::RunLoop run_loop;
base::Closure quit_closure = run_loop.QuitClosure();

Powered by Google App Engine
This is Rietveld 408576698