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

Commit 96ff5c48 authored by Zhijun He's avatar Zhijun He
Browse files

Camera2Client: avoid two consecutive sets of configure_streams calls 

When mediaRecorder starts without an active preview stream, Camera2Client
starts preview then immediately start recording, which could cause the second
configure_streams to call into HAL before any preview request is sent. This
could cause HAL to run into bad state. This change work around this issue
by making sure the first preview request is submitted to the HAL before
start recording.

Bug: 17915062
Change-Id: I94ae64ee76487603695a469240da601ddcb29a66
parent 801a3ff8

services/camera/libcameraservice/api1/Camera2Client.cpp

+4 −0
Original line number Diff line number Diff line
@@ -995,6 +995,10 @@ status_t Camera2Client::startRecordingL(Parameters &params, bool restart) { 
        case Parameters::STOPPED:

            res = startPreviewL(params, false);

            if (res != OK) return res;

            // Make sure first preview request is submitted to the HAL device to avoid

            // two consecutive set of configure_streams being called into the HAL.

            // TODO: Refactor this to avoid initial preview configuration.

            syncWithDevice();

            break;

        case Parameters::PREVIEW:

            // Ready to go