Merge "Some code cleanup"

This commit is contained in:
TreeHugger Robot
2017-09-07 19:55:46 +00:00
committed by Android (Google) Code Review
2 changed files with 32 additions and 75 deletions

View File

@@ -94,19 +94,6 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
} }
}; };
private View.OnClickListener mConditionClickListener = new View.OnClickListener() {
@Override
public void onClick(View v) {
Condition condition = (Condition) v.getTag();
//TODO: get rid of setTag/getTag
mMetricsFeatureProvider.action(mContext,
MetricsEvent.ACTION_SETTINGS_CONDITION_CLICK,
condition.getMetricsConstant());
condition.onPrimaryClick();
}
};
public DashboardAdapter(Context context, Bundle savedInstanceState, public DashboardAdapter(Context context, Bundle savedInstanceState,
List<Condition> conditions, SuggestionParser suggestionParser, List<Condition> conditions, SuggestionParser suggestionParser,
SuggestionDismissController.Callback callback) { SuggestionDismissController.Callback callback) {
@@ -413,7 +400,7 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
holder.itemView.setPadding(0, padding, 0, padding); holder.itemView.setPadding(0, padding, 0, padding);
holder.itemView.setOnClickListener(v -> { holder.itemView.setOnClickListener(v -> {
if (moreSuggestions ) { if (moreSuggestions) {
logSuggestions(); logSuggestions();
} else if (hasConditions) { } else if (hasConditions) {
mMetricsFeatureProvider.action(mContext, mMetricsFeatureProvider.action(mContext,
@@ -454,11 +441,6 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
} }
private void onBindTile(DashboardItemHolder holder, Tile tile) { private void onBindTile(DashboardItemHolder holder, Tile tile) {
if (tile.remoteViews != null) {
final ViewGroup itemView = (ViewGroup) holder.itemView;
itemView.removeAllViews();
itemView.addView(tile.remoteViews.apply(itemView.getContext(), itemView));
} else {
holder.icon.setImageDrawable(mCache.getIcon(tile.icon)); holder.icon.setImageDrawable(mCache.getIcon(tile.icon));
holder.title.setText(tile.title); holder.title.setText(tile.title);
if (!TextUtils.isEmpty(tile.summary)) { if (!TextUtils.isEmpty(tile.summary)) {
@@ -468,7 +450,6 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
holder.summary.setVisibility(View.GONE); holder.summary.setVisibility(View.GONE);
} }
} }
}
private void tintIcons(DashboardCategory category, List<Tile> suggestions) { private void tintIcons(DashboardCategory category, List<Tile> suggestions) {
if (!mDashboardFeatureProvider.shouldTintIcon()) { if (!mDashboardFeatureProvider.shouldTintIcon()) {

View File

@@ -171,30 +171,6 @@ public class DashboardData {
return POSITION_NOT_FOUND; return POSITION_NOT_FOUND;
} }
/**
* Get the count of suggestions to display
*
* The displayable count mainly depends on the {@link #mSuggestionConditionMode}
* and the size of suggestions list.
*
* When in default mode, displayable count couldn't be larger than
* {@link #DEFAULT_SUGGESTION_COUNT}.
*
* When in expanded mode, display all the suggestions.
*
* @return the count of suggestions to display
*/
public int getDisplayableSuggestionCount() {
final int suggestionSize = sizeOf(mSuggestions);
if (mSuggestionConditionMode == HEADER_MODE_COLLAPSED) {
return 0;
}
if (mSuggestionConditionMode == HEADER_MODE_DEFAULT) {
return Math.min(DEFAULT_SUGGESTION_COUNT, suggestionSize);
}
return suggestionSize;
}
/** /**
* Add item into list when {@paramref add} is true. * Add item into list when {@paramref add} is true.
* *
@@ -280,7 +256,7 @@ public class DashboardData {
if (conditions == null) { if (conditions == null) {
return null; return null;
} }
List<Condition> result = new ArrayList<Condition>(); List<Condition> result = new ArrayList<>();
final int size = conditions == null ? 0 : conditions.size(); final int size = conditions == null ? 0 : conditions.size();
for (int i = 0; i < size; i++) { for (int i = 0; i < size; i++) {
final Condition condition = conditions.get(i); final Condition condition = conditions.get(i);
@@ -482,7 +458,7 @@ public class DashboardData {
conditionCount = sizeOf(conditions); conditionCount = sizeOf(conditions);
this.hiddenSuggestionCount = hiddenSuggestionCount; this.hiddenSuggestionCount = hiddenSuggestionCount;
title = conditionCount > 0 ? conditions.get(0).getTitle() : null; title = conditionCount > 0 ? conditions.get(0).getTitle() : null;
conditionIcons = new ArrayList<Icon>(); conditionIcons = new ArrayList<>();
for (int i = 0; conditions != null && i < conditions.size(); i++) { for (int i = 0; conditions != null && i < conditions.size(); i++) {
final Condition condition = conditions.get(i); final Condition condition = conditions.get(i);
conditionIcons.add(condition.getIcon()); conditionIcons.add(condition.getIcon());