commit df3b80e4bfd09d4b44d6d5b22b588afa68cf8a49
author Alex Stetson <alexstetson@google.com> Wed Apr 29 21:48:28 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Wed Apr 29 21:48:28 2020 +0000
tree fbf4d0e28994dbc6ab6bb79785c9811f9f4c62ec
parent fe26a67e485dd70492a4fbbf54b263e0db0b8b7c
parent d1b08e7149f1d27a26c778ab48f8493db5401651

Fix incorrect result breadcrumbs am: d1b08e7149

Change-Id: I36b3a17487b15294ce6b3419fcd8eb8ab1c2d344

src/com/android/settings/intelligence/search/car/CarIntentSearchViewHolder.java

diff --git a/src/com/android/settings/intelligence/search/car/CarIntentSearchViewHolder.java b/src/com/android/settings/intelligence/search/car/CarIntentSearchViewHolder.java
index ba8a165..bc3ef60 100644
--- a/src/com/android/settings/intelligence/search/car/CarIntentSearchViewHolder.java
+++ b/src/com/android/settings/intelligence/search/car/CarIntentSearchViewHolder.java
@@ -19,6 +19,7 @@
 import android.content.Intent;
 import android.content.pm.PackageManager;
 import android.content.pm.ResolveInfo;
+import android.text.TextUtils;
 import android.util.Log;
 import android.view.View;
 
@@ -49,15 +50,7 @@
         } else {
             mIcon.setImageDrawable(result.icon);
         }
-        if (result.breadcrumbs != null && !result.breadcrumbs.isEmpty()) {
-            String breadcrumb = result.breadcrumbs.get(0);
-            int count = result.breadcrumbs.size();
-            for (int i = 1; i < count; i++) {
-                breadcrumb = mContext.getString(R.string.search_breadcrumb_connector,
-                        breadcrumb, result.breadcrumbs.get(i));
-            }
-            mSummary.setText(breadcrumb);
-        }
+        bindBreadcrumbView(result);
 
         itemView.setOnClickListener(v -> {
             fragment.onSearchResultClicked(/* resultViewHolder= */ this, result);
@@ -77,4 +70,23 @@
             }
         });
     }
+
+    private void bindBreadcrumbView(SearchResult result) {
+        if (result.breadcrumbs == null || result.breadcrumbs.isEmpty()) {
+            mSummary.setVisibility(View.GONE);
+            return;
+        }
+        String breadcrumb = result.breadcrumbs.get(0);
+        int count = result.breadcrumbs.size();
+        for (int i = 1; i < count; i++) {
+            breadcrumb = mContext.getString(R.string.search_breadcrumb_connector,
+                    breadcrumb, result.breadcrumbs.get(i));
+        }
+        if (breadcrumb == null || TextUtils.isEmpty(breadcrumb.trim())) {
+            mSummary.setVisibility(View.GONE);
+        } else {
+            mSummary.setText(breadcrumb);
+            mSummary.setVisibility(View.VISIBLE);
+        }
+    }
 }