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

Unified Diff: media/muxers/webm_muxer_unittest.cc

Issue 2612403002: Release video frames earlier in MediaRecorder (Closed)
Patch Set: mcasas@ nits. Created 3 years, 11 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 | « media/muxers/webm_muxer_fuzzertest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/muxers/webm_muxer_unittest.cc
diff --git a/media/muxers/webm_muxer_unittest.cc b/media/muxers/webm_muxer_unittest.cc
index 75a698a87224f5100d638c85187eb01abbb1214a..a65a8850be821a7524821c901903c1ea7cd2a574 100644
--- a/media/muxers/webm_muxer_unittest.cc
+++ b/media/muxers/webm_muxer_unittest.cc
@@ -109,7 +109,7 @@ TEST_P(WebmMuxerTest, OnEncodedVideoTwoFrames) {
.Times(AtLeast(1))
.WillRepeatedly(
WithArgs<0>(Invoke(this, &WebmMuxerTest::SaveEncodedDataLen)));
- webm_muxer_.OnEncodedVideo(video_frame,
+ webm_muxer_.OnEncodedVideo(WebmMuxer::VideoParameters(video_frame),
base::WrapUnique(new std::string(encoded_data)),
base::TimeTicks::Now(), false /* keyframe */);
@@ -199,13 +199,13 @@ TEST_P(WebmMuxerTest, VideoIsStoredWhileWaitingForAudio) {
const scoped_refptr<VideoFrame> video_frame =
VideoFrame::CreateBlackFrame(frame_size);
const std::string encoded_video("thisisanencodedvideopacket");
- webm_muxer_.OnEncodedVideo(video_frame,
+ webm_muxer_.OnEncodedVideo(WebmMuxer::VideoParameters(video_frame),
base::WrapUnique(new std::string(encoded_video)),
base::TimeTicks::Now(), true /* keyframe */);
// A few encoded non key frames.
const int kNumNonKeyFrames = 2;
for (int i = 0; i < kNumNonKeyFrames; ++i) {
- webm_muxer_.OnEncodedVideo(video_frame,
+ webm_muxer_.OnEncodedVideo(WebmMuxer::VideoParameters(video_frame),
base::WrapUnique(new std::string(encoded_video)),
base::TimeTicks::Now(), false /* keyframe */);
}
« no previous file with comments | « media/muxers/webm_muxer_fuzzertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698