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

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: Addressed TODO (V_MPEG4/ISO/AVC) and adapted unit tests 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
« no previous file with comments | « content/renderer/media/media_recorder_handler.cc ('k') | media/muxers/webm_muxer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..53438f889a9857562bded9c76a1b6a6fe1fcba3d 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,8 +227,8 @@ 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 = 16;
base::RunLoop run_loop;
base::Closure quit_closure = run_loop.QuitClosure();
// writeData() is pinged a number of times as the WebM header is written;
@@ -239,6 +245,7 @@ TEST_P(MediaRecorderHandlerTest, EncodeVideoFrames) {
Mock::VerifyAndClearExpectations(this);
{
+ const size_t kEncodedSizeThreshold = 13;
base::RunLoop run_loop;
base::Closure quit_closure = run_loop.QuitClosure();
// The second time around writeData() is called a number of times to write
« no previous file with comments | « content/renderer/media/media_recorder_handler.cc ('k') | media/muxers/webm_muxer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698