tequilaOS/platform_hardware_interfaces
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

VtsHalMediaOmx: fix OmxCodecAllowedTest to match requirement

Browse source 
Bug: 249043823
Bug: 251553217
Test: atest VtsHalMediaOmxV1_0TargetStoreTest
Change-Id: I8015d559d8e7d88a32d1970102167fe4d7566c51
(cherry picked from commit db13ce5daf)
Merged-In: I8015d559d8e7d88a32d1970102167fe4d7566c51
This commit is contained in:
Wonsik Kim 2022-10-06 22:00:06 -07:00 committed by Cherrypicker Worker
parent f38d908d43
commit 86e97b5f11
1 changed files with 10 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
media/omx/1.0/vts/functional/store/VtsHalMediaOmxV1_0TargetStoreTest.cpp Executable file → Normal file
View file

@ -374,21 +374,16 @@ TEST_P(StoreHidlTest, ListRoles) {
} }
} }
static int getFirstApiLevel() {
int boardApiLevel = android::base::GetIntProperty("ro.board.first_api_level", 0);
if (boardApiLevel != 0) {
return boardApiLevel;
}
return android::base::GetIntProperty("ro.product.first_api_level", __ANDROID_API_T__);
}
static bool isTV() { static bool isTV() {
return testing::deviceSupportsFeature("android.software.leanback"); return testing::deviceSupportsFeature("android.software.leanback");
} }
// list components and roles. // list components and roles.
TEST_P(StoreHidlTest, OmxCodecAllowedTest) { TEST_P(StoreHidlTest, OmxCodecAllowedTest) {
static int sBoardFirstApiLevel = android::base::GetIntProperty("ro.board.first_api_level", 0);
if (sBoardFirstApiLevel == 0) {
GTEST_SKIP() << "board first API level not detected";
}
hidl_vec<IOmx::ComponentInfo> componentInfos = getComponentInfoList(omx); hidl_vec<IOmx::ComponentInfo> componentInfos = getComponentInfoList(omx);
for (IOmx::ComponentInfo info : componentInfos) { for (IOmx::ComponentInfo info : componentInfos) {
for (std::string role : info.mRoles) { for (std::string role : info.mRoles) {
@ -396,27 +391,27 @@ TEST_P(StoreHidlTest, OmxCodecAllowedTest) {
role.find("video_encoder") != std::string::npos) { role.find("video_encoder") != std::string::npos) {
// Codec2 is not mandatory on Android TV devices that launched with Android S // Codec2 is not mandatory on Android TV devices that launched with Android S
if (isTV()) { if (isTV()) {
ASSERT_LT(getFirstApiLevel(), __ANDROID_API_T__) ASSERT_LT(sBoardFirstApiLevel, __ANDROID_API_T__)
<< " Component: " << info.mName.c_str() << " Role: " << role.c_str() << " Component: " << info.mName.c_str() << " Role: " << role.c_str()
<< " not allowed for devices launching with Android T and above"; << " not allowed for devices launching with Android T and above";
} else { } else {
std::string codecName = info.mName; std::string codecName = info.mName;
bool isAndroidCodec = (codecName.rfind("OMX.google", 0) != std::string::npos); bool isAndroidCodec = (codecName.rfind("OMX.google", 0) != std::string::npos);
if (isAndroidCodec && (getFirstApiLevel() <= __ANDROID_API_S__)) { if (isAndroidCodec && (sBoardFirstApiLevel <= __ANDROID_API_S__)) {
// refer b/230582620 // refer b/230582620
// S AOSP build did not remove the OMX.google video codecs // S AOSP build did not remove the OMX.google video codecs
// so it is infeasible to require no OMX.google.* video codecs // so it is infeasible to require no OMX.google.* video codecs
// on S launching devices // on S launching devices
} else { } else {
ASSERT_LT(getFirstApiLevel(), __ANDROID_API_S__) ASSERT_LT(sBoardFirstApiLevel, __ANDROID_API_S__)
<< " Component: " << info.mName.c_str() << " Role: " << role.c_str() << " Component: " << info.mName.c_str() << " Role: " << role.c_str()
<< " not allowed for devices launching with Android S and above"; << " not allowed for devices launching with Android S and above";
} }
} }
} }
if (role.find("audio_decoder") != std::string::npos || if (role.find("audio_decoder") != std::string::npos ||
role.find("audio_encoder") != std::string::npos) { role.find("audio_encoder") != std::string::npos) {
ASSERT_LT(getFirstApiLevel(), __ANDROID_API_T__) ASSERT_LT(sBoardFirstApiLevel, __ANDROID_API_T__)
<< " Component: " << info.mName.c_str() << " Role: " << role.c_str() << " Component: " << info.mName.c_str() << " Role: " << role.c_str()
<< " not allowed for devices launching with Android T and above"; << " not allowed for devices launching with Android T and above";
} }