diff options
author | Bjorn Bringert <bringert@android.com> | 2009-07-08 12:38:25 +0100 |
---|---|---|
committer | Bjorn Bringert <bringert@android.com> | 2009-07-08 12:38:25 +0100 |
commit | 0879cef6ddeda7e5e5409fe1277b9786aab32249 (patch) | |
tree | b0474e81773b65a57cba1473ef6fa58d60b8afb9 | |
parent | a206daaed1318005d1534cca2e4eeec56144d104 (diff) | |
download | Launcher-0879cef6ddeda7e5e5409fe1277b9786aab32249.tar.gz |
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.
-rw-r--r-- | src/com/android/launcher/Launcher.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/com/android/launcher/Launcher.java b/src/com/android/launcher/Launcher.java index cb511bc..8863bd9 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) { |