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

Commit 101047f5 authored by Matt Casey's avatar Matt Casey
Browse files

Move image cleanup operations to onStop 

The Activity may still be visible onPause.

Bug: 183277963
Test: Validate cleanup via logs, ensure image doesn't disappear
Change-Id: I8976293c9c0e070d6bc08e08d177765311201cb8
parent 177d8e36

packages/SystemUI/src/com/android/systemui/screenshot/LongScreenshotActivity.java

+7 −7
Original line number Diff line number Diff line
@@ -270,8 +270,14 @@ public class LongScreenshotActivity extends Activity { 


    @Override

    protected void onPause() {

        Log.d(TAG, "onPause finishing=" + isFinishing());

        Log.d(TAG, "onPause");

        super.onPause();

    }



    @Override

    protected void onStop() {

        Log.d(TAG, "onStop finishing=" + isFinishing());

        super.onStop();

        if (isFinishing()) {

            if (mScrollCaptureResponse != null) {

                mScrollCaptureResponse.close();
@@ -299,12 +305,6 @@ public class LongScreenshotActivity extends Activity { 
        }

    }



    @Override

    protected void onStop() {

        Log.d(TAG, "onStop");

        super.onStop();

    }



    @Override

    protected void onDestroy() {

        Log.d(TAG, "onDestroy");