Merge changes I13fbf290,I7440a6bb into main

* changes:
  Switches to HiltTestApplication in tests.
  Extract an interface for EventLog, rename implementation

8 files changed, 170 insertions, 73 deletions

diff --git a/java/src/com/android/intentresolver/ChooserActionFactory.java b/java/src/com/android/intentresolver/ChooserActionFactory.java
index a54e8c62..2c97c0b1 100644
--- a/java/src/com/android/intentresolver/ChooserActionFactory.java
+++ b/java/src/com/android/intentresolver/ChooserActionFactory.java
@@ -98,7 +98,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
     private final @Nullable ChooserAction mModifyShareAction;
     private final Consumer<Boolean> mExcludeSharedTextAction;
     private final Consumer</* @Nullable */ Integer> mFinishCallback;
-    private final EventLog mLogger;
+    private final EventLog mLog;
 
     /**
      * @param context
@@ -117,7 +117,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
             Context context,
             ChooserRequestParameters chooserRequest,
             ChooserIntegratedDeviceComponents integratedDeviceComponents,
-            EventLog logger,
+            EventLog log,
             Consumer<Boolean> onUpdateSharedTextIsExcluded,
             Callable</* @Nullable */ View> firstVisibleImageQuery,
             ActionActivityStarter activityStarter,
@@ -129,7 +129,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
                         chooserRequest.getTargetIntent(),
                         chooserRequest.getReferrerPackageName(),
                         finishCallback,
-                        logger),
+                        log),
                 makeEditButtonRunnable(
                         getEditSharingTarget(
                                 context,
@@ -137,11 +137,11 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
                                 integratedDeviceComponents),
                         firstVisibleImageQuery,
                         activityStarter,
-                        logger),
+                        log),
                 chooserRequest.getChooserActions(),
                 chooserRequest.getModifyShareAction(),
                 onUpdateSharedTextIsExcluded,
-                logger,
+                log,
                 finishCallback);
     }
 
@@ -153,7 +153,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
             List<ChooserAction> customActions,
             @Nullable ChooserAction modifyShareAction,
             Consumer<Boolean> onUpdateSharedTextIsExcluded,
-            EventLog logger,
+            EventLog log,
             Consumer</* @Nullable */ Integer> finishCallback) {
         mContext = context;
         mCopyButtonRunnable = copyButtonRunnable;
@@ -161,7 +161,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
         mCustomActions = ImmutableList.copyOf(customActions);
         mModifyShareAction = modifyShareAction;
         mExcludeSharedTextAction = onUpdateSharedTextIsExcluded;
-        mLogger = logger;
+        mLog = log;
         mFinishCallback = finishCallback;
     }
 
@@ -188,7 +188,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
                     mCustomActions.get(i),
                     mFinishCallback,
                     () -> {
-                        mLogger.logCustomActionSelected(position);
+                        mLog.logCustomActionSelected(position);
                     }
             );
             if (actionRow != null) {
@@ -209,7 +209,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
                 mModifyShareAction,
                 mFinishCallback,
                 () -> {
-                    mLogger.logActionSelected(EventLog.SELECTION_TYPE_MODIFY_SHARE);
+                    mLog.logActionSelected(EventLog.SELECTION_TYPE_MODIFY_SHARE);
                 });
     }
 
@@ -233,7 +233,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
             Intent targetIntent,
             String referrerPackageName,
             Consumer<Integer> finishCallback,
-            EventLog logger) {
+            EventLog log) {
         final ClipData clipData;
         try {
             clipData = extractTextToCopy(targetIntent);
@@ -249,7 +249,7 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
                     Context.CLIPBOARD_SERVICE);
             clipboardManager.setPrimaryClipAsPackage(clipData, referrerPackageName);
 
-            logger.logActionSelected(EventLog.SELECTION_TYPE_COPY);
+            log.logActionSelected(EventLog.SELECTION_TYPE_COPY);
             finishCallback.accept(Activity.RESULT_OK);
         };
     }
