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

Unified Diff: ppapi/tests/test_video_encoder.cc

Issue 1132833002: pepper: add software vp9 encoder support to PPB_VideoEncoder (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tune VP9 parameters Created 5 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
Index: ppapi/tests/test_video_encoder.cc
diff --git a/ppapi/tests/test_video_encoder.cc b/ppapi/tests/test_video_encoder.cc
index b4907cc9773ec535b5a974e3f0ffd8142e6e908f..0e10d998fa4c1dc8b5c196135b0158f21bc12377 100644
--- a/ppapi/tests/test_video_encoder.cc
+++ b/ppapi/tests/test_video_encoder.cc
@@ -39,18 +39,21 @@ std::string TestVideoEncoder::TestCreate() {
callback.output();
ASSERT_GE(video_profiles.size(), 1U);
- bool found_vp8 = false;
+ bool found_vp8 = false, found_vp9 = false;
for (uint32_t i = 0; i < video_profiles.size(); ++i) {
const PP_VideoProfileDescription& description = video_profiles[i];
- if (description.profile == PP_VIDEOPROFILE_VP8_ANY &&
- description.hardware_accelerated == PP_FALSE) {
+ if (description.hardware_accelerated == PP_FALSE) {
ASSERT_GE(description.max_framerate_numerator /
description.max_framerate_denominator,
30U);
- found_vp8 = true;
+ if (description.profile == PP_VIDEOPROFILE_VP8_ANY)
+ found_vp8 = true;
+ else if (description.profile == PP_VIDEOPROFILE_VP9_ANY)
+ found_vp9 = true;
}
}
ASSERT_TRUE(found_vp8);
+ ASSERT_TRUE(found_vp9);
}
// Test that initializing the encoder with incorrect size fails.
{

Powered by Google App Engine
This is Rietveld 408576698