From 0879cef6ddeda7e5e5409fe1277b9786aab32249 Mon Sep 17 00:00:00 2001 From: Bjorn Bringert Date: Wed, 8 Jul 2009 12:38:25 +0100 Subject: [PATCH] Don't call SearchManager.isVisible() in Launcher. SearchManager.isVisible() is a hidden method that should not be used by applications. Also, SearchManager.stopSearch() already checks whether the search dialog is visible before tryinf to dimiss it, so the check in Launcher was redundant. --- src/com/android/launcher/Launcher.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/com/android/launcher/Launcher.java b/src/com/android/launcher/Launcher.java index cb511bc8c9..8863bd9353 100644 --- a/src/com/android/launcher/Launcher.java +++ b/src/com/android/launcher/Launcher.java @@ -1244,9 +1244,7 @@ public final class Launcher extends Activity implements View.OnClickListener, On void stopSearch() { // Close search dialog SearchManager searchManager = (SearchManager) getSystemService(Context.SEARCH_SERVICE); - if (searchManager.isVisible()) { - searchManager.stopSearch(); - } + searchManager.stopSearch(); // Restore search widget to its normal position Search searchWidget = mWorkspace.findSearchWidgetOnCurrentScreen(); if (searchWidget != null) {