@@ -328,10 +328,10 @@ public final class ChooserActionFactory implements ChooserContentPreviewUi.Actio
             TargetInfo editSharingTarget,
             Callable</* @Nullable */ View> firstVisibleImageQuery,
             ActionActivityStarter activityStarter,
-            EventLog logger) {
+            EventLog log) {
         return () -> {
             // Log share completion via edit.
-            logger.logActionSelected(EventLog.SELECTION_TYPE_EDIT);
+            log.logActionSelected(EventLog.SELECTION_TYPE_EDIT);
 
             View firstImageView = null;
             try {
diff --git a/java/src/com/android/intentresolver/ChooserActivity.java b/java/src/com/android/intentresolver/ChooserActivity.java
index ebe6f04b..c27d3bac 100644
--- a/java/src/com/android/intentresolver/ChooserActivity.java
+++ b/java/src/com/android/intentresolver/ChooserActivity.java
@@ -85,6 +85,7 @@ import com.android.intentresolver.grid.ChooserGridAdapter;
 import com.android.intentresolver.icons.DefaultTargetDataLoader;
 import com.android.intentresolver.icons.TargetDataLoader;
 import com.android.intentresolver.logging.EventLog;
+import com.android.intentresolver.logging.EventLogImpl;
 import com.android.intentresolver.measurements.Tracer;
 import com.android.intentresolver.model.AbstractResolverComparator;
 import com.android.intentresolver.model.AppPredictionServiceResolverComparator;
@@ -257,7 +258,7 @@ public class ChooserActivity extends Hilt_ChooserActivity implements
 
         createProfileRecords(
                 new AppPredictorFactory(
-                        this, // TODO: Review w/team, possible side effects?
+                        this,
                         mChooserRequest.getSharedText(),
                         mChooserRequest.getTargetIntentFilter()),
                 mChooserRequest.getTargetIntentFilter());
@@ -275,10 +276,6 @@ public class ChooserActivity extends Hilt_ChooserActivity implements
                 new DefaultTargetDataLoader(this, getLifecycle(), false),
                 /* safeForwardingMode= */ true);
 
-        if (mFeatureFlags.exampleNewSharingMethod()) {
-            // Sample flag usage
-        }
-
         mIntegratedDeviceComponents = getIntegratedDeviceComponents();
 
         mRefinementManager = new ViewModelProvider(this).get(ChooserRefinementManager.class);
@@ -386,7 +383,7 @@ public class ChooserActivity extends Hilt_ChooserActivity implements
         ShortcutLoader shortcutLoader = ActivityManager.isLowRamDeviceStatic()
                     ? null
                     : createShortcutLoader(
-                            this, // TODO: Review w/team, possible side effects?
+                            this,
                             appPredictor,
                             userHandle,
                             targetIntentFilter,
@@ -1111,7 +1108,7 @@ public class ChooserActivity extends Hilt_ChooserActivity implements
 
     protected EventLog getEventLog() {
         if (mEventLog == null) {
-            mEventLog = new EventLog();
+            mEventLog = new EventLogImpl();
         }
         return mEventLog;
     }
diff --git a/java/src/com/android/intentresolver/logging/EventLog.kt b/java/src/com/android/intentresolver/logging/EventLog.kt
new file mode 100644
index 00000000..476bd4bf
--- /dev/null
+++ b/java/src/com/android/intentresolver/logging/EventLog.kt
@@ -0,0 +1,74 @@
+/*
+ * Copyright (C) 2023 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.android.intentresolver.logging
+
+import android.net.Uri
+import android.util.HashedStringCache
+
+/** Logs notable events during ShareSheet usage. */
+interface EventLog {
+
+    companion object {
+        const val SELECTION_TYPE_SERVICE = 1
+        const val SELECTION_TYPE_APP = 2
+        const val SELECTION_TYPE_STANDARD = 3
+        const val SELECTION_TYPE_COPY = 4
+        const val SELECTION_TYPE_NEARBY = 5
+        const val SELECTION_TYPE_EDIT = 6
+        const val SELECTION_TYPE_MODIFY_SHARE = 7
+        const val SELECTION_TYPE_CUSTOM_ACTION = 8
+    }
+
+    fun logChooserActivityShown(isWorkProfile: Boolean, targetMimeType: String?, systemCost: Long)
+
+    fun logShareStarted(
+        packageName: String?,
+        mimeType: String?,
+        appProvidedDirect: Int,
+        appProvidedApp: Int,
+        isWorkprofile: Boolean,
+        previewType: Int,
+        intent: String?,
+        customActionCount: Int,
+        modifyShareActionProvided: Boolean
+    )
+
+    fun logCustomActionSelected(positionPicked: Int)
+    fun logShareTargetSelected(
+        targetType: Int,
+        packageName: String?,
+        positionPicked: Int,
+        directTargetAlsoRanked: Int,
+        numCallerProvided: Int,
+        directTargetHashed: HashedStringCache.HashResult?,
+        isPinned: Boolean,
+        successfullySelected: Boolean,
+        selectionCost: Long
+    )
+
+    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()
+}
diff --git a/java/src/com/android/intentresolver/logging/EventLog.java b/java/src/com/android/intentresolver/logging/EventLogImpl.java
index b30e825b..33e617b1 100644
--- a/java/src/com/android/intentresolver/logging/EventLog.java
+++ b/java/src/com/android/intentresolver/logging/EventLogImpl.java
@@ -38,52 +38,11 @@ import com.android.internal.util.FrameworkStatsLog;
 
 /**
  * Helper for writing Sharesheet atoms to statsd log.
- * @hide
  */
-public class EventLog {
+public class EventLogImpl implements EventLog {
     private static final String TAG = "ChooserActivity";
     private static final boolean DEBUG = true;
 
-    public static final int SELECTION_TYPE_SERVICE = 1;
-    public static final int SELECTION_TYPE_APP = 2;
-    public static final int SELECTION_TYPE_STANDARD = 3;
-    public static final int SELECTION_TYPE_COPY = 4;
-    public static final int SELECTION_TYPE_NEARBY = 5;
-    public static final int SELECTION_TYPE_EDIT = 6;
-    public static final int SELECTION_TYPE_MODIFY_SHARE = 7;
-    public static final int SELECTION_TYPE_CUSTOM_ACTION = 8;
-
-    /**
-     * This shim is provided only for testing. In production, clients will only ever use a
-     * {@link DefaultFrameworkStatsLogger}.
-     */
-    @VisibleForTesting
-    interface FrameworkStatsLogger {
-        /** Overload to use for logging {@code FrameworkStatsLog.SHARESHEET_STARTED}. */
-        void write(
-                int frameworkEventId,
-                int appEventId,
-                String packageName,
-                int instanceId,
-                String mimeType,
-                int numAppProvidedDirectTargets,
-                int numAppProvidedAppTargets,
-                boolean isWorkProfile,
-                int previewType,
-                int intentType,
-                int numCustomActions,
-                boolean modifyShareActionProvided);
-
-        /** Overload to use for logging {@code FrameworkStatsLog.RANKING_SELECTED}. */
-        void write(
-                int frameworkEventId,
-                int appEventId,
-                String packageName,
-                int instanceId,
-                int positionPicked,
-                boolean isPinned);
-    }
-
     private static final int SHARESHEET_INSTANCE_ID_MAX = (1 << 13);
 
     // A small per-notification ID, used for statsd logging.
@@ -95,12 +54,12 @@ public class EventLog {
     private final FrameworkStatsLogger mFrameworkStatsLogger;
     private final MetricsLogger mMetricsLogger;
 
-    public EventLog() {
+    public EventLogImpl() {
         this(new UiEventLoggerImpl(), new DefaultFrameworkStatsLogger(), new MetricsLogger());
     }
 
     @VisibleForTesting
-    EventLog(
+    EventLogImpl(
             UiEventLogger uiEventLogger,
             FrameworkStatsLogger frameworkLogger,
             MetricsLogger metricsLogger) {
@@ -110,6 +69,7 @@ public class EventLog {
     }
 
     /** Records metrics for the start time of the {@link ChooserActivity}. */
+    @Override
     public void logChooserActivityShown(
             boolean isWorkProfile, String targetMimeType, long systemCost) {
         mMetricsLogger.write(new LogMaker(MetricsEvent.ACTION_ACTIVITY_CHOOSER_SHOWN)
@@ -120,6 +80,7 @@ public class EventLog {
     }
 
     /** Logs a UiEventReported event for the system sharesheet completing initial start-up. */
+    @Override
     public void logShareStarted(
             String packageName,
             String mimeType,
@@ -149,6 +110,7 @@ public class EventLog {
      *
      * @param positionPicked index of the custom action within the list of custom actions.
      */
+    @Override
     public void logCustomActionSelected(int positionPicked) {
         mFrameworkStatsLogger.write(FrameworkStatsLog.RANKING_SELECTED,
                 /* event_id = 1 */
@@ -164,6 +126,7 @@ public class EventLog {
      * TODO: document parameters and/or consider breaking up by targetType so we don't have to
      * support an overly-generic signature.
      */
+    @Override
     public void logShareTargetSelected(
             int targetType,
             String packageName,
@@ -209,6 +172,7 @@ public class EventLog {
     }
 
     /** Log when direct share targets were received. */
+    @Override
     public void logDirectShareTargetReceived(int category, int latency) {
         mMetricsLogger.write(new LogMaker(category).setSubtype(latency));
     }
@@ -217,12 +181,14 @@ public class EventLog {
      * Log when we display a preview UI of the specified {@code previewType} as part of our
      * Sharesheet session.
      */
+    @Override
     public void logActionShareWithPreview(int previewType) {
         mMetricsLogger.write(
                 new LogMaker(MetricsEvent.ACTION_SHARE_WITH_PREVIEW).setSubtype(previewType));
     }
 
     /** Log when the user selects an action button with the specified {@code targetType}. */
+    @Override
     public void logActionSelected(int targetType) {
         if (targetType == SELECTION_TYPE_COPY) {
             LogMaker targetLogMaker = new LogMaker(
@@ -238,6 +204,7 @@ public class EventLog {
     }
 
     /** Log a warning that we couldn't display the content preview from the supplied {@code uri}. */
+    @Override
     public void logContentPreviewWarning(Uri uri) {
         // The ContentResolver already logs the exception. Log something more informative.
         Log.w(TAG, "Could not load (" + uri.toString() + ") thumbnail/name for preview. If "
@@ -248,11 +215,13 @@ public class EventLog {
     }
 
     /** Logs a UiEventReported event for the system sharesheet being triggered by the user. */
+    @Override
     public void logSharesheetTriggered() {
         log(SharesheetStandardEvent.SHARESHEET_TRIGGERED, getInstanceId());
     }
 
     /** Logs a UiEventReported event for the system sharesheet completing loading app targets. */
+    @Override
     public void logSharesheetAppLoadComplete() {
         log(SharesheetStandardEvent.SHARESHEET_APP_LOAD_COMPLETE, getInstanceId());
     }
@@ -260,6 +229,7 @@ public class EventLog {
     /**
      * Logs a UiEventReported event for the system sharesheet completing loading service targets.
      */
+    @Override
     public void logSharesheetDirectLoadComplete() {
         log(SharesheetStandardEvent.SHARESHEET_DIRECT_LOAD_COMPLETE, getInstanceId());
     }
@@ -267,6 +237,7 @@ public class EventLog {
     /**
      * Logs a UiEventReported event for the system sharesheet timing out loading service targets.
      */
+    @Override
     public void logSharesheetDirectLoadTimeout() {
         log(SharesheetStandardEvent.SHARESHEET_DIRECT_LOAD_TIMEOUT, getInstanceId());
     }
@@ -275,11 +246,13 @@ public class EventLog {
      * Logs a UiEventReported event for the system sharesheet switching
      * between work and main profile.
      */
+    @Override
     public void logSharesheetProfileChanged() {
         log(SharesheetStandardEvent.SHARESHEET_PROFILE_CHANGED, getInstanceId());
     }
 
     /** Logs a UiEventReported event for the system sharesheet getting expanded or collapsed. */
+    @Override
     public void logSharesheetExpansionChanged(boolean isCollapsed) {
         log(isCollapsed ? SharesheetStandardEvent.SHARESHEET_COLLAPSED :
                 SharesheetStandardEvent.SHARESHEET_EXPANDED, getInstanceId());
@@ -288,6 +261,7 @@ public class EventLog {
     /**
      * Logs a UiEventReported event for the system sharesheet app share ranking timing out.
      */
+    @Override
     public void logSharesheetAppShareRankingTimeout() {
         log(SharesheetStandardEvent.SHARESHEET_APP_SHARE_RANKING_TIMEOUT, getInstanceId());
     }
@@ -295,6 +269,7 @@ public class EventLog {
     /**
      * Logs a UiEventReported event for the system sharesheet when direct share row is empty.
      */
+    @Override
     public void logSharesheetEmptyDirectShareRow() {
         log(SharesheetStandardEvent.SHARESHEET_EMPTY_DIRECT_SHARE_ROW, getInstanceId());
     }
diff --git a/java/src/com/android/intentresolver/logging/FrameworkStatsLogger.kt b/java/src/com/android/intentresolver/logging/FrameworkStatsLogger.kt
new file mode 100644
index 00000000..e0682b9e
--- /dev/null
+++ b/java/src/com/android/intentresolver/logging/FrameworkStatsLogger.kt
@@ -0,0 +1,50 @@
+/*
+ * Copyright (C) 2023 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.android.intentresolver.logging
+
+import com.android.internal.util.FrameworkStatsLog
+
+/** A documenting annotation for FrameworkStatsLog methods and their associated UiEvents. */
+internal annotation class ForUiEvent(vararg val uiEventId: Int)
+
+/** Isolates the specific method signatures to use for each of the logged UiEvents. */
+internal interface FrameworkStatsLogger {
+    @ForUiEvent(FrameworkStatsLog.SHARESHEET_STARTED)
+    fun write(
+        frameworkEventId: Int,
+        appEventId: Int,
+        packageName: String?,
+        instanceId: Int,
+        mimeType: String?,
+        numAppProvidedDirectTargets: Int,
+        numAppProvidedAppTargets: Int,
+        isWorkProfile: Boolean,
+        previewType: Int,
+        intentType: Int,
+        numCustomActions: Int,
+        modifyShareActionProvided: Boolean
+    )
+
+    @ForUiEvent(FrameworkStatsLog.RANKING_SELECTED)
+    fun write(
+        frameworkEventId: Int,
+        appEventId: Int,
+        packageName: String?,
+        instanceId: Int,
+        positionPicked: Int,
+        isPinned: Boolean
+    )
+}
diff --git a/java/src/com/android/intentresolver/model/AbstractResolverComparator.java b/java/src/com/android/intentresolver/model/AbstractResolverComparator.java
index ff2d6a0f..932d8590 100644
--- a/java/src/com/android/intentresolver/model/AbstractResolverComparator.java
+++ b/java/src/com/android/intentresolver/model/AbstractResolverComparator.java
@@ -30,10 +30,11 @@ import android.os.Message;
 import android.os.UserHandle;
 import android.util.Log;
 
-import com.android.intentresolver.logging.EventLog;
 import com.android.intentresolver.ResolvedComponentInfo;
 import com.android.intentresolver.ResolverActivity;
+import com.android.intentresolver.ResolverListController;
 import com.android.intentresolver.chooser.TargetInfo;
+import com.android.intentresolver.logging.EventLog;
 
 import java.text.Collator;
 import java.util.ArrayList;
@@ -75,6 +76,7 @@ public abstract class AbstractResolverComparator implements Comparator<ResolvedC
     private EventLog mEventLog;
 
     protected final Handler mHandler = new Handler(Looper.getMainLooper()) {
+        @Override
         public void handleMessage(Message msg) {
             switch (msg.what) {
                 case RANKER_SERVICE_RESULT:
diff --git a/java/src/com/android/intentresolver/model/AppPredictionServiceResolverComparator.java b/java/src/com/android/intentresolver/model/AppPredictionServiceResolverComparator.java
index 621ae306..12fed698 100644
--- a/java/src/com/android/intentresolver/model/AppPredictionServiceResolverComparator.java
+++ b/java/src/com/android/intentresolver/model/AppPredictionServiceResolverComparator.java
@@ -31,9 +31,9 @@ import android.os.Message;
 import android.os.UserHandle;
 import android.util.Log;
 
-import com.android.intentresolver.logging.EventLog;
 import com.android.intentresolver.ResolvedComponentInfo;
 import com.android.intentresolver.chooser.TargetInfo;
+import com.android.intentresolver.logging.EventLog;
 
 import com.google.android.collect.Lists;
 
diff --git a/java/src/com/android/intentresolver/model/ResolverRankerServiceResolverComparator.java b/java/src/com/android/intentresolver/model/ResolverRankerServiceResolverComparator.java
index 7d473660..42b8a932 100644
--- a/java/src/com/android/intentresolver/model/ResolverRankerServiceResolverComparator.java
+++ b/java/src/com/android/intentresolver/model/ResolverRankerServiceResolverComparator.java
@@ -39,9 +39,9 @@ import android.service.resolver.ResolverRankerService;
 import android.service.resolver.ResolverTarget;
 import android.util.Log;
 
-import com.android.intentresolver.logging.EventLog;
 import com.android.intentresolver.ResolvedComponentInfo;
 import com.android.intentresolver.chooser.TargetInfo;
+import com.android.intentresolver.logging.EventLog;
 import com.android.internal.logging.MetricsLogger;
 import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
 
@@ -101,9 +101,9 @@ public class ResolverRankerServiceResolverComparator extends AbstractResolverCom
      *                        the userSpace provided by context.
      */
     public ResolverRankerServiceResolverComparator(Context launchedFromContext, Intent intent,
-            String referrerPackage, Runnable afterCompute,
-            EventLog eventLog, UserHandle targetUserSpace,
-            ComponentName promoteToFirst) {
+                                                   String referrerPackage, Runnable afterCompute,
+                                                   EventLog eventLog, UserHandle targetUserSpace,
+                                                   ComponentName promoteToFirst) {
         this(launchedFromContext, intent, referrerPackage, afterCompute, eventLog,
                 Lists.newArrayList(targetUserSpace), promoteToFirst);
     }
@@ -117,9 +117,8 @@ public class ResolverRankerServiceResolverComparator extends AbstractResolverCom
      *                            different from the userSpace provided by context.
      */
     public ResolverRankerServiceResolverComparator(Context launchedFromContext, Intent intent,
-            String referrerPackage, Runnable afterCompute,
-            EventLog eventLog, List<UserHandle> targetUserSpaceList,
-            @Nullable ComponentName promoteToFirst) {
+            String referrerPackage, Runnable afterCompute, EventLog eventLog,
+            List<UserHandle> targetUserSpaceList, @Nullable ComponentName promoteToFirst) {
         super(launchedFromContext, intent, targetUserSpaceList, promoteToFirst);
         mCollator = Collator.getInstance(
                 launchedFromContext.getResources().getConfiguration().locale);