| commit | 9ac48cecefa4a3be5ab6abe336f7731098a2507e | [log] [tgz] |
|---|---|---|
| author | TreeHugger Robot <treehugger-gerrit@google.com> | Tue Aug 17 00:57:09 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Tue Aug 17 00:57:09 2021 +0000 |
| tree | 24769e543765749a707dd5770bb275ddbac3e449 | |
| parent | d68a2ef3e695bce2d4918365909653fe1cd7eaef [diff] | |
| parent | aae88ba179080bffe45dfe2845e93dfc64dab0d5 [diff] |
Merge "omx: validate only standard OMX roles" into sc-dev am: af592fb124 am: aae88ba179 Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15528285 Change-Id: Ia99885431518927bd7d5dbd373a4953b2768a3df
diff --git a/media/omx/1.0/vts/functional/store/VtsHalMediaOmxV1_0TargetStoreTest.cpp b/media/omx/1.0/vts/functional/store/VtsHalMediaOmxV1_0TargetStoreTest.cpp index e73196c..8699de3 100644 --- a/media/omx/1.0/vts/functional/store/VtsHalMediaOmxV1_0TargetStoreTest.cpp +++ b/media/omx/1.0/vts/functional/store/VtsHalMediaOmxV1_0TargetStoreTest.cpp
@@ -264,11 +264,13 @@ // Make sure role name follows expected format based on type and // isEncoder - const std::string role_name( - ::android::GetComponentRole(role.isEncoder, role.type.c_str())); - EXPECT_EQ(role_name, role.role) << "Role \"" << role.role << "\" does not match " - << (role.isEncoder ? "an encoder " : "a decoder ") - << "for mime type \"" << role.type << "."; + const char* role_name = ::android::GetComponentRole(role.isEncoder, role.type.c_str()); + if (role_name != nullptr) { + EXPECT_EQ(std::string(role_name), role.role) + << "Role \"" << role.role << "\" does not match " + << (role.isEncoder ? "an encoder " : "a decoder ") << "for media type \"" + << role.type << "."; + } // Check the nodes for this role std::set<const std::string> nodeKeys;