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

Commit 4470222d authored by android-build-team Robot's avatar android-build-team Robot
Browse files

Snap for 5686022 from 95f6e43f to qt-c2f2-release 

Change-Id: I57fb85f821bffa870775545cf7d5174f3221475b
parents 0b4db5ec 95f6e43f

camera/ndk/impl/ACameraDevice.cpp

+16 −6
Original line number Diff line number Diff line
@@ -28,6 +28,10 @@ 


#include "ACameraCaptureSession.inc"



ACameraDevice::~ACameraDevice() {

    mDevice->stopLooper();

}



namespace android {

namespace acam {
@@ -116,15 +120,11 @@ CameraDevice::~CameraDevice() { 
        if (!isClosed()) {

            disconnectLocked(session);

        }

        LOG_ALWAYS_FATAL_IF(mCbLooper != nullptr,

                "CameraDevice looper should've been stopped before ~CameraDevice");

        mCurrentSession = nullptr;

        if (mCbLooper != nullptr) {

            mCbLooper->unregisterHandler(mHandler->id());

            mCbLooper->stop();

    }

}

    mCbLooper.clear();

    mHandler.clear();

}



void

CameraDevice::postSessionMsgAndCleanup(sp<AMessage>& msg) {
@@ -892,6 +892,16 @@ CameraDevice::onCaptureErrorLocked( 
    return;

}



void CameraDevice::stopLooper() {

    Mutex::Autolock _l(mDeviceLock);

    if (mCbLooper != nullptr) {

      mCbLooper->unregisterHandler(mHandler->id());

      mCbLooper->stop();

    }

    mCbLooper.clear();

    mHandler.clear();

}



CameraDevice::CallbackHandler::CallbackHandler(const char* id) : mId(id) {

}

camera/ndk/impl/ACameraDevice.h

+4 −1
Original line number Diff line number Diff line
@@ -109,6 +109,9 @@ class CameraDevice final : public RefBase { 


    inline ACameraDevice* getWrapper() const { return mWrapper; };



    // Stop the looper thread and unregister the handler

    void stopLooper();



  private:

    friend ACameraCaptureSession;

    camera_status_t checkCameraClosedOrErrorLocked() const;
@@ -354,7 +357,7 @@ struct ACameraDevice { 
                  sp<ACameraMetadata> chars) :

            mDevice(new android::acam::CameraDevice(id, cb, chars, this)) {}



    ~ACameraDevice() {};

    ~ACameraDevice();



    /*******************

     * NDK public APIs *

camera/ndk/ndk_vendor/impl/ACameraDevice.cpp

+13 −13
Original line number Diff line number Diff line
@@ -45,13 +45,7 @@ 
using namespace android;



ACameraDevice::~ACameraDevice() {

    Mutex::Autolock _l(mDevice->mDeviceLock);

    if (mDevice->mCbLooper != nullptr) {

      mDevice->mCbLooper->unregisterHandler(mDevice->mHandler->id());

      mDevice->mCbLooper->stop();

    }

    mDevice->mCbLooper.clear();

    mDevice->mHandler.clear();

    mDevice->stopLooper();

}



namespace android {
@@ -140,14 +134,10 @@ CameraDevice::~CameraDevice() { 
            disconnectLocked(session);

        }

        mCurrentSession = nullptr;

        if (mCbLooper != nullptr) {

            mCbLooper->unregisterHandler(mHandler->id());

            mCbLooper->stop();

        LOG_ALWAYS_FATAL_IF(mCbLooper != nullptr,

            "CameraDevice looper should've been stopped before ~CameraDevice");

    }

}

    mCbLooper.clear();

    mHandler.clear();

}



void

CameraDevice::postSessionMsgAndCleanup(sp<AMessage>& msg) {
@@ -1410,6 +1400,16 @@ CameraDevice::checkAndFireSequenceCompleteLocked() { 
    }

}



void CameraDevice::stopLooper() {

    Mutex::Autolock _l(mDeviceLock);

    if (mCbLooper != nullptr) {

      mCbLooper->unregisterHandler(mHandler->id());

      mCbLooper->stop();

    }

    mCbLooper.clear();

    mHandler.clear();

}



/**

  * Camera service callback implementation

  */

camera/ndk/ndk_vendor/impl/ACameraDevice.h

+3 −0
Original line number Diff line number Diff line
@@ -133,6 +133,9 @@ class CameraDevice final : public RefBase { 
    bool setDeviceMetadataQueues();

    inline ACameraDevice* getWrapper() const { return mWrapper; };



    // Stop the looper thread and unregister the handler

    void stopLooper();



  private:

    friend ACameraCaptureSession;

    friend ACameraDevice;