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

Unified Diff: media/cast/receiver/video_decoder_unittest.cc

Issue 899583002: Revert of [Cast] Software encoder support for varying video frame sizes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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/cast/cast.gyp ('k') | media/cast/sender/external_video_encoder.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/receiver/video_decoder_unittest.cc
diff --git a/media/cast/receiver/video_decoder_unittest.cc b/media/cast/receiver/video_decoder_unittest.cc
index 6fcde7af41c8ee24ea2e0289d47bd0d8bba90b88..f777cb38b2660dd737e973080cf6570e320a41a8 100644
--- a/media/cast/receiver/video_decoder_unittest.cc
+++ b/media/cast/receiver/video_decoder_unittest.cc
@@ -3,7 +3,6 @@
// found in the LICENSE file.
#include <cstdlib>
-#include <vector>
#include "base/bind.h"
#include "base/bind_helpers.h"
@@ -23,12 +22,14 @@
namespace {
-const int kStartingWidth = 360;
-const int kStartingHeight = 240;
+const int kWidth = 360;
+const int kHeight = 240;
const int kFrameRate = 10;
VideoSenderConfig GetVideoSenderConfigForTest() {
VideoSenderConfig config = GetDefaultVideoSenderConfig();
+ config.width = kWidth;
+ config.height = kHeight;
config.max_frame_rate = kFrameRate;
return config;
}
@@ -54,7 +55,6 @@
video_decoder_.reset(new VideoDecoder(cast_environment_, GetParam()));
CHECK_EQ(STATUS_VIDEO_INITIALIZED, video_decoder_->InitializationResult());
- next_frame_size_ = gfx::Size(kStartingWidth, kStartingHeight);
next_frame_timestamp_ = base::TimeDelta();
last_frame_id_ = 0;
seen_a_decoded_frame_ = false;
@@ -63,39 +63,34 @@
total_video_frames_decoded_ = 0;
}
- void SetNextFrameSize(const gfx::Size& size) {
- next_frame_size_ = size;
- }
-
// Called from the unit test thread to create another EncodedFrame and push it
// into the decoding pipeline.
void FeedMoreVideo(int num_dropped_frames) {
// Prepare a simulated EncodedFrame to feed into the VideoDecoder.
+ const gfx::Size frame_size(kWidth, kHeight);
const scoped_refptr<VideoFrame> video_frame =
VideoFrame::CreateFrame(VideoFrame::YV12,
- next_frame_size_,
- gfx::Rect(next_frame_size_),
- next_frame_size_,
+ frame_size,
+ gfx::Rect(frame_size),
+ frame_size,
next_frame_timestamp_);
- const base::TimeTicks reference_time =
- base::TimeTicks::UnixEpoch() + next_frame_timestamp_;
next_frame_timestamp_ += base::TimeDelta::FromSeconds(1) / kFrameRate;
PopulateVideoFrame(video_frame.get(), 0);
// Encode |frame| into |encoded_frame->data|.
- scoped_ptr<EncodedFrame> encoded_frame(new EncodedFrame());
+ scoped_ptr<EncodedFrame> encoded_frame(
+ new EncodedFrame());
// Test only supports VP8, currently.
CHECK_EQ(CODEC_VIDEO_VP8, GetParam());
- vp8_encoder_.Encode(video_frame, reference_time, encoded_frame.get());
+ vp8_encoder_.Encode(video_frame, base::TimeTicks(), encoded_frame.get());
// Rewrite frame IDs for testing purposes.
encoded_frame->frame_id = last_frame_id_ + 1 + num_dropped_frames;
- if (encoded_frame->dependency == EncodedFrame::KEY)
+ if (last_frame_id_ == 0)
encoded_frame->referenced_frame_id = encoded_frame->frame_id;
else
encoded_frame->referenced_frame_id = encoded_frame->frame_id - 1;
last_frame_id_ = encoded_frame->frame_id;
- ASSERT_EQ(reference_time, encoded_frame->reference_time);
{
base::AutoLock auto_lock(lock_);
@@ -153,7 +148,6 @@
const scoped_refptr<StandaloneCastEnvironment> cast_environment_;
scoped_ptr<VideoDecoder> video_decoder_;
- gfx::Size next_frame_size_;
base::TimeDelta next_frame_timestamp_;
uint32 last_frame_id_;
bool seen_a_decoded_frame_;
@@ -192,38 +186,7 @@
WaitForAllVideoToBeDecoded();
}
-TEST_P(VideoDecoderTest, DecodesFramesOfVaryingSizes) {
- std::vector<gfx::Size> frame_sizes;
- frame_sizes.push_back(gfx::Size(1280, 720));
- frame_sizes.push_back(gfx::Size(640, 360)); // Shrink both dimensions.
- frame_sizes.push_back(gfx::Size(300, 200)); // Shrink both dimensions again.
- frame_sizes.push_back(gfx::Size(200, 300)); // Same area.
- frame_sizes.push_back(gfx::Size(600, 400)); // Grow both dimensions.
- frame_sizes.push_back(gfx::Size(638, 400)); // Shrink only one dimension.
- frame_sizes.push_back(gfx::Size(638, 398)); // Shrink the other dimension.
- frame_sizes.push_back(gfx::Size(320, 180)); // Shrink both dimensions again.
- frame_sizes.push_back(gfx::Size(322, 180)); // Grow only one dimension.
- frame_sizes.push_back(gfx::Size(322, 182)); // Grow the other dimension.
- frame_sizes.push_back(gfx::Size(1920, 1080)); // Grow both dimensions again.
-
- // Encode one frame at each size.
- for (const auto& frame_size : frame_sizes) {
- SetNextFrameSize(frame_size);
- FeedMoreVideo(0);
- }
-
- // Encode 10 frames at each size.
- for (const auto& frame_size : frame_sizes) {
- SetNextFrameSize(frame_size);
- const int kNumFrames = 10;
- for (int i = 0; i < kNumFrames; ++i)
- FeedMoreVideo(0);
- }
-
- WaitForAllVideoToBeDecoded();
-}
-
-INSTANTIATE_TEST_CASE_P(,
+INSTANTIATE_TEST_CASE_P(VideoDecoderTestScenarios,
VideoDecoderTest,
::testing::Values(CODEC_VIDEO_VP8));
« no previous file with comments | « media/cast/cast.gyp ('k') | media/cast/sender/external_video_encoder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698