Fix lint errors.

- MenuItem.collapseActionView is only availble for API level >=14.
- Rename the dialog variable in UploadResultActivity so it doesn't
  conflict with the dialog variable in the callback.
This commit is contained in:
Jimmy Shih
2012-05-01 18:44:05 -07:00
parent f8cacb01c9
commit 81281e560e
3 changed files with 39 additions and 38 deletions
@@ -49,7 +49,7 @@ public class UploadResultActivity extends Activity {
private SendRequest sendRequest; private SendRequest sendRequest;
private Track track; private Track track;
private String shareUrl; private String shareUrl;
private Dialog dialog; private Dialog resultDialog;
@Override @Override
protected void onCreate(Bundle savedInstanceState) { protected void onCreate(Bundle savedInstanceState) {
@@ -169,8 +169,8 @@ public class UploadResultActivity extends Activity {
} }
}); });
} }
dialog = builder.create(); resultDialog = builder.create();
return dialog; return resultDialog;
} }
/** /**
@@ -192,6 +192,6 @@ public class UploadResultActivity extends Activity {
@VisibleForTesting @VisibleForTesting
Dialog getDialog() { Dialog getDialog() {
return dialog; return resultDialog;
} }
} }
@@ -37,7 +37,7 @@ import java.util.List;
/** /**
* API level 11 specific implementation of the {@link ApiAdapter}. * API level 11 specific implementation of the {@link ApiAdapter}.
* *
* @author Jimmy Shih * @author Jimmy Shih
*/ */
@TargetApi(11) @TargetApi(11)
@@ -54,12 +54,11 @@ public class Api11Adapter extends Api10Adapter {
} }
@Override @Override
public void configureListViewContextualMenu(final Activity activity, ListView listView, final int menuId, public void configureListViewContextualMenu(final Activity activity, ListView listView,
final int actionModeTitleId, final int menuId, final int actionModeTitleId,
final ContextualActionModeCallback contextualActionModeCallback) { final ContextualActionModeCallback contextualActionModeCallback) {
listView.setOnItemLongClickListener(new OnItemLongClickListener() { listView.setOnItemLongClickListener(new OnItemLongClickListener() {
ActionMode actionMode; ActionMode actionMode;
@Override @Override
public boolean onItemLongClick( public boolean onItemLongClick(
AdapterView<?> parent, View view, final int position, final long id) { AdapterView<?> parent, View view, final int position, final long id) {
@@ -72,18 +71,15 @@ public class Api11Adapter extends Api10Adapter {
mode.getMenuInflater().inflate(menuId, menu); mode.getMenuInflater().inflate(menuId, menu);
return true; return true;
} }
@Override @Override
public boolean onPrepareActionMode(ActionMode mode, Menu menu) { public boolean onPrepareActionMode(ActionMode mode, Menu menu) {
// Return false to indicate no change. // Return false to indicate no change.
return false; return false;
} }
@Override @Override
public void onDestroyActionMode(ActionMode mode) { public void onDestroyActionMode(ActionMode mode) {
actionMode = null; actionMode = null;
} }
@Override @Override
public boolean onActionItemClicked(ActionMode mode, MenuItem item) { public boolean onActionItemClicked(ActionMode mode, MenuItem item) {
mode.finish(); mode.finish();
@@ -99,37 +95,13 @@ public class Api11Adapter extends Api10Adapter {
} }
}); });
}; };
@Override @Override
public void configureSearchWidget(Activity activity, final MenuItem menuItem) { public void configureSearchWidget(Activity activity, final MenuItem menuItem) {
SearchManager searchManager = (SearchManager) activity.getSystemService(Context.SEARCH_SERVICE); SearchManager searchManager = (SearchManager) activity.getSystemService(Context.SEARCH_SERVICE);
SearchView searchView = (SearchView) menuItem.getActionView(); SearchView searchView = (SearchView) menuItem.getActionView();
searchView.setSearchableInfo(searchManager.getSearchableInfo(activity.getComponentName())); searchView.setSearchableInfo(searchManager.getSearchableInfo(activity.getComponentName()));
searchView.setQueryRefinementEnabled(true); searchView.setQueryRefinementEnabled(true);
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override
public boolean onQueryTextSubmit(String query) {
menuItem.collapseActionView();
return false;
}
@Override
public boolean onQueryTextChange(String newText) {
return false;
}
});
searchView.setOnSuggestionListener(new SearchView.OnSuggestionListener() {
@Override
public boolean onSuggestionSelect(int position) {
return false;
}
@Override
public boolean onSuggestionClick(int position) {
menuItem.collapseActionView();
return false;
}
});
} }
@Override @Override
@@ -20,10 +20,11 @@ import android.annotation.TargetApi;
import android.app.ActionBar; import android.app.ActionBar;
import android.app.Activity; import android.app.Activity;
import android.view.MenuItem; import android.view.MenuItem;
import android.widget.SearchView;
/** /**
* API level 14 specific implementation of the {@link ApiAdapter}. * API level 14 specific implementation of the {@link ApiAdapter}.
* *
* @author Jimmy Shih * @author Jimmy Shih
*/ */
@TargetApi(14) @TargetApi(14)
@@ -35,7 +36,35 @@ public class Api14Adapter extends Api11Adapter {
actionBar.setHomeButtonEnabled(true); actionBar.setHomeButtonEnabled(true);
actionBar.setDisplayHomeAsUpEnabled(true); actionBar.setDisplayHomeAsUpEnabled(true);
} }
@Override
public void configureSearchWidget(Activity activity, final MenuItem menuItem) {
super.configureSearchWidget(activity, menuItem);
SearchView searchView = (SearchView) menuItem.getActionView();
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override
public boolean onQueryTextSubmit(String query) {
menuItem.collapseActionView();
return false;
}
@Override
public boolean onQueryTextChange(String newText) {
return false;
}
});
searchView.setOnSuggestionListener(new SearchView.OnSuggestionListener() {
@Override
public boolean onSuggestionSelect(int position) {
return false;
}
@Override
public boolean onSuggestionClick(int position) {
menuItem.collapseActionView();
return false;
}
});
}
@Override @Override
public boolean handleSearchKey(MenuItem menuItem) { public boolean handleSearchKey(MenuItem menuItem) {
menuItem.expandActionView(); menuItem.expandActionView();