Merge "Use "play" icon for search dialog button in Browser"

This commit is contained in:
Bjorn Bringert
2010-03-03 14:01:24 -08:00
committed by Android (Google) Code Review
3 changed files with 6 additions and 12 deletions

View File

@@ -464,15 +464,14 @@ public class SearchDialog extends Dialog implements OnItemClickListener, OnItemS
}
}
/**
* Update the text in the search button. Note: This is deprecated functionality, for
* 1.0 compatibility only.
*/
private void updateSearchButton() {
private void updateSearchButton() {
String textLabel = null;
Drawable iconLabel = null;
int textId = mSearchable.getSearchButtonText();
if (textId != 0) {
if (isBrowserSearch()){
iconLabel = getContext().getResources()
.getDrawable(com.android.internal.R.drawable.ic_btn_search_play);
} else if (textId != 0) {
textLabel = mActivityContext.getResources().getString(textId);
} else {
iconLabel = getContext().getResources().
@@ -483,10 +482,6 @@ public class SearchDialog extends Dialog implements OnItemClickListener, OnItemS
}
private void updateSearchAppIcon() {
// In Donut, we special-case the case of the browser to hide the app icon as if it were
// global search, for extra space for url entry.
//
// TODO: Remove this special case once the issue has been reconciled in Eclair.
if (isBrowserSearch()) {
mAppIcon.setImageResource(0);
mAppIcon.setVisibility(View.GONE);
@@ -583,8 +578,7 @@ public class SearchDialog extends Dialog implements OnItemClickListener, OnItemS
}
/**
* Hack to determine whether this is the browser, so we can remove the browser icon
* to the left of the search field.
* Hack to determine whether this is the browser, so we can adjust the UI.
*/
private boolean isBrowserSearch() {
return mLaunchComponent.flattenToShortString().startsWith("com.android.browser/");

Binary file not shown.

After

Width:  |  Height:  |  Size: 960 B

Binary file not shown.

After

Width:  |  Height:  |  Size: 677 B