diff options
| author | 2009-04-29 13:08:39 +0100 | |
|---|---|---|
| committer | 2009-04-29 20:49:56 +0100 | |
| commit | c1f40960644b461b108480b6446c89963441492d (patch) | |
| tree | b2f21eb1777a2e00f2f3edd8bfa0106e88dcb3aa | |
| parent | 80f3b97121477e9ed62193c038772b4d020534df (diff) | |
Remove menu from SearchDialog.
Before, pressing MENU with the SearchDialog open would show
a menu item that launched the search settings activity. Now that
the search settings are accessible from the Settings app, this is
no longer needed.
Fixes http://b/issue?id=1819469 and http://b/issue?id=1817823
| -rw-r--r-- | core/java/android/app/SearchDialog.java | 46 |
1 files changed, 4 insertions, 42 deletions
diff --git a/core/java/android/app/SearchDialog.java b/core/java/android/app/SearchDialog.java index d7ef1032cd2b..4fb17c7183c0 100644 --- a/core/java/android/app/SearchDialog.java +++ b/core/java/android/app/SearchDialog.java @@ -17,6 +17,7 @@ package android.app; import static android.app.SuggestionsAdapter.getColumnString; + import android.content.ActivityNotFoundException; import android.content.BroadcastReceiver; import android.content.ComponentName; @@ -25,8 +26,8 @@ import android.content.Intent; import android.content.IntentFilter; import android.content.pm.ActivityInfo; import android.content.pm.PackageManager; -import android.content.pm.PackageManager.NameNotFoundException; import android.content.pm.ResolveInfo; +import android.content.pm.PackageManager.NameNotFoundException; import android.content.res.Configuration; import android.content.res.Resources; import android.database.Cursor; @@ -44,7 +45,6 @@ import android.util.AttributeSet; import android.util.Log; import android.view.Gravity; import android.view.KeyEvent; -import android.view.Menu; import android.view.MotionEvent; import android.view.View; import android.view.ViewConfiguration; @@ -53,14 +53,14 @@ import android.view.Window; import android.view.WindowManager; import android.view.inputmethod.InputMethodManager; import android.widget.AdapterView; -import android.widget.AdapterView.OnItemClickListener; -import android.widget.AdapterView.OnItemSelectedListener; import android.widget.AutoCompleteTextView; import android.widget.Button; import android.widget.ImageButton; import android.widget.ImageView; import android.widget.ListView; import android.widget.TextView; +import android.widget.AdapterView.OnItemClickListener; +import android.widget.AdapterView.OnItemSelectedListener; import java.util.ArrayList; import java.util.WeakHashMap; @@ -602,44 +602,6 @@ public class SearchDialog extends Dialog implements OnItemClickListener, OnItemS mVoiceButton.setVisibility(visibility); } - /* - * Menu. - */ - - @Override - public boolean onCreateOptionsMenu(Menu menu) { - // Show search settings menu item if anyone handles the intent for it - Intent settingsIntent = new Intent(SearchManager.INTENT_ACTION_SEARCH_SETTINGS); - PackageManager pm = getContext().getPackageManager(); - ActivityInfo activityInfo = settingsIntent.resolveActivityInfo(pm, 0); - if (activityInfo != null) { - settingsIntent.setClassName(activityInfo.applicationInfo.packageName, - activityInfo.name); - String label = getActivityLabel(activityInfo); - menu.add(Menu.NONE, Menu.NONE, Menu.NONE, label) - .setIcon(android.R.drawable.ic_menu_preferences) - .setAlphabeticShortcut('P') - .setIntent(settingsIntent); - return true; - } - return super.onCreateOptionsMenu(menu); - } - - // TODO: shouldn't this be in PackageManager? - private String getActivityLabel(ActivityInfo activityInfo) { - PackageManager pm = getContext().getPackageManager(); - try { - int labelRes = activityInfo.labelRes; - if (labelRes == 0) { - return null; - } - Resources r = pm.getResourcesForApplication(activityInfo.applicationInfo); - return r.getString(labelRes); - } catch (NameNotFoundException ex) { - return null; - } - } - /** * Listeners of various types */ |