pepper: add software vp9 encoder support to PPB_VideoEncoder

ppapi/tests/test_video_encoder.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/tests/test_video_encoder.h"
 
 #include "ppapi/c/pp_codecs.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/cpp/video_encoder.h"
 #include "ppapi/tests/testing_instance.h"
@@ skipping 21 matching lines @@
         callback(instance_->pp_instance(), false);
     callback.WaitForResult(
         video_encoder.GetSupportedProfiles(callback.GetCallback()));
 
     ASSERT_GE(callback.result(), 1U);
 
     const std::vector<PP_VideoProfileDescription> video_profiles =
         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.
   {
     pp::VideoEncoder video_encoder(instance_);
     ASSERT_FALSE(video_encoder.is_null());
     pp::Size video_size(0, 0);
 
     TestCompletionCallback callback(instance_->pp_instance(), false);
     callback.WaitForResult(
         video_encoder.Initialize(PP_VIDEOFRAME_FORMAT_I420,
@@ skipping 29 matching lines @@
 
     TestCompletionCallbackWithOutput<pp::VideoFrame> get_video_frame(
         instance_->pp_instance(), false);
     get_video_frame.WaitForResult(
         video_encoder.GetVideoFrame(get_video_frame.GetCallback()));
     ASSERT_EQ(PP_OK, get_video_frame.result());
 
     pp::Size video_frame_size;
     ASSERT_TRUE(get_video_frame.output().GetSize(&video_frame_size));
     ASSERT_EQ(coded_size.GetArea(), video_frame_size.GetArea());
   }

[bbudge, 2015/08/05 02:07:45]

Can you add a test for VP9? You could macro-ize the above code with a codec
parameter to reduce duplication.

[llandwerlin-old, 2015/08/05 14:55:17]

Done, also split the tests into different functions.

 
   PASS();
 }