summaryrefslogtreecommitdiff
path: root/java
diff options
context:
space:
mode:
author Mark Renouf <mrenouf@google.com> 2023-11-30 15:47:38 -0500
committer Mark Renouf <mrenouf@google.com> 2023-11-30 17:10:39 -0500
commit441be5d729aee09aee7b6f6d03f0761b7c1460bd (patch)
treedef5fb1e733614ca3b46f28fd692f6e62b947d02 /java
parentf12d9d28814082c0843b4a624a893c084a13378f (diff)
Narrow dependencies to actual used data
Currently PreviewViewModel uses only the targetIntent. This change narrows the dependencies to that specific value to reduce coupling and aid cleanup efforts. Bug: 300157408 Test: atest com.android.intentresolver Change-Id: I63ceda3c460f039939a4193970ee4fe078a96467
Diffstat (limited to 'java')
-rw-r--r--java/src/com/android/intentresolver/ChooserActivity.java2
-rw-r--r--java/src/com/android/intentresolver/contentpreview/BasePreviewViewModel.kt3
-rw-r--r--java/src/com/android/intentresolver/contentpreview/PreviewViewModel.kt5
-rw-r--r--java/src/com/android/intentresolver/v2/ChooserActivity.java2
4 files changed, 7 insertions, 5 deletions
diff --git a/java/src/com/android/intentresolver/ChooserActivity.java b/java/src/com/android/intentresolver/ChooserActivity.java
index 2f950baa..aa02be37 100644
--- a/java/src/com/android/intentresolver/ChooserActivity.java
+++ b/java/src/com/android/intentresolver/ChooserActivity.java
@@ -304,7 +304,7 @@ public class ChooserActivity extends Hilt_ChooserActivity implements
.get(BasePreviewViewModel.class);
mChooserContentPreviewUi = new ChooserContentPreviewUi(
getCoroutineScope(getLifecycle()),
- previewViewModel.createOrReuseProvider(mChooserRequest),
+ previewViewModel.createOrReuseProvider(mChooserRequest.getTargetIntent()),
mChooserRequest.getTargetIntent(),
previewViewModel.createOrReuseImageLoader(),
createChooserActionFactory(),
diff --git a/java/src/com/android/intentresolver/contentpreview/BasePreviewViewModel.kt b/java/src/com/android/intentresolver/contentpreview/BasePreviewViewModel.kt
index 103e8bf4..10ee5af1 100644
--- a/java/src/com/android/intentresolver/contentpreview/BasePreviewViewModel.kt
+++ b/java/src/com/android/intentresolver/contentpreview/BasePreviewViewModel.kt
@@ -16,6 +16,7 @@
package com.android.intentresolver.contentpreview
+import android.content.Intent
import androidx.annotation.MainThread
import androidx.lifecycle.ViewModel
import com.android.intentresolver.ChooserRequestParameters
@@ -24,7 +25,7 @@ import com.android.intentresolver.ChooserRequestParameters
abstract class BasePreviewViewModel : ViewModel() {
@MainThread
abstract fun createOrReuseProvider(
- chooserRequest: ChooserRequestParameters
+ targetIntent: Intent
): PreviewDataProvider
@MainThread abstract fun createOrReuseImageLoader(): ImageLoader
diff --git a/java/src/com/android/intentresolver/contentpreview/PreviewViewModel.kt b/java/src/com/android/intentresolver/contentpreview/PreviewViewModel.kt
index b55b8b38..6350756e 100644
--- a/java/src/com/android/intentresolver/contentpreview/PreviewViewModel.kt
+++ b/java/src/com/android/intentresolver/contentpreview/PreviewViewModel.kt
@@ -17,6 +17,7 @@
package com.android.intentresolver.contentpreview
import android.app.Application
+import android.content.Intent
import androidx.annotation.MainThread
import androidx.lifecycle.ViewModel
import androidx.lifecycle.ViewModelProvider
@@ -45,12 +46,12 @@ constructor(
@MainThread
override fun createOrReuseProvider(
- chooserRequest: ChooserRequestParameters
+ targetIntent: Intent
): PreviewDataProvider =
previewDataProvider
?: PreviewDataProvider(
viewModelScope + dispatcher,
- chooserRequest.targetIntent,
+ targetIntent,
application.contentResolver
)
.also { previewDataProvider = it }
diff --git a/java/src/com/android/intentresolver/v2/ChooserActivity.java b/java/src/com/android/intentresolver/v2/ChooserActivity.java
index a854c9e0..c7a8ebab 100644
--- a/java/src/com/android/intentresolver/v2/ChooserActivity.java
+++ b/java/src/com/android/intentresolver/v2/ChooserActivity.java
@@ -290,7 +290,7 @@ public class ChooserActivity extends Hilt_ChooserActivity implements
ChooserRequestParameters chooserRequest = requireChooserRequest();
mChooserContentPreviewUi = new ChooserContentPreviewUi(
getCoroutineScope(getLifecycle()),
- previewViewModel.createOrReuseProvider(chooserRequest),
+ previewViewModel.createOrReuseProvider(chooserRequest.getTargetIntent()),
chooserRequest.getTargetIntent(),
previewViewModel.createOrReuseImageLoader(),
createChooserActionFactory(),