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

Commit 30bc34f1 authored by Jeff Brown's avatar Jeff Brown
Browse files

Fix surface frame size reporting. 

The SurfaceHolder provided by the wallpaper service was not reporting
the correct size in getSurfaceFrame().  This broke an optimization in
the ImageWallpaper.  The old code happened to work because calling
lockCanvas on the SurfaceHolder with a null dirty rectangle happened
to have the side-effect of updating the SurfaceHolder's surface frame
size field because it passed mSurfaceFrame as the dirty rect, causing
mSurfaceFrame to be set to the size of the region to be drawn.

However, relying on this side-effect is wrong.  Among other things,
the dirty region could actually be smaller than the surface frame.

This patch fixes WallpaperService, SurfaceView and ViewRoot to ensure
that the surface frame size is always set explicitly and is not modified
by calls to lockCanvas.

Change-Id: I10948f5ec269409ceaf0f7d32b3f6731e9499ebc
parent b8942108

core/java/android/service/wallpaper/WallpaperService.java

+2 −1
Original line number Diff line number Diff line
@@ -547,6 +547,7 @@ public abstract class WallpaperService extends Service { 
                        mCurHeight = h;

                    }



                    mSurfaceHolder.setSurfaceFrameSize(w, h);

                    mSurfaceHolder.mSurfaceLock.unlock();



                    if (!mSurfaceHolder.mSurface.isValid()) {

core/java/android/view/SurfaceView.java

+10 −2
Original line number Diff line number Diff line
@@ -158,6 +158,7 @@ public class SurfaceView extends View { 
    int mHeight = -1;

    int mFormat = -1;

    final Rect mSurfaceFrame = new Rect();

    Rect mTmpDirty;

    int mLastSurfaceWidth = -1, mLastSurfaceHeight = -1;

    boolean mUpdateWindowNeeded;

    boolean mReportDrawNeeded;
@@ -739,9 +740,16 @@ public class SurfaceView extends View { 


            Canvas c = null;

            if (!mDrawingStopped && mWindow != null) {

                Rect frame = dirty != null ? dirty : mSurfaceFrame;

                if (dirty == null) {

                    if (mTmpDirty == null) {

                        mTmpDirty = new Rect();

                    }

                    mTmpDirty.set(mSurfaceFrame);

                    dirty = mTmpDirty;

                }



                try {

                    c = mSurface.lockCanvas(frame);

                    c = mSurface.lockCanvas(dirty);

                } catch (Exception e) {

                    Log.e(LOG_TAG, "Exception locking surface", e);

                }

core/java/android/view/ViewRoot.java

+1 −0
Original line number Diff line number Diff line
@@ -1082,6 +1082,7 @@ public final class ViewRoot extends Handler implements ViewParent, 
                    //mSurfaceHolder.mSurface.copyFrom(mSurface);

                    mSurfaceHolder.mSurface = mSurface;

                }

                mSurfaceHolder.setSurfaceFrameSize(mWidth, mHeight);

                mSurfaceHolder.mSurfaceLock.unlock();

                if (mSurface.isValid()) {

                    if (!hadSurface) {

core/java/com/android/internal/service/wallpaper/ImageWallpaper.java

+36 −22
Original line number Diff line number Diff line
@@ -88,9 +88,13 @@ public class ImageWallpaper extends WallpaperService { 


        class WallpaperObserver extends BroadcastReceiver {

            public void onReceive(Context context, Intent intent) {

                if (DEBUG) {

                    Log.d(TAG, "onReceive");

                }



                synchronized (mLock) {

                    updateWallpaperLocked();

                    drawFrameLocked(true, false);

                    drawFrameLocked();

                }



                // Assume we are the only one using the wallpaper in this
@@ -101,6 +105,10 @@ public class ImageWallpaper extends WallpaperService { 


        @Override

        public void onCreate(SurfaceHolder surfaceHolder) {

            if (DEBUG) {

                Log.d(TAG, "onCreate");

            }



            super.onCreate(surfaceHolder);

            IntentFilter filter = new IntentFilter(Intent.ACTION_WALLPAPER_CHANGED);

            mReceiver = new WallpaperObserver();
@@ -120,9 +128,18 @@ public class ImageWallpaper extends WallpaperService { 


        @Override

        public void onVisibilityChanged(boolean visible) {

            if (DEBUG) {

                Log.d(TAG, "onVisibilityChanged: visible=" + visible);

            }



            synchronized (mLock) {

                if (mVisible != visible) {

                    if (DEBUG) {

                        Log.d(TAG, "Visibility changed to visible=" + visible);

                    }

                    mVisible = visible;

                drawFrameLocked(false, false);

                    drawFrameLocked();

                }

            }

        }
@@ -135,6 +152,12 @@ public class ImageWallpaper extends WallpaperService { 
        public void onOffsetsChanged(float xOffset, float yOffset,

                float xOffsetStep, float yOffsetStep,

                int xPixels, int yPixels) {

            if (DEBUG) {

                Log.d(TAG, "onOffsetsChanged: xOffset=" + xOffset + ", yOffset=" + yOffset

                        + ", xOffsetStep=" + xOffsetStep + ", yOffsetStep=" + yOffsetStep

                        + ", xPixels=" + xPixels + ", yPixels=" + yPixels);

            }



            synchronized (mLock) {

                if (mXOffset != xOffset || mYOffset != yOffset) {

                    if (DEBUG) {
@@ -142,36 +165,27 @@ public class ImageWallpaper extends WallpaperService { 
                    }

                    mXOffset = xOffset;

                    mYOffset = yOffset;

                    drawFrameLocked(false, true);

                } else {

                    drawFrameLocked(false, false);

                    mOffsetsChanged = true;

                }

                drawFrameLocked();

            }

        }



        @Override

        public void onSurfaceChanged(SurfaceHolder holder, int format, int width, int height) {

            if (DEBUG) {

                Log.d(TAG, "onSurfaceChanged: width=" + width + ", height=" + height);

            }



            super.onSurfaceChanged(holder, format, width, height);



            synchronized (mLock) {

                drawFrameLocked(true, false);

            }

        }



        @Override

        public void onSurfaceCreated(SurfaceHolder holder) {

            super.onSurfaceCreated(holder);

                mRedrawNeeded = true;

                drawFrameLocked();

            }



        @Override

        public void onSurfaceDestroyed(SurfaceHolder holder) {

            super.onSurfaceDestroyed(holder);

        }



        void drawFrameLocked(boolean redrawNeeded, boolean offsetsChanged) {

            mRedrawNeeded |= redrawNeeded;

            mOffsetsChanged |= offsetsChanged;



        void drawFrameLocked() {

            if (!mVisible) {

                if (DEBUG) {

                    Log.d(TAG, "Suppressed drawFrame since wallpaper is not visible.");

core/java/com/android/internal/view/BaseSurfaceHolder.java

+17 −2
Original line number Diff line number Diff line
@@ -49,6 +49,7 @@ public abstract class BaseSurfaceHolder implements SurfaceHolder { 
    

    int mType = -1;

    final Rect mSurfaceFrame = new Rect();

    Rect mTmpDirty;

    

    public abstract void onUpdateSurface();

    public abstract void onRelayoutContainer();
@@ -171,9 +172,16 @@ public abstract class BaseSurfaceHolder implements SurfaceHolder { 


        Canvas c = null;

        if (onAllowLockCanvas()) {

            Rect frame = dirty != null ? dirty : mSurfaceFrame;

            if (dirty == null) {

                if (mTmpDirty == null) {

                    mTmpDirty = new Rect();

                }

                mTmpDirty.set(mSurfaceFrame);

                dirty = mTmpDirty;

            }



            try {

                c = mSurface.lockCanvas(frame);

                c = mSurface.lockCanvas(dirty);

            } catch (Exception e) {

                Log.e(TAG, "Exception locking surface", e);

            }
@@ -215,4 +223,11 @@ public abstract class BaseSurfaceHolder implements SurfaceHolder { 
    public Rect getSurfaceFrame() {

        return mSurfaceFrame;

    }



    public void setSurfaceFrameSize(int width, int height) {

        mSurfaceFrame.top = 0;

        mSurfaceFrame.left = 0;

        mSurfaceFrame.right = width;

        mSurfaceFrame.bottom = height;

    }

};