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

Commit 742684e0 authored by Alec Mouri's avatar Alec Mouri Committed by Automerger Merge Worker
Browse files

Merge "Drive SurfaceView visibility on renderthread" into udc-dev am: 0f351cb6 am: add4d9d5 

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



Change-Id: I690b75c2adf1d7f35487904b80d648ec6db0b6a9
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 3e17ded3 add4d9d5

core/java/android/view/SurfaceView.java

+11 −21
Original line number Diff line number Diff line
@@ -851,11 +851,15 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
            }

            mParentSurfaceSequenceId = viewRoot.getSurfaceSequenceId();



            // Only control visibility if we're not hardware-accelerated. Otherwise we'll

            // let renderthread drive since offscreen SurfaceControls should not be visible.

            if (!isHardwareAccelerated()) {

                if (mViewVisibility) {

                    surfaceUpdateTransaction.show(mSurfaceControl);

                } else {

                    surfaceUpdateTransaction.hide(mSurfaceControl);

                }

            }



            updateBackgroundVisibility(surfaceUpdateTransaction);

            updateBackgroundColor(surfaceUpdateTransaction);
@@ -1417,12 +1421,10 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
    }



    private final Rect mRTLastReportedPosition = new Rect();

    private final Point mRTLastReportedSurfaceSize = new Point();



    private class SurfaceViewPositionUpdateListener implements RenderNode.PositionUpdateListener {

        private final int mRtSurfaceWidth;

        private final int mRtSurfaceHeight;

        private boolean mRtFirst = true;

        private final SurfaceControl.Transaction mPositionChangedTransaction =

                new SurfaceControl.Transaction();
@@ -1433,15 +1435,6 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 


        @Override

        public void positionChanged(long frameNumber, int left, int top, int right, int bottom) {

            if (!mRtFirst && (mRTLastReportedPosition.left == left

                    && mRTLastReportedPosition.top == top

                    && mRTLastReportedPosition.right == right

                    && mRTLastReportedPosition.bottom == bottom

                    && mRTLastReportedSurfaceSize.x == mRtSurfaceWidth

                    && mRTLastReportedSurfaceSize.y == mRtSurfaceHeight)) {

                return;

            }

            mRtFirst = false;

            try {

                if (DEBUG_POSITION) {

                    Log.d(TAG, String.format(
@@ -1452,8 +1445,8 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
                }

                synchronized (mSurfaceControlLock) {

                    if (mSurfaceControl == null) return;



                    mRTLastReportedPosition.set(left, top, right, bottom);

                    mRTLastReportedSurfaceSize.set(mRtSurfaceWidth, mRtSurfaceHeight);

                    onSetSurfacePositionAndScale(mPositionChangedTransaction, mSurfaceControl,

                            mRTLastReportedPosition.left /*positionLeft*/,

                            mRTLastReportedPosition.top /*positionTop*/,
@@ -1461,11 +1454,9 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
                                    / (float) mRtSurfaceWidth /*postScaleX*/,

                            mRTLastReportedPosition.height()

                                    / (float) mRtSurfaceHeight /*postScaleY*/);

                    if (mViewVisibility) {

                        // b/131239825



                    mPositionChangedTransaction.show(mSurfaceControl);

                }

                }

                applyOrMergeTransaction(mPositionChangedTransaction, frameNumber);

            } catch (Exception ex) {

                Log.e(TAG, "Exception from repositionChild", ex);
@@ -1490,7 +1481,6 @@ public class SurfaceView extends View implements ViewRootImpl.SurfaceChangedCall 
                        System.identityHashCode(this), frameNumber));

            }

            mRTLastReportedPosition.setEmpty();

            mRTLastReportedSurfaceSize.set(-1, -1);



            // positionLost can be called while UI thread is un-paused.

            synchronized (mSurfaceControlLock) {