From cfd972848d9a4bd873c3740d28673db8edb2fb88 Mon Sep 17 00:00:00 2001 From: Andrey Epin Date: Fri, 15 Dec 2023 16:47:33 -0800 Subject: Some code cleanup Bug: 309960444 Test: atest com.android.intentresolver.v2 Flag: ACONFIG com.android.intentresolver.modular_framework Development Change-Id: I5be0f1c9659be091d7337e0eebfcf2e85466a1c8 --- .../android/intentresolver/v2/ChooserActivity.java | 54 ++++++++++------------ 1 file changed, 24 insertions(+), 30 deletions(-) (limited to 'java/src') diff --git a/java/src/com/android/intentresolver/v2/ChooserActivity.java b/java/src/com/android/intentresolver/v2/ChooserActivity.java index 2d2c71af..e281613d 100644 --- a/java/src/com/android/intentresolver/v2/ChooserActivity.java +++ b/java/src/com/android/intentresolver/v2/ChooserActivity.java @@ -345,22 +345,22 @@ public class ChooserActivity extends Hilt_ChooserActivity implements mShouldDisplayLandscape = shouldDisplayLandscape(getResources().getConfiguration().orientation); - ChooserRequestParameters chooserRequest = getChooserRequest(); - if (chooserRequest != null) { - setRetainInOnStop(chooserRequest.shouldRetainInOnStop()); - } - ChooserRequestParameters chooserRequest1 = getChooserRequest(); - if (chooserRequest1 != null) { - createProfileRecords( - new AppPredictorFactory( - this, - chooserRequest1.getSharedText(), - chooserRequest1.getTargetIntentFilter() - ), - chooserRequest1.getTargetIntentFilter() - ); + if (chooserRequest == null) { + finish(); + return; } + + setRetainInOnStop(chooserRequest.shouldRetainInOnStop()); + createProfileRecords( + new AppPredictorFactory( + this, + chooserRequest.getSharedText(), + chooserRequest.getTargetIntentFilter() + ), + chooserRequest.getTargetIntentFilter() + ); + Intent intent = mLogic.getTargetIntent(); List initialIntents = mLogic.getInitialIntents(); TargetDataLoader targetDataLoader = mLogic.getTargetDataLoader(); @@ -431,11 +431,6 @@ public class ChooserActivity extends Hilt_ChooserActivity implements : "")); } - if (getChooserRequest() == null) { - finish(); - return; - } - getEventLog().logSharesheetTriggered(); mRefinementManager = new ViewModelProvider(this).get(ChooserRefinementManager.class); mRefinementManager.getRefinementCompletion().observe(this, completion -> { @@ -465,11 +460,10 @@ public class ChooserActivity extends Hilt_ChooserActivity implements BasePreviewViewModel previewViewModel = new ViewModelProvider(this, createPreviewViewModelFactory()) .get(BasePreviewViewModel.class); - ChooserRequestParameters chooserRequest2 = requireChooserRequest(); mChooserContentPreviewUi = new ChooserContentPreviewUi( getCoroutineScope(getLifecycle()), - previewViewModel.createOrReuseProvider(chooserRequest2.getTargetIntent()), - chooserRequest2.getTargetIntent(), + previewViewModel.createOrReuseProvider(chooserRequest.getTargetIntent()), + chooserRequest.getTargetIntent(), previewViewModel.createOrReuseImageLoader(), createChooserActionFactory(), mEnterTransitionAnimationDelegate, @@ -484,7 +478,7 @@ public class ChooserActivity extends Hilt_ChooserActivity implements mChooserShownTime = System.currentTimeMillis(); final long systemCost = mChooserShownTime - mIntentReceivedTime.get(); getEventLog().logChooserActivityShown( - isWorkProfile(), chooserRequest2.getTargetType(), systemCost); + isWorkProfile(), chooserRequest.getTargetType(), systemCost); if (mResolverDrawerLayout != null) { mResolverDrawerLayout.addOnLayoutChangeListener(this::handleLayoutChange); @@ -499,15 +493,15 @@ public class ChooserActivity extends Hilt_ChooserActivity implements } getEventLog().logShareStarted( mLogic.getReferrerPackageName(), - chooserRequest2.getTargetType(), - chooserRequest2.getCallerChooserTargets().size(), - (chooserRequest2.getInitialIntents() == null) - ? 0 : chooserRequest2.getInitialIntents().length, + chooserRequest.getTargetType(), + chooserRequest.getCallerChooserTargets().size(), + (chooserRequest.getInitialIntents() == null) + ? 0 : chooserRequest.getInitialIntents().length, isWorkProfile(), mChooserContentPreviewUi.getPreferredContentPreview(), - chooserRequest2.getTargetAction(), - chooserRequest2.getChooserActions().size(), - chooserRequest2.getModifyShareAction() != null + chooserRequest.getTargetAction(), + chooserRequest.getChooserActions().size(), + chooserRequest.getModifyShareAction() != null ); mEnterTransitionAnimationDelegate.postponeTransition(); -- cgit v1.2.3-59-g8ed1b