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

Unified Diff: source/libvpx/test/frame_size_tests.cc

Issue 478033002: libvpx: Pull from upstream (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/deps/third_party/libvpx/
Patch Set: Created 6 years, 4 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 | « source/libvpx/test/fdct8x8_test.cc ('k') | source/libvpx/test/sad_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: source/libvpx/test/frame_size_tests.cc
===================================================================
--- source/libvpx/test/frame_size_tests.cc (revision 290053)
+++ source/libvpx/test/frame_size_tests.cc (working copy)
@@ -7,13 +7,9 @@
* in the file PATENTS. All contributing project authors may
* be found in the AUTHORS file in the root of the source tree.
*/
-#include <climits>
-#include <vector>
#include "third_party/googletest/src/include/gtest/gtest.h"
#include "test/codec_factory.h"
-#include "test/encode_test_driver.h"
-#include "test/i420_video_source.h"
-#include "test/util.h"
+#include "test/video_source.h"
namespace {
@@ -33,10 +29,7 @@
virtual bool HandleDecodeResult(const vpx_codec_err_t res_dec,
const libvpx_test::VideoSource &video,
libvpx_test::Decoder *decoder) {
- EXPECT_EQ(expected_res_, res_dec)
- << "Expected " << expected_res_
- << "but got " << res_dec;
-
+ EXPECT_EQ(expected_res_, res_dec) << decoder->DecodeError();
return !::testing::Test::HasFailure();
}
@@ -62,16 +55,6 @@
video.set_limit(2);
expected_res_ = VPX_CODEC_CORRUPT_FRAME;
ASSERT_NO_FATAL_FAILURE(RunLoop(&video));
-#else
- // If we are on a 32 bit platform we can't possibly allocate enough memory
- // for the largest video frame size (64kx64k). This test checks that we
- // properly return a memory error.
- if (sizeof(size_t) == 4) {
- video.SetSize(65535, 65535);
- video.set_limit(2);
- expected_res_ = VPX_CODEC_MEM_ERROR;
- ASSERT_NO_FATAL_FAILURE(RunLoop(&video));
- }
#endif
}
@@ -89,8 +72,7 @@
// one for each lag in frames (for 2 pass), and then one for each possible
// reference buffer (8) - we can end up with up to 30 buffers of roughly this
// size or almost 1 gig of memory.
- // TODO(jzern): restore this to at least 4096x4096 after issue #828 is fixed.
- video.SetSize(4096, 2160);
+ video.SetSize(4096, 4096);
video.set_limit(2);
expected_res_ = VPX_CODEC_OK;
ASSERT_NO_FATAL_FAILURE(RunLoop(&video));
« no previous file with comments | « source/libvpx/test/fdct8x8_test.cc ('k') | source/libvpx/test/sad_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698