Remove Suggestion v1 code.
Change-Id: Ie6e9c8f2b1b5b609d44e287accb9fbbef9054d34 Fixes: 68719093 Test: robotests
This commit is contained in:
@@ -21,7 +21,6 @@ package com.android.settings.core;
|
||||
*/
|
||||
public class FeatureFlags {
|
||||
public static final String SEARCH_V2 = "settings_search_v2";
|
||||
public static final String SUGGESTIONS_V2 = "new_settings_suggestion";
|
||||
public static final String APP_INFO_V2 = "settings_app_info_v2";
|
||||
public static final String CONNECTED_DEVICE_V2 = "settings_connected_device_v2";
|
||||
public static final String BATTERY_SETTINGS_V2 = "settings_battery_v2";
|
||||
|
@@ -39,7 +39,6 @@ public class MetricsFeatureProvider {
|
||||
|
||||
protected void installLogWriters() {
|
||||
mLoggerWriters.add(new EventLogWriter());
|
||||
mLoggerWriters.add(new SettingSuggestionsLogWriter());
|
||||
}
|
||||
|
||||
public void visible(Context context, int source, int category) {
|
||||
|
@@ -1,91 +0,0 @@
|
||||
/*
|
||||
* Copyright (C) 2017 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package com.android.settings.core.instrumentation;
|
||||
|
||||
import android.content.Context;
|
||||
import android.util.Pair;
|
||||
|
||||
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
|
||||
import com.android.settings.dashboard.suggestions.EventStore;
|
||||
|
||||
/**
|
||||
* {@link LogWriter} that writes setting suggestion related logs.
|
||||
*/
|
||||
public class SettingSuggestionsLogWriter implements LogWriter {
|
||||
|
||||
private EventStore mEventStore;
|
||||
|
||||
@Override
|
||||
public void visible(Context context, int source, int category) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void hidden(Context context, int category) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void action(Context context, int category, Pair<Integer, Object>... taggedData) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void actionWithSource(Context context, int source, int category) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void action(int category, int value, Pair<Integer, Object>... taggedData) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void action(int category, boolean value, Pair<Integer, Object>... taggedData) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void action(Context context, int category, int value) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void action(Context context, int category, boolean value) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void action(Context context, int category, String pkg,
|
||||
Pair<Integer, Object>... taggedData) {
|
||||
if (mEventStore == null) {
|
||||
mEventStore = new EventStore(context);
|
||||
}
|
||||
switch (category) {
|
||||
case MetricsEvent.ACTION_SHOW_SETTINGS_SUGGESTION:
|
||||
mEventStore.writeEvent(pkg, EventStore.EVENT_SHOWN);
|
||||
break;
|
||||
case MetricsEvent.ACTION_SETTINGS_DISMISS_SUGGESTION:
|
||||
mEventStore.writeEvent(pkg, EventStore.EVENT_DISMISSED);
|
||||
break;
|
||||
case MetricsEvent.ACTION_SETTINGS_SUGGESTION:
|
||||
mEventStore.writeEvent(pkg, EventStore.EVENT_CLICKED);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public void count(Context context, String name, int value) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void histogram(Context context, String name, int bucket) {
|
||||
}
|
||||
|
||||
}
|
@@ -52,7 +52,6 @@ import com.android.settings.overlay.FeatureFactory;
|
||||
import com.android.settingslib.Utils;
|
||||
import com.android.settingslib.drawer.DashboardCategory;
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
import com.android.settingslib.suggestions.SuggestionParser;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
@@ -61,7 +60,6 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
implements SummaryLoader.SummaryConsumer {
|
||||
public static final String TAG = "DashboardAdapter";
|
||||
private static final String STATE_SUGGESTION_LIST = "suggestion_list";
|
||||
private static final String STATE_SUGGESTION_LIST_V2 = "suggestion_list_v2";
|
||||
private static final String STATE_CATEGORY_LIST = "category_list";
|
||||
private static final String STATE_SUGGESTIONS_SHOWN_LOGGED = "suggestions_shown_logged";
|
||||
|
||||
@@ -69,8 +67,6 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
static final String STATE_SUGGESTION_CONDITION_MODE = "suggestion_condition_mode";
|
||||
@VisibleForTesting
|
||||
static final int SUGGESTION_CONDITION_HEADER_POSITION = 0;
|
||||
@VisibleForTesting
|
||||
static final int MAX_SUGGESTION_TO_SHOW = 5;
|
||||
|
||||
private final IconCache mCache;
|
||||
private final Context mContext;
|
||||
@@ -80,7 +76,6 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
private final ArrayList<String> mSuggestionsShownLogged;
|
||||
private boolean mFirstFrameDrawn;
|
||||
private RecyclerView mRecyclerView;
|
||||
private SuggestionParser mSuggestionParser;
|
||||
private SuggestionAdapter mSuggestionAdapter;
|
||||
private SuggestionDismissController mSuggestionDismissHandler;
|
||||
private SuggestionDismissController.Callback mCallback;
|
||||
@@ -97,13 +92,10 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
};
|
||||
|
||||
public DashboardAdapter(Context context, Bundle savedInstanceState,
|
||||
List<Condition> conditions, SuggestionParser suggestionParser,
|
||||
SuggestionControllerMixin suggestionControllerMixin,
|
||||
List<Condition> conditions, SuggestionControllerMixin suggestionControllerMixin,
|
||||
SuggestionDismissController.Callback callback) {
|
||||
|
||||
// @deprecated In favor of suggestionsV2 below.
|
||||
List<Tile> suggestions = null;
|
||||
List<Suggestion> suggestionsV2 = null;
|
||||
List<Suggestion> suggestions = null;
|
||||
DashboardCategory category = null;
|
||||
int suggestionConditionMode = DashboardData.HEADER_MODE_DEFAULT;
|
||||
|
||||
@@ -113,14 +105,12 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
mMetricsFeatureProvider = factory.getMetricsFeatureProvider();
|
||||
mDashboardFeatureProvider = factory.getDashboardFeatureProvider(context);
|
||||
mCache = new IconCache(context);
|
||||
mSuggestionParser = suggestionParser;
|
||||
mCallback = callback;
|
||||
|
||||
setHasStableIds(true);
|
||||
|
||||
if (savedInstanceState != null) {
|
||||
suggestions = savedInstanceState.getParcelableArrayList(STATE_SUGGESTION_LIST);
|
||||
suggestionsV2 = savedInstanceState.getParcelableArrayList(STATE_SUGGESTION_LIST_V2);
|
||||
category = savedInstanceState.getParcelable(STATE_CATEGORY_LIST);
|
||||
suggestionConditionMode = savedInstanceState.getInt(
|
||||
STATE_SUGGESTION_CONDITION_MODE, suggestionConditionMode);
|
||||
@@ -133,37 +123,15 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
mDashboardData = new DashboardData.Builder()
|
||||
.setConditions(conditions)
|
||||
.setSuggestions(suggestions)
|
||||
.setSuggestionsV2(suggestionsV2)
|
||||
.setCategory(category)
|
||||
.setSuggestionConditionMode(suggestionConditionMode)
|
||||
.build();
|
||||
}
|
||||
|
||||
public List<Tile> getSuggestions() {
|
||||
return mDashboardData.getSuggestions();
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #setCategory(DashboardCategory)} and
|
||||
* {@link #setSuggestionsV2(List)}.
|
||||
*/
|
||||
@Deprecated
|
||||
public void setCategoriesAndSuggestions(DashboardCategory category, List<Tile> suggestions) {
|
||||
tintIcons(category, suggestions);
|
||||
|
||||
public void setSuggestions(List<Suggestion> data) {
|
||||
final DashboardData prevData = mDashboardData;
|
||||
mDashboardData = new DashboardData.Builder(prevData)
|
||||
.setSuggestions(suggestions.subList(0,
|
||||
Math.min(suggestions.size(), MAX_SUGGESTION_TO_SHOW)))
|
||||
.setCategory(category)
|
||||
.build();
|
||||
notifyDashboardDataChanged(prevData);
|
||||
}
|
||||
|
||||
public void setSuggestionsV2(List<Suggestion> data) {
|
||||
final DashboardData prevData = mDashboardData;
|
||||
mDashboardData = new DashboardData.Builder(prevData)
|
||||
.setSuggestionsV2(data)
|
||||
.setSuggestions(data)
|
||||
.build();
|
||||
notifyDashboardDataChanged(prevData);
|
||||
}
|
||||
@@ -187,30 +155,8 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
notifyDashboardDataChanged(prevData);
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #onSuggestionDismissed(Suggestion)}.
|
||||
*/
|
||||
@Deprecated
|
||||
public void onSuggestionDismissed(Tile suggestion) {
|
||||
final List<Tile> suggestions = mDashboardData.getSuggestions();
|
||||
if (suggestions == null || suggestions.isEmpty()) {
|
||||
return;
|
||||
}
|
||||
if (suggestions.size() == 1) {
|
||||
// The only suggestion is dismissed, and the the empty suggestion container will
|
||||
// remain as the dashboard item. Need to refresh the dashboard list.
|
||||
final DashboardData prevData = mDashboardData;
|
||||
mDashboardData = new DashboardData.Builder(prevData)
|
||||
.setSuggestions(null)
|
||||
.build();
|
||||
notifyDashboardDataChanged(prevData);
|
||||
} else {
|
||||
mSuggestionAdapter.removeSuggestion(suggestion);
|
||||
}
|
||||
}
|
||||
|
||||
public void onSuggestionDismissed(Suggestion suggestion) {
|
||||
final List<Suggestion> list = mDashboardData.getSuggestionsV2();
|
||||
final List<Suggestion> list = mDashboardData.getSuggestions();
|
||||
if (list == null || list.size() == 0) {
|
||||
return;
|
||||
}
|
||||
@@ -219,7 +165,7 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
// remain as the dashboard item. Need to refresh the dashboard list.
|
||||
final DashboardData prevData = mDashboardData;
|
||||
mDashboardData = new DashboardData.Builder(prevData)
|
||||
.setSuggestionsV2(null)
|
||||
.setSuggestions(null)
|
||||
.build();
|
||||
notifyDashboardDataChanged(prevData);
|
||||
} else {
|
||||
@@ -309,18 +255,10 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
return mDashboardData.getItemEntityById(itemId);
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #getSuggestionV2(int)}.
|
||||
*/
|
||||
@Deprecated
|
||||
public Tile getSuggestion(int position) {
|
||||
public Suggestion getSuggestion(int position) {
|
||||
return mSuggestionAdapter.getSuggestion(position);
|
||||
}
|
||||
|
||||
public Suggestion getSuggestionV2(int position) {
|
||||
return mSuggestionAdapter.getSuggestionsV2(position);
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
void notifyDashboardDataChanged(DashboardData prevData) {
|
||||
if (mFirstFrameDrawn && prevData != null) {
|
||||
@@ -422,26 +360,17 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
int position) {
|
||||
// If there is suggestions to show, it will be at position 0 as we don't show the suggestion
|
||||
// header anymore.
|
||||
final List<Suggestion> suggestionsV2 = mDashboardData.getSuggestionsV2();
|
||||
final List<Tile> suggestions = mDashboardData.getSuggestions();
|
||||
final List<Suggestion> suggestions = mDashboardData.getSuggestions();
|
||||
|
||||
boolean conditionOnly = true;
|
||||
if (position == SUGGESTION_CONDITION_HEADER_POSITION) {
|
||||
if (suggestions != null && suggestions.size() > 0) {
|
||||
if (suggestions != null && suggestions.size() > 0) {
|
||||
conditionOnly = false;
|
||||
mSuggestionAdapter = new SuggestionAdapter(mContext, mSuggestionControllerMixin,
|
||||
(List<Tile>) mDashboardData.getItemEntityByPosition(position),
|
||||
null, mSuggestionsShownLogged);
|
||||
mSuggestionDismissHandler = new SuggestionDismissController(mContext,
|
||||
holder.data, mSuggestionControllerMixin, mSuggestionParser, mCallback);
|
||||
holder.data.setAdapter(mSuggestionAdapter);
|
||||
} else if (suggestionsV2 != null && suggestionsV2.size() > 0) {
|
||||
conditionOnly = false;
|
||||
mSuggestionAdapter = new SuggestionAdapter(mContext, mSuggestionControllerMixin,
|
||||
null, (List<Suggestion>) mDashboardData.getItemEntityByPosition(position),
|
||||
(List<Suggestion>) mDashboardData.getItemEntityByPosition(position),
|
||||
mSuggestionsShownLogged);
|
||||
mSuggestionDismissHandler = new SuggestionDismissController(mContext,
|
||||
holder.data, mSuggestionControllerMixin, null /* parser */, mCallback);
|
||||
holder.data, mSuggestionControllerMixin, mCallback);
|
||||
holder.data.setAdapter(mSuggestionAdapter);
|
||||
}
|
||||
}
|
||||
@@ -494,14 +423,10 @@ public class DashboardAdapter extends RecyclerView.Adapter<DashboardAdapter.Dash
|
||||
|
||||
void onSaveInstanceState(Bundle outState) {
|
||||
final DashboardCategory category = mDashboardData.getCategory();
|
||||
final List<Tile> suggestions = mDashboardData.getSuggestions();
|
||||
final List<Suggestion> suggestionV2 = mDashboardData.getSuggestionsV2();
|
||||
final List<Suggestion> suggestions = mDashboardData.getSuggestions();
|
||||
if (suggestions != null) {
|
||||
outState.putParcelableArrayList(STATE_SUGGESTION_LIST, new ArrayList<>(suggestions));
|
||||
}
|
||||
if (suggestionV2 != null) {
|
||||
outState.putParcelableArrayList(STATE_SUGGESTION_LIST_V2,
|
||||
new ArrayList<>(suggestionV2));
|
||||
outState.putParcelableArrayList(STATE_SUGGESTION_LIST,
|
||||
new ArrayList<>(suggestions));
|
||||
}
|
||||
if (category != null) {
|
||||
outState.putParcelable(STATE_CATEGORY_LIST, category);
|
||||
|
@@ -69,16 +69,14 @@ public class DashboardData {
|
||||
private final List<Item> mItems;
|
||||
private final DashboardCategory mCategory;
|
||||
private final List<Condition> mConditions;
|
||||
private final List<Tile> mSuggestions;
|
||||
private final List<Suggestion> mSuggestionsV2;
|
||||
private final List<Suggestion> mSuggestions;
|
||||
@HeaderMode
|
||||
private final int mSuggestionConditionMode;
|
||||
|
||||
private DashboardData(Builder builder) {
|
||||
mCategory = builder.mCategory;
|
||||
mConditions = builder.mConditions;
|
||||
mSuggestions = builder.mSuggestions;
|
||||
mSuggestionsV2 = builder.mSuggestionsV2;
|
||||
mSuggestions = builder.mSuggestionsV2;
|
||||
mSuggestionConditionMode = builder.mSuggestionConditionMode;
|
||||
|
||||
mItems = new ArrayList<>();
|
||||
@@ -123,14 +121,10 @@ public class DashboardData {
|
||||
return mConditions;
|
||||
}
|
||||
|
||||
public List<Tile> getSuggestions() {
|
||||
public List<Suggestion> getSuggestions() {
|
||||
return mSuggestions;
|
||||
}
|
||||
|
||||
public List<Suggestion> getSuggestionsV2() {
|
||||
return mSuggestionsV2;
|
||||
}
|
||||
|
||||
public int getSuggestionConditionMode() {
|
||||
return mSuggestionConditionMode;
|
||||
}
|
||||
@@ -197,22 +191,15 @@ public class DashboardData {
|
||||
* and mIsShowingAll, mSuggestionConditionMode flag.
|
||||
*/
|
||||
private void buildItemsData() {
|
||||
final boolean useSuggestionV2 = mSuggestionsV2 != null;
|
||||
final boolean hasSuggestions = useSuggestionV2
|
||||
? sizeOf(mSuggestionsV2) > 0
|
||||
: sizeOf(mSuggestions) > 0;
|
||||
final boolean hasSuggestions = sizeOf(mSuggestions) > 0;
|
||||
final List<Condition> conditions = getConditionsToShow(mConditions);
|
||||
final boolean hasConditions = sizeOf(conditions) > 0;
|
||||
|
||||
final List<Tile> suggestions = getSuggestionsToShow(mSuggestions);
|
||||
final List<Suggestion> suggestionsV2 = getSuggestionsV2ToShow(mSuggestionsV2);
|
||||
final List<Suggestion> suggestions = getSuggestionsToShow(mSuggestions);
|
||||
|
||||
final int hiddenSuggestion;
|
||||
if (useSuggestionV2) {
|
||||
hiddenSuggestion = hasSuggestions ? sizeOf(mSuggestionsV2) - sizeOf(suggestionsV2) : 0;
|
||||
} else {
|
||||
hiddenSuggestion = hasSuggestions ? sizeOf(mSuggestions) - sizeOf(suggestions) : 0;
|
||||
}
|
||||
final int hiddenSuggestion = hasSuggestions
|
||||
? sizeOf(mSuggestions) - sizeOf(suggestions)
|
||||
: 0;
|
||||
|
||||
final boolean hasSuggestionAndCollapsed = hasSuggestions
|
||||
&& mSuggestionConditionMode == HEADER_MODE_COLLAPSED;
|
||||
@@ -231,15 +218,8 @@ public class DashboardData {
|
||||
R.layout.suggestion_condition_header,
|
||||
STABLE_ID_SUGGESTION_CONDITION_MIDDLE_HEADER, onlyHasConditionAndCollapsed);
|
||||
|
||||
if (useSuggestionV2) {
|
||||
addToItemList(suggestionsV2, R.layout.suggestion_condition_container,
|
||||
STABLE_ID_SUGGESTION_CONTAINER, sizeOf(suggestionsV2) > 0);
|
||||
} else {
|
||||
/* Suggestion container. This is the card view that contains the list of suggestions.
|
||||
* This will be added whenever the suggestion list is not empty */
|
||||
addToItemList(suggestions, R.layout.suggestion_condition_container,
|
||||
STABLE_ID_SUGGESTION_CONTAINER, sizeOf(suggestions) > 0);
|
||||
}
|
||||
addToItemList(suggestions, R.layout.suggestion_condition_container,
|
||||
STABLE_ID_SUGGESTION_CONTAINER, sizeOf(suggestions) > 0);
|
||||
|
||||
/* Second suggestion/condition header. This will be added when there is at least one
|
||||
* suggestion or condition that is not currently displayed, and the user can expand the
|
||||
@@ -296,22 +276,7 @@ public class DashboardData {
|
||||
return result;
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #getSuggestionsV2ToShow}.
|
||||
*/
|
||||
@Deprecated
|
||||
private List<Tile> getSuggestionsToShow(List<Tile> suggestions) {
|
||||
if (suggestions == null || mSuggestionConditionMode == HEADER_MODE_COLLAPSED) {
|
||||
return null;
|
||||
}
|
||||
if (mSuggestionConditionMode != HEADER_MODE_DEFAULT
|
||||
|| suggestions.size() <= DEFAULT_SUGGESTION_COUNT) {
|
||||
return suggestions;
|
||||
}
|
||||
return suggestions.subList(0, DEFAULT_SUGGESTION_COUNT);
|
||||
}
|
||||
|
||||
private List<Suggestion> getSuggestionsV2ToShow(List<Suggestion> suggestions) {
|
||||
private List<Suggestion> getSuggestionsToShow(List<Suggestion> suggestions) {
|
||||
if (suggestions == null || mSuggestionConditionMode == HEADER_MODE_COLLAPSED) {
|
||||
return null;
|
||||
}
|
||||
@@ -333,11 +298,6 @@ public class DashboardData {
|
||||
|
||||
private DashboardCategory mCategory;
|
||||
private List<Condition> mConditions;
|
||||
/**
|
||||
* @deprecated in favor of SuggestionList
|
||||
*/
|
||||
@Deprecated
|
||||
private List<Tile> mSuggestions;
|
||||
private List<Suggestion> mSuggestionsV2;
|
||||
|
||||
public Builder() {
|
||||
@@ -346,8 +306,7 @@ public class DashboardData {
|
||||
public Builder(DashboardData dashboardData) {
|
||||
mCategory = dashboardData.mCategory;
|
||||
mConditions = dashboardData.mConditions;
|
||||
mSuggestions = dashboardData.mSuggestions;
|
||||
mSuggestionsV2 = dashboardData.mSuggestionsV2;
|
||||
mSuggestionsV2 = dashboardData.mSuggestions;
|
||||
mSuggestionConditionMode = dashboardData.mSuggestionConditionMode;
|
||||
}
|
||||
|
||||
@@ -361,16 +320,7 @@ public class DashboardData {
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #setSuggestionsV2(List)})}
|
||||
*/
|
||||
@Deprecated
|
||||
public Builder setSuggestions(List<Tile> suggestions) {
|
||||
this.mSuggestions = suggestions;
|
||||
return this;
|
||||
}
|
||||
|
||||
public Builder setSuggestionsV2(List<Suggestion> suggestions) {
|
||||
public Builder setSuggestions(List<Suggestion> suggestions) {
|
||||
this.mSuggestionsV2 = suggestions;
|
||||
return this;
|
||||
}
|
||||
|
@@ -19,7 +19,6 @@ package com.android.settings.dashboard;
|
||||
import android.app.Activity;
|
||||
import android.app.LoaderManager;
|
||||
import android.content.Context;
|
||||
import android.os.AsyncTask;
|
||||
import android.os.Bundle;
|
||||
import android.os.Handler;
|
||||
import android.service.settings.suggestions.Suggestion;
|
||||
@@ -41,19 +40,14 @@ import com.android.settings.dashboard.conditional.FocusRecyclerView;
|
||||
import com.android.settings.dashboard.conditional.FocusRecyclerView.FocusListener;
|
||||
import com.android.settings.dashboard.suggestions.SuggestionControllerMixin;
|
||||
import com.android.settings.dashboard.suggestions.SuggestionDismissController;
|
||||
import com.android.settings.dashboard.suggestions.SuggestionFeatureProvider;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
import com.android.settings.widget.ActionBarShadowController;
|
||||
import com.android.settingslib.drawer.CategoryKey;
|
||||
import com.android.settingslib.drawer.DashboardCategory;
|
||||
import com.android.settingslib.drawer.SettingsDrawerActivity;
|
||||
import com.android.settingslib.drawer.SettingsDrawerActivity.CategoryListener;
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
import com.android.settingslib.suggestions.SuggestionList;
|
||||
import com.android.settingslib.suggestions.SuggestionParser;
|
||||
import com.android.settingslib.utils.ThreadUtils;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
|
||||
public class DashboardSummary extends InstrumentedFragment
|
||||
@@ -73,11 +67,9 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
private DashboardAdapter mAdapter;
|
||||
private SummaryLoader mSummaryLoader;
|
||||
private ConditionManager mConditionManager;
|
||||
private SuggestionParser mSuggestionParser;
|
||||
private LinearLayoutManager mLayoutManager;
|
||||
private SuggestionControllerMixin mSuggestionControllerMixin;
|
||||
private DashboardFeatureProvider mDashboardFeatureProvider;
|
||||
private SuggestionFeatureProvider mSuggestionFeatureProvider;
|
||||
private boolean isOnCategoriesChangedCalled;
|
||||
private boolean mOnConditionsChangedCalled;
|
||||
|
||||
@@ -92,13 +84,9 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
@Override
|
||||
public void onAttach(Context context) {
|
||||
super.onAttach(context);
|
||||
mSuggestionFeatureProvider = FeatureFactory.getFactory(context)
|
||||
.getSuggestionFeatureProvider(context);
|
||||
if (mSuggestionFeatureProvider.isSuggestionV2Enabled(context)) {
|
||||
Log.d(TAG, "Suggestion v2 is enabled, creating SuggestionControllerMixin");
|
||||
mSuggestionControllerMixin = new SuggestionControllerMixin(context, this /* host */,
|
||||
getLifecycle());
|
||||
}
|
||||
Log.d(TAG, "Creating SuggestionControllerMixin");
|
||||
mSuggestionControllerMixin = new SuggestionControllerMixin(context, this /* host */,
|
||||
getLifecycle());
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -121,10 +109,6 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
|
||||
mConditionManager = ConditionManager.get(activity, false);
|
||||
getLifecycle().addObserver(mConditionManager);
|
||||
if (mSuggestionFeatureProvider.isSuggestionEnabled(activity)) {
|
||||
mSuggestionParser = new SuggestionParser(activity,
|
||||
mSuggestionFeatureProvider.getSharedPrefs(activity), R.xml.suggestion_ordering);
|
||||
}
|
||||
if (DEBUG_TIMING) {
|
||||
Log.d(TAG, "onCreate took " + (System.currentTimeMillis() - startTime) + " ms");
|
||||
}
|
||||
@@ -211,8 +195,7 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
mDashboard.setHasFixedSize(true);
|
||||
mDashboard.setListener(this);
|
||||
mAdapter = new DashboardAdapter(getContext(), bundle, mConditionManager.getConditions(),
|
||||
mSuggestionParser, mSuggestionControllerMixin,
|
||||
this /* SuggestionDismissController.Callback */);
|
||||
mSuggestionControllerMixin, this /* SuggestionDismissController.Callback */);
|
||||
mDashboard.setAdapter(mAdapter);
|
||||
mDashboard.setItemAnimator(new DashboardItemAnimator());
|
||||
mSummaryLoader.setSummaryConsumer(mAdapter);
|
||||
@@ -228,16 +211,7 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
|
||||
@VisibleForTesting
|
||||
void rebuildUI() {
|
||||
if (!mSuggestionFeatureProvider.isSuggestionEnabled(getContext())) {
|
||||
Log.d(TAG, "Suggestion v1 feature is disabled, skipping suggestion v1");
|
||||
ThreadUtils.postOnBackgroundThread(() -> updateCategory());
|
||||
} else {
|
||||
new SuggestionLoader().execute();
|
||||
// Set categories on their own if loading suggestions takes too long.
|
||||
mHandler.postDelayed(() -> {
|
||||
updateCategory();
|
||||
}, MAX_WAIT_MILLIS);
|
||||
}
|
||||
ThreadUtils.postOnBackgroundThread(() -> updateCategory());
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -274,19 +248,9 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
|
||||
@Override
|
||||
public Suggestion getSuggestionAt(int position) {
|
||||
return mAdapter.getSuggestionV2(position);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Tile getSuggestionForPosition(int position) {
|
||||
return mAdapter.getSuggestion(position);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onSuggestionDismissed(Tile suggestion) {
|
||||
mAdapter.onSuggestionDismissed(suggestion);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onSuggestionDismissed(Suggestion suggestion) {
|
||||
mAdapter.onSuggestionDismissed(suggestion);
|
||||
@@ -295,7 +259,7 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
@Override
|
||||
public void onSuggestionReady(List<Suggestion> suggestions) {
|
||||
mStagingSuggestions = suggestions;
|
||||
mAdapter.setSuggestionsV2(suggestions);
|
||||
mAdapter.setSuggestions(suggestions);
|
||||
if (mStagingCategory != null) {
|
||||
Log.d(TAG, "Category has loaded, setting category from suggestionReady");
|
||||
mHandler.removeCallbacksAndMessages(null);
|
||||
@@ -303,49 +267,6 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #mSuggestionControllerMixin}.
|
||||
*/
|
||||
@Deprecated
|
||||
private class SuggestionLoader extends AsyncTask<Void, Void, List<Tile>> {
|
||||
@Override
|
||||
protected List<Tile> doInBackground(Void... params) {
|
||||
final Context context = getContext();
|
||||
boolean isSmartSuggestionEnabled =
|
||||
mSuggestionFeatureProvider.isSmartSuggestionEnabled(context);
|
||||
final SuggestionList sl = mSuggestionParser.getSuggestions(isSmartSuggestionEnabled);
|
||||
final List<Tile> suggestions = sl.getSuggestions();
|
||||
|
||||
if (isSmartSuggestionEnabled) {
|
||||
List<String> suggestionIds = new ArrayList<>(suggestions.size());
|
||||
for (Tile suggestion : suggestions) {
|
||||
suggestionIds.add(mSuggestionFeatureProvider.getSuggestionIdentifier(
|
||||
context, suggestion));
|
||||
}
|
||||
// TODO: create a Suggestion class to maintain the id and other info
|
||||
mSuggestionFeatureProvider.rankSuggestions(suggestions, suggestionIds);
|
||||
}
|
||||
for (int i = 0; i < suggestions.size(); i++) {
|
||||
Tile suggestion = suggestions.get(i);
|
||||
if (mSuggestionFeatureProvider.isSuggestionComplete(context,
|
||||
suggestion.intent.getComponent())) {
|
||||
suggestions.remove(i--);
|
||||
}
|
||||
}
|
||||
if (sl.isExclusiveSuggestionCategory()) {
|
||||
mSuggestionFeatureProvider.filterExclusiveSuggestions(suggestions);
|
||||
}
|
||||
return suggestions;
|
||||
}
|
||||
|
||||
@Override
|
||||
protected void onPostExecute(List<Tile> tiles) {
|
||||
// tell handler that suggestions were loaded quickly enough
|
||||
mHandler.removeCallbacksAndMessages(null);
|
||||
updateCategoryAndSuggestion(tiles);
|
||||
}
|
||||
}
|
||||
|
||||
@WorkerThread
|
||||
void updateCategory() {
|
||||
final DashboardCategory category = mDashboardFeatureProvider.getTilesForCategory(
|
||||
@@ -356,7 +277,7 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
Log.d(TAG, "Suggestion has loaded, setting suggestion/category");
|
||||
ThreadUtils.postOnMainThread(() -> {
|
||||
if (mStagingSuggestions != null) {
|
||||
mAdapter.setSuggestionsV2(mStagingSuggestions);
|
||||
mAdapter.setSuggestions(mStagingSuggestions);
|
||||
}
|
||||
mAdapter.setCategory(mStagingCategory);
|
||||
});
|
||||
@@ -365,26 +286,4 @@ public class DashboardSummary extends InstrumentedFragment
|
||||
mHandler.postDelayed(() -> mAdapter.setCategory(mStagingCategory), MAX_WAIT_MILLIS);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #mSuggestionControllerMixin}.
|
||||
*/
|
||||
@Deprecated
|
||||
@VisibleForTesting
|
||||
void updateCategoryAndSuggestion(List<Tile> suggestions) {
|
||||
final Activity activity = getActivity();
|
||||
if (activity == null) {
|
||||
return;
|
||||
}
|
||||
|
||||
final DashboardCategory category = mDashboardFeatureProvider.getTilesForCategory(
|
||||
CategoryKey.CATEGORY_HOMEPAGE);
|
||||
mSummaryLoader.updateSummaryToCache(category);
|
||||
if (suggestions != null) {
|
||||
mAdapter.setCategoriesAndSuggestions(category, suggestions);
|
||||
} else {
|
||||
mAdapter.setCategory(category);
|
||||
}
|
||||
}
|
||||
|
||||
}
|
||||
|
@@ -1,102 +0,0 @@
|
||||
/*
|
||||
* Copyright (C) 2017 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License
|
||||
*/
|
||||
package com.android.settings.dashboard.suggestions;
|
||||
|
||||
import android.content.Context;
|
||||
import android.content.SharedPreferences;
|
||||
import android.util.Log;
|
||||
|
||||
import java.util.Arrays;
|
||||
import java.util.HashSet;
|
||||
import java.util.Set;
|
||||
|
||||
/**
|
||||
* Stores suggestion related statistics.
|
||||
*/
|
||||
public class EventStore {
|
||||
|
||||
public static final String TAG = "SuggestionEventStore";
|
||||
|
||||
public static final String EVENT_SHOWN = "shown";
|
||||
public static final String EVENT_DISMISSED = "dismissed";
|
||||
public static final String EVENT_CLICKED = "clicked";
|
||||
public static final String METRIC_LAST_EVENT_TIME = "last_event_time";
|
||||
public static final String METRIC_COUNT = "count";
|
||||
|
||||
private static final Set<String> EVENTS = new HashSet<String>(
|
||||
Arrays.asList(new String[] {EVENT_SHOWN, EVENT_DISMISSED, EVENT_CLICKED}));
|
||||
private static final Set<String> METRICS = new HashSet<String>(
|
||||
Arrays.asList(new String[] {METRIC_LAST_EVENT_TIME, METRIC_COUNT}));
|
||||
|
||||
private final SharedPreferences mSharedPrefs;
|
||||
|
||||
public EventStore(Context context) {
|
||||
mSharedPrefs = context.getSharedPreferences(TAG, Context.MODE_PRIVATE);
|
||||
}
|
||||
|
||||
/**
|
||||
* Writes individual log events.
|
||||
* @param pkgName: Package for which this event is reported.
|
||||
* @param eventType: Type of event (one of {@link #EVENTS}).
|
||||
*/
|
||||
public void writeEvent(String pkgName, String eventType) {
|
||||
if (!EVENTS.contains(eventType)) {
|
||||
Log.w(TAG, "Reported event type " + eventType + " is not a valid type!");
|
||||
return;
|
||||
}
|
||||
final String lastTimePrefKey = getPrefKey(pkgName, eventType, METRIC_LAST_EVENT_TIME);
|
||||
final String countPrefKey = getPrefKey(pkgName, eventType, METRIC_COUNT);
|
||||
writePref(lastTimePrefKey, System.currentTimeMillis());
|
||||
writePref(countPrefKey, readPref(countPrefKey, (long) 0) + 1);
|
||||
}
|
||||
|
||||
/**
|
||||
* Reads metric of the the reported events (e.g., counts).
|
||||
* @param pkgName: Package for which this metric is queried.
|
||||
* @param eventType: Type of event (one of {@link #EVENTS}).
|
||||
* @param metricType: Type of the queried metric (one of {@link #METRICS}).
|
||||
* @return the corresponding metric.
|
||||
*/
|
||||
public long readMetric(String pkgName, String eventType, String metricType) {
|
||||
if (!EVENTS.contains(eventType)) {
|
||||
Log.w(TAG, "Reported event type " + eventType + " is not a valid event!");
|
||||
return 0;
|
||||
} else if (!METRICS.contains(metricType)) {
|
||||
Log.w(TAG, "Required stat type + " + metricType + " is not a valid stat!");
|
||||
return 0;
|
||||
}
|
||||
return readPref(getPrefKey(pkgName, eventType, metricType), (long) 0);
|
||||
}
|
||||
|
||||
private void writePref(String prefKey, long value) {
|
||||
mSharedPrefs.edit().putLong(prefKey, value).apply();
|
||||
}
|
||||
|
||||
private long readPref(String prefKey, Long defaultValue) {
|
||||
return mSharedPrefs.getLong(prefKey, defaultValue);
|
||||
}
|
||||
|
||||
private String getPrefKey(String pkgName, String eventType, String statType) {
|
||||
return new StringBuilder()
|
||||
.append("setting_suggestion_")
|
||||
.append(pkgName)
|
||||
.append("_")
|
||||
.append(eventType)
|
||||
.append("_")
|
||||
.append(statType)
|
||||
.toString();
|
||||
}
|
||||
}
|
@@ -27,12 +27,10 @@ import android.view.ViewGroup;
|
||||
|
||||
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.SettingsActivity;
|
||||
import com.android.settings.core.instrumentation.MetricsFeatureProvider;
|
||||
import com.android.settings.dashboard.DashboardAdapter.DashboardItemHolder;
|
||||
import com.android.settings.dashboard.DashboardAdapter.IconCache;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
|
||||
import java.util.List;
|
||||
import java.util.Objects;
|
||||
@@ -42,26 +40,20 @@ public class SuggestionAdapter extends RecyclerView.Adapter<DashboardItemHolder>
|
||||
|
||||
private final Context mContext;
|
||||
private final MetricsFeatureProvider mMetricsFeatureProvider;
|
||||
private final SuggestionFeatureProvider mSuggestionFeatureProvider;
|
||||
@Deprecated // in favor of mSuggestionsV2
|
||||
private final List<Tile> mSuggestions;
|
||||
private final List<Suggestion> mSuggestionsV2;
|
||||
private final List<Suggestion> mSuggestions;
|
||||
private final IconCache mCache;
|
||||
private final List<String> mSuggestionsShownLogged;
|
||||
private final SuggestionControllerMixin mSuggestionControllerMixin;
|
||||
|
||||
public SuggestionAdapter(Context context, SuggestionControllerMixin suggestionControllerMixin,
|
||||
List<Tile> suggestions, List<Suggestion> suggestionsV2,
|
||||
List<String> suggestionsShownLogged) {
|
||||
List<Suggestion> suggestions, List<String> suggestionsShownLogged) {
|
||||
mContext = context;
|
||||
mSuggestionControllerMixin = suggestionControllerMixin;
|
||||
mSuggestions = suggestions;
|
||||
mSuggestionsV2 = suggestionsV2;
|
||||
mSuggestionsShownLogged = suggestionsShownLogged;
|
||||
mCache = new IconCache(context);
|
||||
final FeatureFactory factory = FeatureFactory.getFactory(context);
|
||||
mMetricsFeatureProvider = factory.getMetricsFeatureProvider();
|
||||
mSuggestionFeatureProvider = factory.getSuggestionFeatureProvider(context);
|
||||
|
||||
setHasStableIds(true);
|
||||
}
|
||||
@@ -74,15 +66,11 @@ public class SuggestionAdapter extends RecyclerView.Adapter<DashboardItemHolder>
|
||||
|
||||
@Override
|
||||
public void onBindViewHolder(DashboardItemHolder holder, int position) {
|
||||
if (mSuggestions != null) {
|
||||
bindSuggestionTile(holder, position);
|
||||
} else {
|
||||
bindSuggestion(holder, position);
|
||||
}
|
||||
bindSuggestion(holder, position);
|
||||
}
|
||||
|
||||
private void bindSuggestion(DashboardItemHolder holder, int position) {
|
||||
final Suggestion suggestion = mSuggestionsV2.get(position);
|
||||
final Suggestion suggestion = mSuggestions.get(position);
|
||||
final String id = suggestion.getId();
|
||||
if (!mSuggestionsShownLogged.contains(id)) {
|
||||
mMetricsFeatureProvider.action(
|
||||
@@ -101,7 +89,7 @@ public class SuggestionAdapter extends RecyclerView.Adapter<DashboardItemHolder>
|
||||
}
|
||||
final View divider = holder.itemView.findViewById(R.id.divider);
|
||||
if (divider != null) {
|
||||
divider.setVisibility(position < mSuggestionsV2.size() - 1 ? View.VISIBLE : View.GONE);
|
||||
divider.setVisibility(position < mSuggestions.size() - 1 ? View.VISIBLE : View.GONE);
|
||||
}
|
||||
View clickHandler = holder.itemView;
|
||||
// If a view with @android:id/primary is defined, use that as the click handler
|
||||
@@ -121,110 +109,32 @@ public class SuggestionAdapter extends RecyclerView.Adapter<DashboardItemHolder>
|
||||
});
|
||||
}
|
||||
|
||||
/**
|
||||
* @deprecated in favor {@link #bindSuggestion(DashboardItemHolder, int)}.
|
||||
*/
|
||||
@Deprecated
|
||||
private void bindSuggestionTile(DashboardItemHolder holder, int position) {
|
||||
final Tile suggestion = (Tile) mSuggestions.get(position);
|
||||
final String suggestionId = mSuggestionFeatureProvider.getSuggestionIdentifier(
|
||||
mContext, suggestion);
|
||||
// This is for cases when a suggestion is dismissed and the next one comes to view
|
||||
if (!mSuggestionsShownLogged.contains(suggestionId)) {
|
||||
mMetricsFeatureProvider.action(
|
||||
mContext, MetricsEvent.ACTION_SHOW_SETTINGS_SUGGESTION, suggestionId,
|
||||
mSuggestionFeatureProvider.getLoggingTaggedData(mContext));
|
||||
mSuggestionsShownLogged.add(suggestionId);
|
||||
}
|
||||
if (suggestion.remoteViews != null) {
|
||||
final ViewGroup itemView = (ViewGroup) holder.itemView;
|
||||
itemView.removeAllViews();
|
||||
itemView.addView(suggestion.remoteViews.apply(itemView.getContext(), itemView));
|
||||
} else {
|
||||
holder.icon.setImageDrawable(mCache.getIcon(suggestion.icon));
|
||||
holder.title.setText(suggestion.title);
|
||||
if (!TextUtils.isEmpty(suggestion.summary)) {
|
||||
holder.summary.setText(suggestion.summary);
|
||||
holder.summary.setVisibility(View.VISIBLE);
|
||||
} else {
|
||||
holder.summary.setVisibility(View.GONE);
|
||||
}
|
||||
}
|
||||
final View divider = holder.itemView.findViewById(R.id.divider);
|
||||
if (divider != null) {
|
||||
divider.setVisibility(position < mSuggestions.size() - 1 ? View.VISIBLE : View.GONE);
|
||||
}
|
||||
View clickHandler = holder.itemView;
|
||||
// If a view with @android:id/primary is defined, use that as the click handler
|
||||
// instead.
|
||||
final View primaryAction = holder.itemView.findViewById(android.R.id.primary);
|
||||
if (primaryAction != null) {
|
||||
clickHandler = primaryAction;
|
||||
}
|
||||
|
||||
clickHandler.setOnClickListener(v -> {
|
||||
mMetricsFeatureProvider.action(mContext,
|
||||
MetricsEvent.ACTION_SETTINGS_SUGGESTION, suggestionId,
|
||||
mSuggestionFeatureProvider.getLoggingTaggedData(mContext));
|
||||
((SettingsActivity) mContext).startSuggestion(suggestion.intent);
|
||||
});
|
||||
}
|
||||
|
||||
@Override
|
||||
public long getItemId(int position) {
|
||||
if (mSuggestions != null) {
|
||||
return Objects.hash(mSuggestions.get(position).title);
|
||||
} else {
|
||||
return Objects.hash(mSuggestionsV2.get(position).getId());
|
||||
}
|
||||
return Objects.hash(mSuggestions.get(position).getId());
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getItemViewType(int position) {
|
||||
if (mSuggestions != null) {
|
||||
Tile suggestion = getSuggestion(position);
|
||||
|
||||
return suggestion.remoteViews != null
|
||||
? R.layout.suggestion_tile_remote_container
|
||||
: R.layout.suggestion_tile;
|
||||
final Suggestion suggestion = getSuggestion(position);
|
||||
if ((suggestion.getFlags() & Suggestion.FLAG_HAS_BUTTON) != 0) {
|
||||
return R.layout.suggestion_tile_with_button;
|
||||
} else {
|
||||
final Suggestion suggestion = getSuggestionsV2(position);
|
||||
if ((suggestion.getFlags() & Suggestion.FLAG_HAS_BUTTON) != 0) {
|
||||
return R.layout.suggestion_tile_with_button;
|
||||
} else {
|
||||
return R.layout.suggestion_tile;
|
||||
}
|
||||
return R.layout.suggestion_tile;
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getItemCount() {
|
||||
if (mSuggestions != null) {
|
||||
return mSuggestions.size();
|
||||
} else {
|
||||
return mSuggestionsV2.size();
|
||||
}
|
||||
return mSuggestions.size();
|
||||
}
|
||||
|
||||
public Tile getSuggestion(int position) {
|
||||
public Suggestion getSuggestion(int position) {
|
||||
final long itemId = getItemId(position);
|
||||
if (mSuggestions == null) {
|
||||
return null;
|
||||
}
|
||||
for (Tile tile : mSuggestions) {
|
||||
if (Objects.hash(tile.title) == itemId) {
|
||||
return tile;
|
||||
}
|
||||
}
|
||||
return null;
|
||||
}
|
||||
|
||||
public Suggestion getSuggestionsV2(int position) {
|
||||
final long itemId = getItemId(position);
|
||||
if (mSuggestionsV2 == null) {
|
||||
return null;
|
||||
}
|
||||
for (Suggestion suggestion : mSuggestionsV2) {
|
||||
for (Suggestion suggestion : mSuggestions) {
|
||||
if (Objects.hash(suggestion.getId()) == itemId) {
|
||||
return suggestion;
|
||||
}
|
||||
@@ -232,13 +142,8 @@ public class SuggestionAdapter extends RecyclerView.Adapter<DashboardItemHolder>
|
||||
return null;
|
||||
}
|
||||
|
||||
public void removeSuggestion(Tile suggestion) {
|
||||
public void removeSuggestion(Suggestion suggestion) {
|
||||
mSuggestions.remove(suggestion);
|
||||
notifyDataSetChanged();
|
||||
}
|
||||
|
||||
public void removeSuggestion(Suggestion suggestion) {
|
||||
mSuggestionsV2.remove(suggestion);
|
||||
notifyDataSetChanged();
|
||||
}
|
||||
}
|
||||
|
@@ -23,27 +23,10 @@ import android.support.v7.widget.helper.ItemTouchHelper;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.overlay.FeatureFactory;
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
import com.android.settingslib.suggestions.SuggestionParser;
|
||||
|
||||
public class SuggestionDismissController extends ItemTouchHelper.SimpleCallback {
|
||||
|
||||
public interface Callback {
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #getSuggestionAt(int)}
|
||||
* Returns suggestion tile data from the callback
|
||||
*/
|
||||
@Deprecated
|
||||
Tile getSuggestionForPosition(int position);
|
||||
|
||||
/**
|
||||
* @deprecated in favor of {@link #onSuggestionDismissed(Suggestion)}
|
||||
* Called when a suggestion is dismissed.
|
||||
*/
|
||||
@Deprecated
|
||||
void onSuggestionDismissed(Tile suggestion);
|
||||
|
||||
/**
|
||||
* Returns suggestion tile data from the callback
|
||||
*/
|
||||
@@ -58,20 +41,13 @@ public class SuggestionDismissController extends ItemTouchHelper.SimpleCallback
|
||||
private final Context mContext;
|
||||
private final SuggestionFeatureProvider mSuggestionFeatureProvider;
|
||||
private final SuggestionControllerMixin mSuggestionMixin;
|
||||
|
||||
/**
|
||||
* @deprecated in favor of the new Suggestion backend.
|
||||
*/
|
||||
@Deprecated
|
||||
private final SuggestionParser mSuggestionParser;
|
||||
private final Callback mCallback;
|
||||
|
||||
public SuggestionDismissController(Context context, RecyclerView recyclerView,
|
||||
SuggestionControllerMixin suggestionMixin, SuggestionParser parser, Callback callback) {
|
||||
SuggestionControllerMixin suggestionMixin, Callback callback) {
|
||||
super(0, ItemTouchHelper.START | ItemTouchHelper.END);
|
||||
mSuggestionMixin = suggestionMixin;
|
||||
mContext = context;
|
||||
mSuggestionParser = parser;
|
||||
mSuggestionFeatureProvider = FeatureFactory.getFactory(context)
|
||||
.getSuggestionFeatureProvider(context);
|
||||
mCallback = callback;
|
||||
@@ -103,13 +79,7 @@ public class SuggestionDismissController extends ItemTouchHelper.SimpleCallback
|
||||
}
|
||||
final int position = viewHolder.getAdapterPosition();
|
||||
final Suggestion suggestionV2 = mCallback.getSuggestionAt(position);
|
||||
if (suggestionV2 != null) {
|
||||
mSuggestionFeatureProvider.dismissSuggestion(mContext, mSuggestionMixin, suggestionV2);
|
||||
mCallback.onSuggestionDismissed(suggestionV2);
|
||||
} else {
|
||||
final Tile suggestion = mCallback.getSuggestionForPosition(position);
|
||||
mSuggestionFeatureProvider.dismissSuggestion(mContext, mSuggestionParser, suggestion);
|
||||
mCallback.onSuggestionDismissed(suggestion);
|
||||
}
|
||||
mSuggestionFeatureProvider.dismissSuggestion(mContext, mSuggestionMixin, suggestionV2);
|
||||
mCallback.onSuggestionDismissed(suggestionV2);
|
||||
}
|
||||
}
|
||||
|
@@ -24,7 +24,6 @@ import android.support.annotation.NonNull;
|
||||
import android.util.Pair;
|
||||
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
import com.android.settingslib.suggestions.SuggestionParser;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
@@ -33,14 +32,6 @@ public interface SuggestionFeatureProvider {
|
||||
|
||||
/**
|
||||
* Whether or not the whole suggestion feature is enabled.
|
||||
*
|
||||
* @deprecated in favor of {@link #isSuggestionV2Enabled(Context)}
|
||||
*/
|
||||
@Deprecated
|
||||
boolean isSuggestionEnabled(Context context);
|
||||
|
||||
/**
|
||||
* Whether or not the suggestion v2 feature is enabled.
|
||||
*/
|
||||
boolean isSuggestionV2Enabled(Context context);
|
||||
|
||||
@@ -62,44 +53,17 @@ public interface SuggestionFeatureProvider {
|
||||
*/
|
||||
SharedPreferences getSharedPrefs(Context context);
|
||||
|
||||
/**
|
||||
* Ranks the list of suggestions in place.
|
||||
*
|
||||
* @param suggestions List of suggestion Tiles
|
||||
* @param suggestionIds List of suggestion ids corresponding to the suggestion tiles.
|
||||
* @deprecated in favor of SettingsIntelligence
|
||||
*/
|
||||
@Deprecated
|
||||
void rankSuggestions(final List<Tile> suggestions, List<String> suggestionIds);
|
||||
|
||||
/**
|
||||
* Only keep top few suggestions from exclusive suggestions.
|
||||
*/
|
||||
void filterExclusiveSuggestions(List<Tile> suggestions);
|
||||
|
||||
/**
|
||||
* Dismisses a suggestion.
|
||||
*
|
||||
* @deprecated in favor of {@link #dismissSuggestion(Context, SuggestionControllerMixin,
|
||||
* Suggestion)}
|
||||
*/
|
||||
@Deprecated
|
||||
void dismissSuggestion(Context context, SuggestionParser parser, Tile suggestion);
|
||||
|
||||
/**
|
||||
* Dismisses a suggestion.
|
||||
*/
|
||||
void dismissSuggestion(Context context, SuggestionControllerMixin suggestionMixin,
|
||||
Suggestion suggestion);
|
||||
|
||||
/**
|
||||
* Returns an identifier for the suggestion
|
||||
*
|
||||
* @deprecated in favor or {@link Suggestion#getId()}
|
||||
*/
|
||||
@Deprecated
|
||||
String getSuggestionIdentifier(Context context, Tile suggestion);
|
||||
|
||||
/**
|
||||
* Returns common tagged data for suggestion logging.
|
||||
*/
|
||||
|
@@ -16,19 +16,15 @@
|
||||
|
||||
package com.android.settings.dashboard.suggestions;
|
||||
|
||||
import static com.android.settings.core.FeatureFlags.SUGGESTIONS_V2;
|
||||
|
||||
import android.app.ActivityManager;
|
||||
import android.content.ComponentName;
|
||||
import android.content.ContentResolver;
|
||||
import android.content.Context;
|
||||
import android.content.SharedPreferences;
|
||||
import android.content.pm.PackageManager;
|
||||
import android.provider.Settings.Secure;
|
||||
import android.service.settings.suggestions.Suggestion;
|
||||
import android.support.annotation.NonNull;
|
||||
import android.support.annotation.VisibleForTesting;
|
||||
import android.util.FeatureFlagUtils;
|
||||
import android.util.Log;
|
||||
import android.util.Pair;
|
||||
|
||||
@@ -54,7 +50,6 @@ import com.android.settings.support.NewDeviceIntroSuggestionActivity;
|
||||
import com.android.settings.wallpaper.WallpaperSuggestionActivity;
|
||||
import com.android.settings.wifi.WifiCallingSuggestionActivity;
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
import com.android.settingslib.suggestions.SuggestionParser;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
@@ -65,23 +60,13 @@ public class SuggestionFeatureProviderImpl implements SuggestionFeatureProvider
|
||||
|
||||
private static final String SHARED_PREF_FILENAME = "suggestions";
|
||||
|
||||
private final SuggestionRanker mSuggestionRanker;
|
||||
private final MetricsFeatureProvider mMetricsFeatureProvider;
|
||||
|
||||
@Override
|
||||
public boolean isSuggestionEnabled(Context context) {
|
||||
final ActivityManager am =
|
||||
(ActivityManager) context.getSystemService(Context.ACTIVITY_SERVICE);
|
||||
boolean isLowRamDevice = am.isLowRamDevice();
|
||||
return !isLowRamDevice && !isV2Enabled(context);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean isSuggestionV2Enabled(Context context) {
|
||||
final ActivityManager am =
|
||||
(ActivityManager) context.getSystemService(Context.ACTIVITY_SERVICE);
|
||||
boolean isLowRamDevice = am.isLowRamDevice();
|
||||
return !isLowRamDevice && isV2Enabled(context);
|
||||
return !am.isLowRamDevice();
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -91,10 +76,6 @@ public class SuggestionFeatureProviderImpl implements SuggestionFeatureProvider
|
||||
"com.android.settings.intelligence.suggestions.SuggestionService");
|
||||
}
|
||||
|
||||
private static boolean isV2Enabled(Context context) {
|
||||
return FeatureFlagUtils.isEnabled(context, SUGGESTIONS_V2);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean isSmartSuggestionEnabled(Context context) {
|
||||
return false;
|
||||
@@ -143,17 +124,10 @@ public class SuggestionFeatureProviderImpl implements SuggestionFeatureProvider
|
||||
|
||||
public SuggestionFeatureProviderImpl(Context context) {
|
||||
final Context appContext = context.getApplicationContext();
|
||||
mSuggestionRanker = new SuggestionRanker(
|
||||
new SuggestionFeaturizer(new EventStore(appContext)));
|
||||
mMetricsFeatureProvider = FeatureFactory.getFactory(appContext)
|
||||
.getMetricsFeatureProvider();
|
||||
}
|
||||
|
||||
@Override
|
||||
public void rankSuggestions(final List<Tile> suggestions, List<String> suggestionIds) {
|
||||
mSuggestionRanker.rankSuggestions(suggestions, suggestionIds);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void filterExclusiveSuggestions(List<Tile> suggestions) {
|
||||
if (suggestions == null) {
|
||||
@@ -165,26 +139,6 @@ public class SuggestionFeatureProviderImpl implements SuggestionFeatureProvider
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public void dismissSuggestion(Context context, SuggestionParser parser, Tile suggestion) {
|
||||
if (parser == null || suggestion == null || context == null) {
|
||||
return;
|
||||
}
|
||||
final Pair<Integer, Object>[] taggedData = getLoggingTaggedData(context);
|
||||
|
||||
mMetricsFeatureProvider.action(
|
||||
context, MetricsEvent.ACTION_SETTINGS_DISMISS_SUGGESTION,
|
||||
getSuggestionIdentifier(context, suggestion),
|
||||
taggedData);
|
||||
if (!parser.dismissSuggestion(suggestion)) {
|
||||
return;
|
||||
}
|
||||
context.getPackageManager().setComponentEnabledSetting(
|
||||
suggestion.intent.getComponent(),
|
||||
PackageManager.COMPONENT_ENABLED_STATE_DISABLED,
|
||||
PackageManager.DONT_KILL_APP);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void dismissSuggestion(Context context, SuggestionControllerMixin mixin,
|
||||
Suggestion suggestion) {
|
||||
@@ -197,25 +151,10 @@ public class SuggestionFeatureProviderImpl implements SuggestionFeatureProvider
|
||||
mixin.dismissSuggestion(suggestion);
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getSuggestionIdentifier(Context context, Tile suggestion) {
|
||||
if (suggestion.intent == null || suggestion.intent.getComponent() == null
|
||||
|| context == null) {
|
||||
return "unknown_suggestion";
|
||||
}
|
||||
String packageName = suggestion.intent.getComponent().getPackageName();
|
||||
if (packageName.equals(context.getPackageName())) {
|
||||
// Since Settings provides several suggestions, fill in the class instead of the
|
||||
// package for these.
|
||||
packageName = suggestion.intent.getComponent().getClassName();
|
||||
}
|
||||
return packageName;
|
||||
}
|
||||
|
||||
@Override
|
||||
public Pair<Integer, Object>[] getLoggingTaggedData(Context context) {
|
||||
final boolean isSmartSuggestionEnabled = isSmartSuggestionEnabled(context);
|
||||
return new Pair[]{Pair.create(
|
||||
return new Pair[] {Pair.create(
|
||||
MetricsEvent.FIELD_SETTINGS_SMART_SUGGESTIONS_ENABLED,
|
||||
isSmartSuggestionEnabled ? 1 : 0)};
|
||||
}
|
||||
|
@@ -1,106 +0,0 @@
|
||||
/*
|
||||
* Copyright (C) 2017 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License
|
||||
*/
|
||||
package com.android.settings.dashboard.suggestions;
|
||||
|
||||
import java.util.HashMap;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
|
||||
/**
|
||||
* Creates a set of interaction features (i.e., metrics) to represent each setting suggestion. These
|
||||
* features currently include normalized time from previous events (shown, dismissed and clicked)
|
||||
* for any particular suggestion and also counts of these events. These features are used as signals
|
||||
* to find the best ranking for suggestion items.
|
||||
*/
|
||||
public class SuggestionFeaturizer {
|
||||
|
||||
// Key of the features used for ranking.
|
||||
public static final String FEATURE_IS_SHOWN = "is_shown";
|
||||
public static final String FEATURE_IS_DISMISSED = "is_dismissed";
|
||||
public static final String FEATURE_IS_CLICKED = "is_clicked";
|
||||
public static final String FEATURE_TIME_FROM_LAST_SHOWN = "time_from_last_shown";
|
||||
public static final String FEATURE_TIME_FROM_LAST_DISMISSED = "time_from_last_dismissed";
|
||||
public static final String FEATURE_TIME_FROM_LAST_CLICKED = "time_from_last_clicked";
|
||||
public static final String FEATURE_SHOWN_COUNT = "shown_count";
|
||||
public static final String FEATURE_DISMISSED_COUNT = "dismissed_count";
|
||||
public static final String FEATURE_CLICKED_COUNT = "clicked_count";
|
||||
|
||||
// The following numbers are estimated from histograms.
|
||||
public static final double TIME_NORMALIZATION_FACTOR = 2e10;
|
||||
public static final double COUNT_NORMALIZATION_FACTOR = 500;
|
||||
|
||||
private final EventStore mEventStore;
|
||||
|
||||
/**
|
||||
* Constructor
|
||||
*
|
||||
* @param eventStore An instance of {@code EventStore} which maintains the recorded suggestion
|
||||
* events.
|
||||
*/
|
||||
public SuggestionFeaturizer(EventStore eventStore) {
|
||||
mEventStore = eventStore;
|
||||
}
|
||||
|
||||
/**
|
||||
* Extracts the features for each package name.
|
||||
*
|
||||
* @param pkgNames: List of package names for which features are queried.
|
||||
* @return A Map containing the features, keyed by the package names. Each map value contains
|
||||
* another map with key-value pairs of the features.
|
||||
*/
|
||||
public Map<String, Map<String, Double>> featurize(List<String> pkgNames) {
|
||||
Map<String, Map<String, Double>> features = new HashMap<>();
|
||||
Long curTimeMs = System.currentTimeMillis();
|
||||
for (String pkgName : pkgNames) {
|
||||
Map<String, Double> featureMap = new HashMap<>();
|
||||
features.put(pkgName, featureMap);
|
||||
Long lastShownTime = mEventStore
|
||||
.readMetric(pkgName, EventStore.EVENT_SHOWN, EventStore.METRIC_LAST_EVENT_TIME);
|
||||
Long lastDismissedTime = mEventStore.readMetric(pkgName, EventStore.EVENT_DISMISSED,
|
||||
EventStore.METRIC_LAST_EVENT_TIME);
|
||||
Long lastClickedTime = mEventStore.readMetric(pkgName, EventStore.EVENT_CLICKED,
|
||||
EventStore.METRIC_LAST_EVENT_TIME);
|
||||
featureMap.put(FEATURE_IS_SHOWN, booleanToDouble(lastShownTime > 0));
|
||||
featureMap.put(FEATURE_IS_DISMISSED, booleanToDouble(lastDismissedTime > 0));
|
||||
featureMap.put(FEATURE_IS_CLICKED, booleanToDouble(lastClickedTime > 0));
|
||||
featureMap.put(FEATURE_TIME_FROM_LAST_SHOWN,
|
||||
normalizedTimeDiff(curTimeMs, lastShownTime));
|
||||
featureMap.put(FEATURE_TIME_FROM_LAST_DISMISSED,
|
||||
normalizedTimeDiff(curTimeMs, lastDismissedTime));
|
||||
featureMap.put(FEATURE_TIME_FROM_LAST_CLICKED,
|
||||
normalizedTimeDiff(curTimeMs, lastClickedTime));
|
||||
featureMap.put(FEATURE_SHOWN_COUNT, normalizedCount(mEventStore
|
||||
.readMetric(pkgName, EventStore.EVENT_SHOWN, EventStore.METRIC_COUNT)));
|
||||
featureMap.put(FEATURE_DISMISSED_COUNT, normalizedCount(mEventStore
|
||||
.readMetric(pkgName, EventStore.EVENT_DISMISSED, EventStore.METRIC_COUNT)));
|
||||
featureMap.put(FEATURE_CLICKED_COUNT, normalizedCount(mEventStore
|
||||
.readMetric(pkgName, EventStore.EVENT_CLICKED, EventStore.METRIC_COUNT)));
|
||||
}
|
||||
return features;
|
||||
}
|
||||
|
||||
private static double booleanToDouble(boolean bool) {
|
||||
return bool ? 1 : 0;
|
||||
}
|
||||
|
||||
private static double normalizedTimeDiff(long curTimeMs, long preTimeMs) {
|
||||
return Math.min(1, (curTimeMs - preTimeMs) / TIME_NORMALIZATION_FACTOR);
|
||||
}
|
||||
|
||||
private static double normalizedCount(long count) {
|
||||
return Math.min(1, count / COUNT_NORMALIZATION_FACTOR);
|
||||
}
|
||||
}
|
@@ -1,83 +0,0 @@
|
||||
/*
|
||||
* Copyright (C) 2017 The Android Open Source Project
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License
|
||||
*/
|
||||
package com.android.settings.dashboard.suggestions;
|
||||
|
||||
import com.android.settingslib.drawer.Tile;
|
||||
|
||||
import android.support.annotation.VisibleForTesting;
|
||||
|
||||
import java.util.Collections;
|
||||
import java.util.Comparator;
|
||||
import java.util.HashMap;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
|
||||
public class SuggestionRanker {
|
||||
|
||||
private static final String TAG = "SuggestionRanker";
|
||||
|
||||
// The following coefficients form a linear model, which mixes the features to obtain a
|
||||
// relevance metric for ranking the suggestion items. This model is learned with off-line data
|
||||
// by training a binary classifier to detect the clicked items. The higher the obtained
|
||||
// relevance metric, the higher chance of getting clicked.
|
||||
private static final Map<String, Double> WEIGHTS = new HashMap<String, Double>() {{
|
||||
put(SuggestionFeaturizer.FEATURE_IS_SHOWN, 5.05140842519);
|
||||
put(SuggestionFeaturizer.FEATURE_IS_DISMISSED, 2.29641455171);
|
||||
put(SuggestionFeaturizer.FEATURE_IS_CLICKED, -2.98812233623);
|
||||
put(SuggestionFeaturizer.FEATURE_TIME_FROM_LAST_SHOWN, 5.02807250202);
|
||||
put(SuggestionFeaturizer.FEATURE_TIME_FROM_LAST_DISMISSED, 2.49589700842);
|
||||
put(SuggestionFeaturizer.FEATURE_TIME_FROM_LAST_CLICKED, -4.3377039948);
|
||||
put(SuggestionFeaturizer.FEATURE_SHOWN_COUNT, -2.35993512546);
|
||||
}};
|
||||
|
||||
private final SuggestionFeaturizer mSuggestionFeaturizer;
|
||||
|
||||
private final Map<Tile, Double> relevanceMetrics;
|
||||
|
||||
Comparator<Tile> suggestionComparator = new Comparator<Tile>() {
|
||||
@Override
|
||||
public int compare(Tile suggestion1, Tile suggestion2) {
|
||||
return relevanceMetrics.get(suggestion1) < relevanceMetrics.get(suggestion2) ? 1 : -1;
|
||||
}
|
||||
};
|
||||
|
||||
public SuggestionRanker(SuggestionFeaturizer suggestionFeaturizer) {
|
||||
mSuggestionFeaturizer = suggestionFeaturizer;
|
||||
relevanceMetrics = new HashMap<Tile, Double>();
|
||||
}
|
||||
|
||||
public void rankSuggestions(final List<Tile> suggestions, List<String> suggestionIds) {
|
||||
relevanceMetrics.clear();
|
||||
Map<String, Map<String, Double>> features = mSuggestionFeaturizer.featurize(suggestionIds);
|
||||
for (int i = 0; i < suggestionIds.size(); i++) {
|
||||
relevanceMetrics.put(suggestions.get(i),
|
||||
getRelevanceMetric(features.get(suggestionIds.get(i))));
|
||||
}
|
||||
Collections.sort(suggestions, suggestionComparator);
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
double getRelevanceMetric(Map<String, Double> features) {
|
||||
double sum = 0;
|
||||
if (features == null) {
|
||||
return sum;
|
||||
}
|
||||
for (String feature : WEIGHTS.keySet()) {
|
||||
sum += WEIGHTS.get(feature) * features.get(feature);
|
||||
}
|
||||
return sum;
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user