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

Commit c9755754 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "CameraSource: Remove CHECK if set preview target fails with... 

Merge "CameraSource: Remove CHECK if set preview target fails with CameraSource" am: 0cc62bf0 am: 5ad0be28 am: 2dfd7787 am: 526b6539 am: 489a9b50

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/1906683



Change-Id: I475981e08a84b9b58f2f44f3f069b14944baf73b
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 7dd4886c 489a9b50

media/libstagefright/CameraSource.cpp

+5 −3
Original line number Diff line number Diff line
@@ -564,9 +564,11 @@ status_t CameraSource::initWithCameraAccess( 
    // Set the preview display. Skip this if mSurface is null because

    // applications may already set a surface to the camera.

    if (mSurface != NULL) {

        // This CHECK is good, since we just passed the lock/unlock

        // check earlier by calling mCamera->setParameters().

        CHECK_EQ((status_t)OK, mCamera->setPreviewTarget(mSurface));

        // Surface may be set incorrectly or could already be used even if we just

        // passed the lock/unlock check earlier by calling mCamera->setParameters().

        if ((err = mCamera->setPreviewTarget(mSurface)) != OK) {

            return err;

        }

    }



    // Use buffer queue to receive video buffers from camera