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

Commit c20dd276 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Sharesheet - Resolve crash when no targets are present"

parents 10c0ac64 0f457e6f

core/java/com/android/internal/app/ChooserActivity.java

+2 −1
Original line number Diff line number Diff line
@@ -237,7 +237,7 @@ public class ChooserActivity extends ResolverActivity { 
                    if (DEBUG) {

                        Log.d(TAG, "CHOOSER_TARGET_SERVICE_WATCHDOG_TIMEOUT; unbinding services");

                    }

                    if (isDestroyed()) {

                    if (mChooserListAdapter == null || isDestroyed()) {

                        break;

                    }

                    unbindRemainingServices();
@@ -822,6 +822,7 @@ public class ChooserActivity extends ResolverActivity { 
            mRefinementResultReceiver = null;

        }

        unbindRemainingServices();

        mChooserHandler.removeMessages(CHOOSER_TARGET_SERVICE_WATCHDOG_TIMEOUT);

        mChooserHandler.removeMessages(CHOOSER_TARGET_SERVICE_RESULT);

        if (USE_PREDICTION_MANAGER_FOR_DIRECT_TARGETS) {

            mAppPredictor.unregisterPredictionUpdates(mAppPredictorCallback);

core/tests/coretests/src/com/android/internal/app/ChooserActivityTest.java

+2 −2
Original line number Diff line number Diff line
@@ -599,7 +599,7 @@ public class ChooserActivityTest { 


        mActivityRule.launchActivity(Intent.createChooser(sendIntent, null));

        waitForIdle();

        verify(mockLogger, Mockito.times(2)).write(logMakerCaptor.capture());

        verify(mockLogger, Mockito.times(3)).write(logMakerCaptor.capture());

        // First invocation is from onCreate

        assertThat(logMakerCaptor.getAllValues().get(1).getCategory(),

                is(MetricsEvent.ACTION_SHARE_WITH_PREVIEW));
@@ -629,7 +629,7 @@ public class ChooserActivityTest { 
        ArgumentCaptor<LogMaker> logMakerCaptor = ArgumentCaptor.forClass(LogMaker.class);

        mActivityRule.launchActivity(Intent.createChooser(sendIntent, null));

        waitForIdle();

        verify(mockLogger, Mockito.times(2)).write(logMakerCaptor.capture());

        verify(mockLogger, Mockito.times(3)).write(logMakerCaptor.capture());

        // First invocation is from onCreate

        assertThat(logMakerCaptor.getAllValues().get(1).getCategory(),

                is(MetricsEvent.ACTION_SHARE_WITH_PREVIEW));