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

Commit 1c44dc1c authored by Romain Guy's avatar Romain Guy
Browse files

@#!$# 

Bug #7275145

Removing the window background when it's not visible is a good idea
to save on performance. Removing *all* the backgrounds is just a silly
idea that leads to weird looking results that drive graphics engineers
mad for a day until they figure out that everything is working properly
and that the app is simply asking the system to draw whatever garbage
is left in the GPU's graphics buffers.

Change-Id: Icd12cd4ec6fc313f80acfd8deb7133a17ea11302
parent 32ffa821

src/com/android/gallery3d/app/MoviePlayer.java

+1 −21
Original line number Diff line number Diff line
@@ -74,8 +74,8 @@ public class MoviePlayer implements 
    private static final long RESUMEABLE_TIMEOUT = 3 * 60 * 1000; // 3 mins



    private Context mContext;

    private final VideoView mVideoView;

    private final View mRootView;

    private final VideoView mVideoView;

    private final Bookmarker mBookmarker;

    private final Uri mUri;

    private final Handler mHandler = new Handler();
@@ -104,14 +104,6 @@ public class MoviePlayer implements 
        }

    };



    private final Runnable mRemoveBackground = new Runnable() {

        @SuppressWarnings("deprecation")

        @Override

        public void run() {

            mRootView.setBackgroundDrawable(null);

        }

    };



    private final Runnable mProgressChecker = new Runnable() {

        @Override

        public void run() {
@@ -199,19 +191,7 @@ public class MoviePlayer implements 
                if ((diff & View.SYSTEM_UI_FLAG_HIDE_NAVIGATION) != 0

                        && (visibility & View.SYSTEM_UI_FLAG_HIDE_NAVIGATION) == 0) {

                    mController.show();



                    // We need to set the background to clear ghosting images

                    // when ActionBar slides in. However, if we keep the background,

                    // there will be one additional layer in HW composer, which is bad

                    // to battery. As a solution, we remove the background when we

                    // hide the action bar

                    mHandler.removeCallbacks(mRemoveBackground);

                    mRootView.setBackgroundColor(Color.BLACK);

                } else {

                    mHandler.removeCallbacks(mRemoveBackground);



                    // Wait for the slide out animation, one second should be enough

                    mHandler.postDelayed(mRemoveBackground, 1000);

                }

            }

        });