diff options
author | James Zern <jzern@google.com> | 2016-04-28 13:09:08 -0700 |
---|---|---|
committer | James Zern <jzern@google.com> | 2016-04-28 16:26:34 -0700 |
commit | ef877d846f7c5a023033ed5201938f31e4ff2e35 (patch) | |
tree | 0a0bd6eb28bdd4e5cf762e262fe40e0bbd51596b | |
parent | 78cde4f593ef7b7014d64219a8986e79c115cdb9 (diff) | |
download | libvpx-ef877d846f7c5a023033ed5201938f31e4ff2e35.tar libvpx-ef877d846f7c5a023033ed5201938f31e4ff2e35.tar.gz libvpx-ef877d846f7c5a023033ed5201938f31e4ff2e35.tar.bz2 libvpx-ef877d846f7c5a023033ed5201938f31e4ff2e35.zip |
vp9_ethread_test: replace FramePktHook w/DecompressedFrameHook
this avoids the decoder test which was only correct for vp9, vp10 was
missed in the earlier change
Change-Id: Ib789c906d440c0e4169052cf64c74d5e4b196caa
-rw-r--r-- | test/vp9_ethread_test.cc | 38 |
1 files changed, 14 insertions, 24 deletions
diff --git a/test/vp9_ethread_test.cc b/test/vp9_ethread_test.cc index 4983d7fe7..f0b8cef57 100644 --- a/test/vp9_ethread_test.cc +++ b/test/vp9_ethread_test.cc @@ -29,16 +29,9 @@ class VPxEncoderThreadTest encoding_mode_(GET_PARAM(1)), set_cpu_used_(GET_PARAM(2)) { init_flags_ = VPX_CODEC_USE_PSNR; - vpx_codec_dec_cfg_t cfg = vpx_codec_dec_cfg_t(); - cfg.w = 1280; - cfg.h = 720; - decoder_ = codec_->CreateDecoder(cfg, 0); - md5_.clear(); } - virtual ~VPxEncoderThreadTest() { - delete decoder_; - } + virtual ~VPxEncoderThreadTest() {} virtual void SetUp() { InitializeConfig(); @@ -81,31 +74,28 @@ class VPxEncoderThreadTest } } - virtual void FramePktHook(const vpx_codec_cx_pkt_t *pkt) { -#if CONFIG_VP9_DECODER - const vpx_codec_err_t res = decoder_->DecodeFrame( - reinterpret_cast<uint8_t*>(pkt->data.frame.buf), pkt->data.frame.sz); + virtual void DecompressedFrameHook(const vpx_image_t &img, + vpx_codec_pts_t /*pts*/) { + ::libvpx_test::MD5 md5_res; + md5_res.Add(&img); + md5_.push_back(md5_res.Get()); + } + + virtual bool HandleDecodeResult(const vpx_codec_err_t res, + const libvpx_test::VideoSource& /*video*/, + libvpx_test::Decoder * /*decoder*/) { if (res != VPX_CODEC_OK) { - abort_ = true; - ASSERT_EQ(VPX_CODEC_OK, res); + EXPECT_EQ(VPX_CODEC_OK, res); + return false; } - const vpx_image_t *img = decoder_->GetDxData().Next(); - if (img) { - ::libvpx_test::MD5 md5_res; - md5_res.Add(img); - md5_.push_back(md5_res.Get()); - } -#else - ASSERT_EQ(NULL, decoder_); -#endif + return true; } bool encoder_initialized_; int tiles_; ::libvpx_test::TestMode encoding_mode_; int set_cpu_used_; - ::libvpx_test::Decoder *decoder_; std::vector<std::string> md5_; }; |