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

Commit 11839395 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

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



Change-Id: I3672ff9dfc3b611f38f37aedfbaa24638686c7c3
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents b47346c0 c1065f69

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());