From 11e237e6c6544254a374c7cfbbf5433dd99063f6 Mon Sep 17 00:00:00 2001 From: Rakesh Iyer Date: Tue, 8 Mar 2016 17:20:21 -0800 Subject: [PATCH] Fix issue with app launch from CarNavigationBar. Seems like mCurrentPackage name can get stale and the optimization to cache is isn't all that needed so remove it in the name of app launch reliability. Bug: 27505472 Change-Id: I641284c1392f8ff1f392f8dbf2c31177a64b4326 --- .../statusbar/car/CarNavigationBarController.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/car/CarNavigationBarController.java b/packages/SystemUI/src/com/android/systemui/statusbar/car/CarNavigationBarController.java index 8a93c5b0d1c29..bb43899c44e13 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/car/CarNavigationBarController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/car/CarNavigationBarController.java @@ -71,7 +71,6 @@ class CarNavigationBarController { private List mNavButtons = new ArrayList(); private int mCurrentFacetIndex; - private String mCurrentPackageName; private SparseBooleanArray mFacetHasMultipleAppsCache = new SparseBooleanArray(); public CarNavigationBarController(Context context, @@ -84,7 +83,6 @@ class CarNavigationBarController { } public void taskChanged(String packageName) { - mCurrentPackageName = packageName; // If the package name belongs to a filter, then highlight appropriate button in // the navigation bar. if (mFacetPackageMap.containsKey(packageName)) { @@ -298,12 +296,6 @@ class CarNavigationBarController { return; } - // Don't launch the lens picker if it's already running and the - // user clicks the same facet - if (packageName.equals(mCurrentPackageName) && index == mCurrentFacetIndex) { - return; - } - intent.putExtra(EXTRA_FACET_CATEGORIES, mFacetCategories.get(index)); intent.putExtra(EXTRA_FACET_PACKAGES, mFacetPackages.get(index)); // The facet is identified by the index in which it was added to the nav bar.