From f760ae087ee9f3dae6b7533d5d4c2525103fd1cf Mon Sep 17 00:00:00 2001
From: Andrey Yepin <ayepin@google.com>
Date: Mon, 19 Aug 2024 21:23:31 -0700
Subject: Log payload selection changes

Bug: 329068612
Test: manual testing with an injected debug logging
Test: atest IntentResolver-tests-unit
Flag: EXEMPT ui events logging
Change-Id: Iac9e2bcf93375d5761be5287b6cf9297be399e77
---
 .../domain/interactor/SelectablePreviewInteractor.kt      |  3 +++
 .../domain/interactor/SelectablePreviewsInteractor.kt     |  4 +++-
 java/src/com/android/intentresolver/logging/EventLog.kt   | 15 +++++++++++++++
 .../com/android/intentresolver/logging/EventLogImpl.java  |  5 +++++
 4 files changed, 26 insertions(+), 1 deletion(-)

(limited to 'java/src')

diff --git a/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewInteractor.kt b/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewInteractor.kt
index d52a71a1..8f18ebe0 100644
--- a/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewInteractor.kt
+++ b/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewInteractor.kt
@@ -18,6 +18,7 @@ package com.android.intentresolver.contentpreview.payloadtoggle.domain.interacto
 
 import android.net.Uri
 import com.android.intentresolver.contentpreview.payloadtoggle.shared.model.PreviewModel
+import com.android.intentresolver.logging.EventLog
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.map
 
@@ -25,6 +26,7 @@ import kotlinx.coroutines.flow.map
 class SelectablePreviewInteractor(
     private val key: PreviewModel,
     private val selectionInteractor: SelectionInteractor,
+    private val eventLog: EventLog,
 ) {
     val uri: Uri = key.uri
 
@@ -33,6 +35,7 @@ class SelectablePreviewInteractor(
 
     /** Sets whether this preview is selected by the user. */
     fun setSelected(isSelected: Boolean) {
+        eventLog.logPayloadSelectionChanged()
         if (isSelected) {
             selectionInteractor.select(key)
         } else {
diff --git a/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewsInteractor.kt b/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewsInteractor.kt
index a578d0e2..d0ac8d4a 100644
--- a/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewsInteractor.kt
+++ b/java/src/com/android/intentresolver/contentpreview/payloadtoggle/domain/interactor/SelectablePreviewsInteractor.kt
@@ -19,6 +19,7 @@ package com.android.intentresolver.contentpreview.payloadtoggle.domain.interacto
 import com.android.intentresolver.contentpreview.payloadtoggle.data.repository.CursorPreviewsRepository
 import com.android.intentresolver.contentpreview.payloadtoggle.shared.model.PreviewModel
 import com.android.intentresolver.contentpreview.payloadtoggle.shared.model.PreviewsModel
+import com.android.intentresolver.logging.EventLog
 import javax.inject.Inject
 import kotlinx.coroutines.flow.Flow
 
@@ -27,6 +28,7 @@ class SelectablePreviewsInteractor
 constructor(
     private val previewsRepo: CursorPreviewsRepository,
     private val selectionInteractor: SelectionInteractor,
+    private val eventLog: EventLog,
 ) {
     /** Keys of previews available for display in Shareousel. */
     val previews: Flow<PreviewsModel?>
@@ -36,5 +38,5 @@ constructor(
      * Returns a [SelectablePreviewInteractor] that can be used to interact with the individual
      * preview associated with [key].
      */
-    fun preview(key: PreviewModel) = SelectablePreviewInteractor(key, selectionInteractor)
+    fun preview(key: PreviewModel) = SelectablePreviewInteractor(key, selectionInteractor, eventLog)
 }
diff --git a/java/src/com/android/intentresolver/logging/EventLog.kt b/java/src/com/android/intentresolver/logging/EventLog.kt
index 476bd4bf..b92f0732 100644
--- a/java/src/com/android/intentresolver/logging/EventLog.kt
+++ b/java/src/com/android/intentresolver/logging/EventLog.kt
@@ -47,6 +47,7 @@ interface EventLog {
     )
 
     fun logCustomActionSelected(positionPicked: Int)
+
     fun logShareTargetSelected(
         targetType: Int,
         packageName: String?,
@@ -60,15 +61,29 @@ interface EventLog {
     )
 
     fun logDirectShareTargetReceived(category: Int, latency: Int)
+
     fun logActionShareWithPreview(previewType: Int)
+
     fun logActionSelected(targetType: Int)
+
     fun logContentPreviewWarning(uri: Uri?)
+
     fun logSharesheetTriggered()
+
     fun logSharesheetAppLoadComplete()
+
     fun logSharesheetDirectLoadComplete()
+
     fun logSharesheetDirectLoadTimeout()
+
     fun logSharesheetProfileChanged()
+
     fun logSharesheetExpansionChanged(isCollapsed: Boolean)
+
     fun logSharesheetAppShareRankingTimeout()
+
     fun logSharesheetEmptyDirectShareRow()
+
+    /** Log payload selection */
+    fun logPayloadSelectionChanged()
 }
diff --git a/java/src/com/android/intentresolver/logging/EventLogImpl.java b/java/src/com/android/intentresolver/logging/EventLogImpl.java
index 8d04dd28..8e9543bc 100644
--- a/java/src/com/android/intentresolver/logging/EventLogImpl.java
+++ b/java/src/com/android/intentresolver/logging/EventLogImpl.java
@@ -273,6 +273,11 @@ public class EventLogImpl implements EventLog {
         log(SharesheetStandardEvent.SHARESHEET_EMPTY_DIRECT_SHARE_ROW, mInstanceId);
     }
 
+    @Override
+    public void logPayloadSelectionChanged() {
+        log(SharesheetStandardEvent.SHARESHEET_PAYLOAD_TOGGLED, mInstanceId);
+    }
+
     /**
      * Logs a UiEventReported event for a given share activity
      * @param event
-- 
cgit v1.2.3-59-g8ed1b