From 03c8853c0902f5226af7fa36d7e2c7a45837efd1 Mon Sep 17 00:00:00 2001
From: Andrey Epin <ayepin@google.com>
Date: Tue, 24 Jan 2023 20:55:16 -0800
Subject: Fix pinning dialog for Chooser direct targets

Show target pinning dialog for direct share targets.

Fix: 266647478
Test: manual testing
Test: a new unit test
Change-Id: I3ae25d7ce9c6d4c00aada2063eb3f285f4d3ea26
---
 java/src/com/android/intentresolver/ChooserActivity.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'java/src')

diff --git a/java/src/com/android/intentresolver/ChooserActivity.java b/java/src/com/android/intentresolver/ChooserActivity.java
index dce8bde1..97459333 100644
--- a/java/src/com/android/intentresolver/ChooserActivity.java
+++ b/java/src/com/android/intentresolver/ChooserActivity.java
@@ -1535,9 +1535,9 @@ public class ChooserActivity extends ResolverActivity implements
                                 .getActiveListAdapter()
                                 .targetInfoForPosition(
                                         selectedPosition, /* filtered= */ true);
-                        // ItemViewHolder contents should always be "display resolve info"
-                        // targets, but check just to make sure.
-                        if (longPressedTargetInfo.isDisplayResolveInfo()) {
+                        // Only a direct share target or an app target is expected
+                        if (longPressedTargetInfo.isDisplayResolveInfo()
+                                || longPressedTargetInfo.isSelectableTargetInfo()) {
                             showTargetDetails(longPressedTargetInfo);
                         }
                     }
-- 
cgit v1.2.3-59-g8ed1b