Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0938e261 authored by kongchenrui's avatar kongchenrui Committed by Automerger Merge Worker
Browse files

Fix the problem that the data field of Entry in the... 

Fix the problem that the data field of Entry in the verifyLogicalOrUltraHighResCameraMetadata function is empty am: 7549241c am: 65ebae18 am: c1065f69 am: 11839395 am: fccc9c8f am: c4645fe0

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2228941



Change-Id: Icd8852129e41316ad9beb0f1942ff2027556ea3b
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0f1032f1 c4645fe0

camera/provider/2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp

+8 −3
Original line number Diff line number Diff line
@@ -7883,6 +7883,7 @@ void CameraHidlTest::verifyLogicalOrUltraHighResCameraMetadata( 
            }

        }



        camera_metadata_t* staticMetadata;

        camera_metadata_ro_entry physicalMultiResStreamConfigs;

        camera_metadata_ro_entry physicalStreamConfigs;

        camera_metadata_ro_entry physicalMaxResolutionStreamConfigs;
@@ -7901,8 +7902,9 @@ void CameraHidlTest::verifyLogicalOrUltraHighResCameraMetadata( 


            ret = subDevice->getCameraCharacteristics([&](auto status, const auto& chars) {

                ASSERT_EQ(Status::OK, status);

                const camera_metadata_t* staticMetadata =

                        reinterpret_cast<const camera_metadata_t*>(chars.data());

                staticMetadata = clone_camera_metadata(

                        reinterpret_cast<const camera_metadata_t*>(chars.data()));

                ASSERT_NE(nullptr, staticMetadata);

                rc = getSystemCameraKind(staticMetadata, &physSystemCameraKind);

                ASSERT_EQ(rc, Status::OK);

                // Make sure that the system camera kind of a non-hidden
@@ -7936,7 +7938,9 @@ void CameraHidlTest::verifyLogicalOrUltraHighResCameraMetadata( 
                        verifyCameraCharacteristics(status, chars);

                        verifyMonochromeCharacteristics(chars, deviceVersion);



                        auto staticMetadata = (const camera_metadata_t*)chars.data();

                        staticMetadata = clone_camera_metadata(

                                reinterpret_cast<const camera_metadata_t*>(chars.data()));

                        ASSERT_NE(nullptr, staticMetadata);

                        retcode = find_camera_metadata_ro_entry(

                                staticMetadata, ANDROID_CONTROL_ZOOM_RATIO_RANGE, &entry);

                        bool subCameraHasZoomRatioRange = (0 == retcode && entry.count == 2);
@@ -8064,6 +8068,7 @@ void CameraHidlTest::verifyLogicalOrUltraHighResCameraMetadata( 
                }

            }

        }

        free_camera_metadata(staticMetadata);

    }



    // If a multi-resolution stream is supported, there must be at least one

camera/provider/aidl/vts/camera_aidl_test.cpp

+3 −4
Original line number Diff line number Diff line
@@ -1181,6 +1181,7 @@ void CameraAidlTest::verifyLogicalOrUltraHighResCameraMetadata( 
        camera_metadata_ro_entry physicalMultiResStreamConfigs;

        camera_metadata_ro_entry physicalStreamConfigs;

        camera_metadata_ro_entry physicalMaxResolutionStreamConfigs;

        CameraMetadata physChars;

        bool isUltraHighRes = false;

        std::unordered_set<int32_t> subCameraPrivacyTestPatterns;

        if (isPublicId) {
@@ -1189,12 +1190,11 @@ void CameraAidlTest::verifyLogicalOrUltraHighResCameraMetadata( 
            ASSERT_TRUE(ret.isOk());

            ASSERT_NE(subDevice, nullptr);



            CameraMetadata subDeviceChars;

            ret = subDevice->getCameraCharacteristics(&subDeviceChars);

            ret = subDevice->getCameraCharacteristics(&physChars);

            ASSERT_TRUE(ret.isOk());



            const camera_metadata_t* staticMetadata =

                    reinterpret_cast<const camera_metadata_t*>(subDeviceChars.metadata.data());

                    reinterpret_cast<const camera_metadata_t*>(physChars.metadata.data());

            retStatus = getSystemCameraKind(staticMetadata, &physSystemCameraKind);

            ASSERT_EQ(retStatus, Status::OK);
@@ -1215,7 +1215,6 @@ void CameraAidlTest::verifyLogicalOrUltraHighResCameraMetadata( 
            getPrivacyTestPatternModes(staticMetadata, &subCameraPrivacyTestPatterns);

        } else {

            // Check camera characteristics for hidden camera id

            CameraMetadata physChars;

            ndk::ScopedAStatus ret =

                    device->getPhysicalCameraCharacteristics(physicalId, &physChars);

            ASSERT_TRUE(ret.isOk());