diff options
author | David Staessens <dstaessens@google.com> | 2020-09-29 13:21:11 +0900 |
---|---|---|
committer | Chih-Yu Huang <akahuang@google.com> | 2021-01-12 15:56:41 +0900 |
commit | 762e4695f89b20e7f30e83dc3a6db3a73f5a369e (patch) | |
tree | 692c20c210e100dd335041351ce6061a7ef2ac92 /tests | |
parent | bc6c579f89d847f6d440e3f5fd798d6762590fcd (diff) | |
download | v4l2_codec2-762e4695f89b20e7f30e83dc3a6db3a73f5a369e.tar.gz |
v4l2_codec2: Add new SW decoder names to the e2e decoder test.
This CL adds the new names used for the SW decoders on Android R to the
e2e decoder test. The old names are currently still supported as an
alias of the new decoder names, but we should switch to using the new
names to make the tests future-proof.
New SW encoder names:
- OMX.google.h264.decoder -> c2.android.avc.decoder
- OMX.google.vp8.decoder -> c2.android.vp8.decoder
- OMX.google.vp9.decoder -> c2.android.vp9.decoder
Bug: None
Test: tast run DUT arc.VideoDecodeAccel.h264 on hatch
Change-Id: Ie84fcbe4f93943e7bc6be4d2d95163c369c06047
Diffstat (limited to 'tests')
-rw-r--r-- | tests/c2_e2e_test/jni/mediacodec_decoder.cpp | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/tests/c2_e2e_test/jni/mediacodec_decoder.cpp b/tests/c2_e2e_test/jni/mediacodec_decoder.cpp index d033cb0..92d87bc 100644 --- a/tests/c2_e2e_test/jni/mediacodec_decoder.cpp +++ b/tests/c2_e2e_test/jni/mediacodec_decoder.cpp @@ -31,6 +31,8 @@ constexpr int kTimeoutWaitForInputUs = 1000; // 1 millisecond constexpr size_t kTimeoutMaxRetries = 500; // Helper function to get possible C2 hardware decoder names from |type|. +// Note: A single test APK is built for both ARC++ and ARCVM, so both the VDA decoder and the new +// V4L2 decoder names need to be specified here. std::vector<const char*> GetC2VideoDecoderNames(VideoCodecType type) { switch (type) { case VideoCodecType::H264: @@ -45,14 +47,16 @@ std::vector<const char*> GetC2VideoDecoderNames(VideoCodecType type) { } // Helper function to get possible software decoder names from |type|. +// Note: A single test APK is built for both ARC++ and ARCVM, so both the OMX decoder used on +// Android P and the c2.android decoder used on Android R need to be specified here. std::vector<const char*> GetSwVideoDecoderNames(VideoCodecType type) { switch (type) { case VideoCodecType::H264: - return {"OMX.google.h264.decoder"}; + return {"c2.android.avc.decoder", "OMX.google.h264.decoder"}; case VideoCodecType::VP8: - return {"OMX.google.vp8.decoder"}; + return {"c2.android.vp8.decoder", "OMX.google.vp8.decoder"}; case VideoCodecType::VP9: - return {"OMX.google.vp9.decoder"}; + return {"c2.android.vp9.decoder", "OMX.google.vp9.decoder"}; default: // unknown type return {}; } |