| commit | aae88ba179080bffe45dfe2845e93dfc64dab0d5 | [log] [tgz] |
|---|---|---|
| author | TreeHugger Robot <treehugger-gerrit@google.com> | Tue Aug 17 00:44:04 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Tue Aug 17 00:44:04 2021 +0000 |
| tree | a10b14e9e80f546393ffab5b72e0c7f2cfb97227 | |
| parent | 0cf1bf28c9f81ef2451c36bac325e648ff3edca0 [diff] | |
| parent | af592fb12458d90109f48cd38096f48842e59aa7 [diff] |
Merge "omx: validate only standard OMX roles" into sc-dev am: af592fb124 Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15528285 Change-Id: I5e2f8f8594b750f120ab4e971f6e3efaa14583a0
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;