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

Commit 3d3692b2 authored by Rob Carr's avatar Rob Carr Committed by Automerger Merge Worker
Browse files

Merge "SurfaceView: Fix overlocking of mSurfaceLock" into tm-dev am: ec6577d1 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/17808703



Change-Id: I6a5e4b0e613d3bfa418453defa5cd9bcc4b35aad
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 37b47793 ec6577d1

core/java/android/view/SurfaceView.java

+112 −104
Original line number Diff line number Diff line
@@ -720,8 +720,13 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
    private void releaseSurfaces(boolean releaseSurfacePackage) {

        mSurfaceAlpha = 1f;

	

        synchronized (mSurfaceControlLock) {

        mSurfaceLock.lock();

        try {

            mSurface.destroy();

        } finally {

            mSurfaceLock.unlock();

        }

        synchronized (mSurfaceControlLock) {

            if (mBlastBufferQueue != null) {

                mBlastBufferQueue.destroy();

                mBlastBufferQueue = null;
@@ -770,8 +775,6 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
            Transaction surfaceUpdateTransaction) {

        boolean realSizeChanged = false;



        mSurfaceLock.lock();

        try {

        mDrawingStopped = !mVisible;



        if (DEBUG) Log.i(TAG, System.identityHashCode(this) + " "
@@ -811,7 +814,6 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
            setBufferSize(surfaceUpdateTransaction);

        }

        if (sizeChanged || creating || !isHardwareAccelerated()) {



            // Set a window crop when creating the surface or changing its size to

            // crop the buffer to the surface size since the buffer producer may

            // use SCALING_MODE_SCALE and submit a larger size than the surface
@@ -849,7 +851,6 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
        }

        applyTransactionOnVriDraw(surfaceUpdateTransaction);

        updateEmbeddedAccessibilityMatrix(false);



         mSurfaceFrame.left = 0;

        mSurfaceFrame.top = 0;

        if (translator == null) {
@@ -866,9 +867,7 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
                || mLastSurfaceHeight != surfaceHeight;

        mLastSurfaceWidth = surfaceWidth;

        mLastSurfaceHeight = surfaceHeight;

        } finally {

            mSurfaceLock.unlock();

        }



        return realSizeChanged;

    }
@@ -1103,21 +1102,30 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
     *                          Surface for compatibility reasons.

     */

    private void copySurface(boolean surfaceControlCreated, boolean bufferSizeChanged) {

        if (surfaceControlCreated) {

            mSurface.copyFrom(mBlastBufferQueue);

        }



        if (bufferSizeChanged && getContext().getApplicationInfo().targetSdkVersion

                < Build.VERSION_CODES.O) {

        // Some legacy applications use the underlying native {@link Surface} object

        // as a key to whether anything has changed. In these cases, updates to the

        // existing {@link Surface} will be ignored when the size changes.

        // Therefore, we must explicitly recreate the {@link Surface} in these

        // cases.

        boolean needsWorkaround = bufferSizeChanged &&

            getContext().getApplicationInfo().targetSdkVersion < Build.VERSION_CODES.O;

       if (!surfaceControlCreated && !needsWorkaround) {

           return;

       }

       mSurfaceLock.lock();

       try {

           if (surfaceControlCreated) {

               mSurface.copyFrom(mBlastBufferQueue);

           }



           if (needsWorkaround) {

               if (mBlastBufferQueue != null) {

                   mSurface.transferFrom(mBlastBufferQueue.createSurfaceWithHandle());

               }

           }

       } finally {

           mSurfaceLock.unlock();

       }

    }



    private void setBufferSize(Transaction transaction) {