Remove condition manager v1 code.
Change-Id: I866c1f7c493c6080ccf766ecfc8e6b00dafeecd9
Fixes: 112485407
Test: robotests
diff --git a/src/com/android/settings/core/FeatureFlags.java b/src/com/android/settings/core/FeatureFlags.java
index c8ae82f..c6b9d83 100644
--- a/src/com/android/settings/core/FeatureFlags.java
+++ b/src/com/android/settings/core/FeatureFlags.java
@@ -25,5 +25,4 @@
public static final String AUDIO_SWITCHER_SETTINGS = "settings_audio_switcher";
public static final String DYNAMIC_HOMEPAGE = "settings_dynamic_homepage";
public static final String HEARING_AID_SETTINGS = "settings_bluetooth_hearing_aid";
- public static final String CONDITION_MANAGER_V2 = "settings_condition_manager_v2";
}
diff --git a/src/com/android/settings/dashboard/DashboardAdapter.java b/src/com/android/settings/dashboard/DashboardAdapter.java
index 5a8df91..5e2e57d 100644
--- a/src/com/android/settings/dashboard/DashboardAdapter.java
+++ b/src/com/android/settings/dashboard/DashboardAdapter.java
@@ -40,10 +40,9 @@
import com.android.settings.R.id;
import com.android.settings.dashboard.DashboardData.ConditionHeaderData;
import com.android.settings.dashboard.suggestions.SuggestionAdapter;
-import com.android.settings.homepage.conditional.Condition;
import com.android.settings.homepage.conditional.ConditionAdapter;
-import com.android.settings.homepage.conditional.v2.ConditionManager;
-import com.android.settings.homepage.conditional.v2.ConditionalCard;
+import com.android.settings.homepage.conditional.ConditionManager;
+import com.android.settings.homepage.conditional.ConditionalCard;
import com.android.settings.overlay.FeatureFactory;
import com.android.settings.widget.RoundedHomepageIcon;
import com.android.settingslib.core.instrumentation.MetricsFeatureProvider;
@@ -88,7 +87,7 @@
};
public DashboardAdapter(Context context, Bundle savedInstanceState,
- List<Condition> conditions, ConditionManager conditionManager,
+ ConditionManager conditionManager,
SuggestionControllerMixinCompat suggestionControllerMixin, Lifecycle lifecycle) {
DashboardCategory category = null;
@@ -116,8 +115,7 @@
}
mDashboardData = new DashboardData.Builder()
- .setConditions(conditions)
- .setConditionsV2(
+ .setConditions(
conditionManager == null ? null : conditionManager.getDisplayableCards())
.setSuggestions(mSuggestionAdapter.getSuggestions())
.setCategory(category)
@@ -142,7 +140,7 @@
notifyDashboardDataChanged(prevData);
}
- public void setConditions(List<Condition> conditions) {
+ public void setConditions(List<ConditionalCard> conditions) {
final DashboardData prevData = mDashboardData;
Log.d(TAG, "adapter setConditions called");
mDashboardData = new DashboardData.Builder(prevData)
@@ -151,15 +149,6 @@
notifyDashboardDataChanged(prevData);
}
- public void setConditionsV2(List<ConditionalCard> conditions) {
- final DashboardData prevData = mDashboardData;
- Log.d(TAG, "adapter setConditions called");
- mDashboardData = new DashboardData.Builder(prevData)
- .setConditionsV2(conditions)
- .build();
- notifyDashboardDataChanged(prevData);
- }
-
@Override
public void onSuggestionClosed(Suggestion suggestion) {
final List<Suggestion> list = mDashboardData.getSuggestions();
@@ -301,31 +290,12 @@
@VisibleForTesting
void onBindCondition(final ConditionContainerHolder holder, int position) {
- final List conditions = (List) mDashboardData.getItemEntityByPosition(position);
- final List<Condition> conditionsV1;
- final List<ConditionalCard> conditionsV2;
- if (conditions == null || conditions.isEmpty()) {
- conditionsV1 = null;
- conditionsV2 = null;
- } else if (conditions.get(0) instanceof Condition) {
- conditionsV1 = conditions;
- conditionsV2 = null;
- } else {
- conditionsV1 = null;
- conditionsV2 = conditions;
- }
- if (conditionsV2 == null) {
- final ConditionAdapter adapter = new ConditionAdapter(mContext,
- conditionsV1, mDashboardData.isConditionExpanded());
- adapter.addDismissHandling(holder.data);
- holder.data.setAdapter(adapter);
- } else {
- final com.android.settings.homepage.conditional.v2.ConditionAdapter adapter =
- new com.android.settings.homepage.conditional.v2.ConditionAdapter(
- mContext, mConditionManager, conditionsV2,
- mDashboardData.isConditionExpanded());
- holder.data.setAdapter(adapter);
- }
+ final List<ConditionalCard> conditions =
+ (List) mDashboardData.getItemEntityByPosition(position);
+ final ConditionAdapter adapter = new ConditionAdapter(
+ mContext, mConditionManager, conditions,
+ mDashboardData.isConditionExpanded());
+ holder.data.setAdapter(adapter);
holder.data.setLayoutManager(new LinearLayoutManager(mContext));
}
diff --git a/src/com/android/settings/dashboard/DashboardData.java b/src/com/android/settings/dashboard/DashboardData.java
index 9feff58..db296ab 100644
--- a/src/com/android/settings/dashboard/DashboardData.java
+++ b/src/com/android/settings/dashboard/DashboardData.java
@@ -24,8 +24,7 @@
import androidx.recyclerview.widget.DiffUtil;
import com.android.settings.R;
-import com.android.settings.homepage.conditional.Condition;
-import com.android.settings.homepage.conditional.v2.ConditionalCard;
+import com.android.settings.homepage.conditional.ConditionalCard;
import com.android.settingslib.drawer.DashboardCategory;
import com.android.settingslib.drawer.Tile;
@@ -57,15 +56,13 @@
private final List<Item> mItems;
private final DashboardCategory mCategory;
- private final List<Condition> mConditions;
- private final List<ConditionalCard> mConditionsV2;
+ private final List<ConditionalCard> mConditions;
private final List<Suggestion> mSuggestions;
private final boolean mConditionExpanded;
private DashboardData(Builder builder) {
mCategory = builder.mCategory;
mConditions = builder.mConditions;
- mConditionsV2 = builder.mConditionsV2;
mSuggestions = builder.mSuggestions;
mConditionExpanded = builder.mConditionExpanded;
mItems = new ArrayList<>();
@@ -106,7 +103,7 @@
return mCategory;
}
- public List<Condition> getConditions() {
+ public List<ConditionalCard> getConditions() {
return mConditions;
}
@@ -168,7 +165,8 @@
/**
* Add item into list when {@paramref add} is true.
*
- * @param item maybe {@link Condition}, {@link Tile}, {@link DashboardCategory} or null
+ * @param item maybe {@link ConditionalCard}, {@link Tile}, {@link DashboardCategory}
+ * or null
* @param type type of the item, and value is the layout id
* @param stableId The stable id for this item
* @param add flag about whether to add item into list
@@ -184,11 +182,8 @@
* and mIsShowingAll, mConditionExpanded flag.
*/
private void buildItemsData() {
- final List<Condition> conditionsV1 = getConditionsToShow(mConditions);
- final boolean hasConditionsV1 = sizeOf(conditionsV1) > 0;
- final List<ConditionalCard> conditionsV2 = mConditionsV2;
- final boolean hasConditionsV2 = sizeOf(conditionsV2) > 0;
- final boolean hasConditions = hasConditionsV1 || hasConditionsV2;
+ final List<ConditionalCard> conditions = mConditions;
+ final boolean hasConditions = sizeOf(conditions) > 0;
final List<Suggestion> suggestions = getSuggestionsToShow(mSuggestions);
final boolean hasSuggestions = sizeOf(suggestions) > 0;
@@ -203,20 +198,14 @@
STABLE_ID_SUGGESTION_CONDITION_DIVIDER, hasSuggestions && hasConditions);
/* Condition header. This will be present when there is condition and it is collapsed */
- addToItemList(new ConditionHeaderData(conditionsV1, conditionsV2),
+ addToItemList(new ConditionHeaderData(conditions),
R.layout.condition_header,
STABLE_ID_CONDITION_HEADER, hasConditions && !mConditionExpanded);
/* Condition container. This is the card view that contains the list of conditions.
* This will be added whenever the condition list is not empty and expanded */
- if (hasConditionsV1) {
- addToItemList(conditionsV1, R.layout.condition_container,
- STABLE_ID_CONDITION_CONTAINER, hasConditionsV1 && mConditionExpanded);
- }
- if (hasConditionsV2) {
- addToItemList(conditionsV2, R.layout.condition_container,
- STABLE_ID_CONDITION_CONTAINER, hasConditionsV2 && mConditionExpanded);
- }
+ addToItemList(conditions, R.layout.condition_container,
+ STABLE_ID_CONDITION_CONTAINER, hasConditions && mConditionExpanded);
/* Condition footer. This will be present when there is condition and it is expanded */
addToItemList(null /* item */, R.layout.condition_footer,
@@ -236,21 +225,6 @@
return list == null ? 0 : list.size();
}
- private List<Condition> getConditionsToShow(List<Condition> conditions) {
- if (conditions == null) {
- return null;
- }
- List<Condition> result = new ArrayList<>();
- final int size = conditions == null ? 0 : conditions.size();
- for (int i = 0; i < size; i++) {
- final Condition condition = conditions.get(i);
- if (condition.shouldShow()) {
- result.add(condition);
- }
- }
- return result;
- }
-
private List<Suggestion> getSuggestionsToShow(List<Suggestion> suggestions) {
if (suggestions == null) {
return null;
@@ -270,8 +244,7 @@
*/
public static class Builder {
private DashboardCategory mCategory;
- private List<Condition> mConditions;
- private List<ConditionalCard> mConditionsV2;
+ private List<ConditionalCard> mConditions;
private List<Suggestion> mSuggestions;
private boolean mConditionExpanded;
@@ -281,7 +254,6 @@
public Builder(DashboardData dashboardData) {
mCategory = dashboardData.mCategory;
mConditions = dashboardData.mConditions;
- mConditionsV2 = dashboardData.mConditionsV2;
mSuggestions = dashboardData.mSuggestions;
mConditionExpanded = dashboardData.mConditionExpanded;
}
@@ -291,16 +263,11 @@
return this;
}
- public Builder setConditions(List<Condition> conditions) {
+ public Builder setConditions(List<ConditionalCard> conditions) {
this.mConditions = conditions;
return this;
}
- public Builder setConditionsV2(List<ConditionalCard> conditions) {
- this.mConditionsV2 = conditions;
- return this;
- }
-
public Builder setSuggestions(List<Suggestion> suggestions) {
this.mSuggestions = suggestions;
return this;
@@ -374,7 +341,7 @@
}
/**
- * The main data object in item, usually is a {@link Tile}, {@link Condition}
+ * The main data object in item, usually is a {@link Tile}, {@link ConditionalCard}
* object. This object can also be null when the
* item is an divider line. Please refer to {@link #buildItemsData()} for
* detail usage of the Item.
@@ -448,22 +415,15 @@
public final CharSequence title;
public final int conditionCount;
- public ConditionHeaderData(List<Condition> conditions, List<ConditionalCard> conditionsV2) {
- if (conditionsV2 == null) {
- conditionCount = sizeOf(conditions);
- title = conditionCount > 0 ? conditions.get(0).getTitle() : null;
- conditionIcons = new ArrayList<>();
- for (int i = 0; conditions != null && i < conditions.size(); i++) {
- final Condition condition = conditions.get(i);
- conditionIcons.add(condition.getIcon());
- }
- } else {
- conditionCount = sizeOf(conditionsV2);
- title = conditionCount > 0 ? conditionsV2.get(0).getTitle() : null;
- conditionIcons = new ArrayList<>();
- for (ConditionalCard card : conditionsV2) {
- conditionIcons.add(card.getIcon());
- }
+ public ConditionHeaderData(List<ConditionalCard> conditions) {
+ conditionCount = sizeOf(conditions);
+ title = conditionCount > 0 ? conditions.get(0).getTitle() : null;
+ conditionIcons = new ArrayList<>();
+ if (conditions == null) {
+ return;
+ }
+ for (ConditionalCard card : conditions) {
+ conditionIcons.add(card.getIcon());
}
}
}
diff --git a/src/com/android/settings/dashboard/DashboardSummary.java b/src/com/android/settings/dashboard/DashboardSummary.java
index 18f6999..4575482 100644
--- a/src/com/android/settings/dashboard/DashboardSummary.java
+++ b/src/com/android/settings/dashboard/DashboardSummary.java
@@ -37,11 +37,10 @@
import com.android.settings.core.SettingsBaseActivity;
import com.android.settings.core.SettingsBaseActivity.CategoryListener;
import com.android.settings.dashboard.suggestions.SuggestionFeatureProvider;
-import com.android.settings.homepage.conditional.Condition;
import com.android.settings.homepage.conditional.ConditionListener;
-import com.android.settings.homepage.conditional.ConditionManager;
import com.android.settings.homepage.conditional.FocusRecyclerView;
import com.android.settings.homepage.conditional.FocusRecyclerView.FocusListener;
+import com.android.settings.homepage.conditional.ConditionManager;
import com.android.settings.overlay.FeatureFactory;
import com.android.settings.widget.ActionBarShadowController;
import com.android.settingslib.drawer.CategoryKey;
@@ -74,7 +73,6 @@
private DashboardAdapter mAdapter;
private SummaryLoader mSummaryLoader;
private ConditionManager mConditionManager;
- private com.android.settings.homepage.conditional.v2.ConditionManager mConditionManager2;
private LinearLayoutManager mLayoutManager;
private SuggestionControllerMixinCompat mSuggestionControllerMixin;
private DashboardFeatureProvider mDashboardFeatureProvider;
@@ -123,18 +121,9 @@
mSummaryLoader = new SummaryLoader(activity, CategoryKey.CATEGORY_HOMEPAGE);
- if (com.android.settings.homepage.conditional.v2.ConditionManager.isEnabled(activity)) {
- mConditionManager = null;
- mConditionManager2 =
- new com.android.settings.homepage.conditional.v2.ConditionManager(
- activity, this /* listener */);
- } else {
- mConditionManager = ConditionManager.get(activity, false);
- mConditionManager2 = null;
- }
- if (mConditionManager2 == null) {
- getSettingsLifecycle().addObserver(mConditionManager);
- }
+ mConditionManager =
+ new ConditionManager(
+ activity, this /* listener */);
if (savedInstanceState != null) {
mIsOnCategoriesChangedCalled =
savedInstanceState.getBoolean(STATE_CATEGORIES_CHANGE_CALLED);
@@ -157,17 +146,7 @@
((SettingsBaseActivity) getActivity()).addCategoryListener(this);
mSummaryLoader.setListening(true);
- final int metricsCategory = getMetricsCategory();
- if (mConditionManager2 == null) {
- for (Condition c : mConditionManager.getConditions()) {
- if (c.shouldShow()) {
- mMetricsFeatureProvider.visible(getContext(), metricsCategory,
- c.getMetricsConstant());
- }
- }
- } else {
- mConditionManager2.startMonitoringStateChange();
- }
+ mConditionManager.startMonitoringStateChange();
if (DEBUG_TIMING) {
Log.d(TAG, "onResume took " + (System.currentTimeMillis() - startTime) + " ms");
}
@@ -179,42 +158,16 @@
((SettingsBaseActivity) getActivity()).remCategoryListener(this);
mSummaryLoader.setListening(false);
- if (mConditionManager2 == null) {
- for (Condition c : mConditionManager.getConditions()) {
- if (c.shouldShow()) {
- mMetricsFeatureProvider.hidden(getContext(), c.getMetricsConstant());
- }
- }
- }
- // Unregister condition listeners.
- if (mConditionManager != null) {
- mConditionManager.remListener(this);
- }
- if (mConditionManager2 != null) {
- mConditionManager2.stopMonitoringStateChange();
- }
+ mConditionManager.stopMonitoringStateChange();
}
@Override
public void onWindowFocusChanged(boolean hasWindowFocus) {
long startTime = System.currentTimeMillis();
- if (mConditionManager2 == null) {
- if (hasWindowFocus) {
- Log.d(TAG, "Listening for condition changes");
- mConditionManager.addListener(this);
- Log.d(TAG, "conditions refreshed");
- mConditionManager.refreshAll();
- } else {
- Log.d(TAG, "Stopped listening for condition changes");
- mConditionManager.remListener(this);
- }
+ if (hasWindowFocus) {
+ mConditionManager.startMonitoringStateChange();
} else {
- // TODO(b/112485407): Register monitoring for condition manager v2.
- if (hasWindowFocus) {
- mConditionManager2.startMonitoringStateChange();
- } else {
- mConditionManager2.stopMonitoringStateChange();
- }
+ mConditionManager.stopMonitoringStateChange();
}
if (DEBUG_TIMING) {
Log.d(TAG, "onWindowFocusChanged took "
@@ -248,8 +201,7 @@
mDashboard.setListener(this);
mDashboard.setItemAnimator(new DashboardItemAnimator());
mAdapter = new DashboardAdapter(getContext(), bundle,
- mConditionManager == null ? null : mConditionManager.getConditions(),
- mConditionManager2,
+ mConditionManager,
mSuggestionControllerMixin,
getSettingsLifecycle());
mDashboard.setAdapter(mAdapter);
@@ -290,14 +242,10 @@
// constructor when we create the view, the first handling is not necessary.
// But, on the subsequent calls we need to handle it because there might be real changes to
// conditions.
- if (mOnConditionsChangedCalled || mConditionManager2 != null) {
+ if (mOnConditionsChangedCalled) {
final boolean scrollToTop =
mLayoutManager.findFirstCompletelyVisibleItemPosition() <= 1;
- if (mConditionManager2 == null) {
- mAdapter.setConditions(mConditionManager.getConditions());
- } else {
- mAdapter.setConditionsV2(mConditionManager2.getDisplayableCards());
- }
+ mAdapter.setConditions(mConditionManager.getDisplayableCards());
if (scrollToTop) {
mDashboard.scrollToPosition(0);
diff --git a/src/com/android/settings/homepage/conditional/AbnormalRingerConditionBase.java b/src/com/android/settings/homepage/conditional/AbnormalRingerConditionBase.java
deleted file mode 100644
index b6a67af..0000000
--- a/src/com/android/settings/homepage/conditional/AbnormalRingerConditionBase.java
+++ /dev/null
@@ -1,84 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.Intent;
-import android.content.IntentFilter;
-import android.media.AudioManager;
-import android.provider.Settings;
-
-import com.android.settings.R;
-
-@Deprecated
-public abstract class AbnormalRingerConditionBase extends Condition {
-
- private final IntentFilter mFilter;
-
- protected final AudioManager mAudioManager;
-
- private final RingerModeChangeReceiver mReceiver;
-
- AbnormalRingerConditionBase(ConditionManager manager) {
- super(manager);
- mAudioManager =
- (AudioManager) mManager.getContext().getSystemService(Context.AUDIO_SERVICE);
- mReceiver = new RingerModeChangeReceiver(this);
-
- mFilter = new IntentFilter(AudioManager.INTERNAL_RINGER_MODE_CHANGED_ACTION);
- manager.getContext().registerReceiver(mReceiver, mFilter);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[]{
- mManager.getContext().getText(R.string.condition_device_muted_action_turn_on_sound)
- };
- }
-
- @Override
- public void onPrimaryClick() {
- mManager.getContext().startActivity(
- new Intent(Settings.ACTION_SOUND_SETTINGS)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK));
- }
-
- @Override
- public void onActionClick(int index) {
- mAudioManager.setRingerModeInternal(AudioManager.RINGER_MODE_NORMAL);
- mAudioManager.setStreamVolume(AudioManager.STREAM_RING, 1, 0 /* flags */);
- refreshState();
- }
-
- static class RingerModeChangeReceiver extends BroadcastReceiver {
-
- private final AbnormalRingerConditionBase mCondition;
-
- public RingerModeChangeReceiver(AbnormalRingerConditionBase condition) {
- mCondition = condition;
- }
-
- @Override
- public void onReceive(Context context, Intent intent) {
- final String action = intent.getAction();
- if (AudioManager.INTERNAL_RINGER_MODE_CHANGED_ACTION.equals(action)) {
- mCondition.refreshState();
- }
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/AbnormalRingerConditionController.java b/src/com/android/settings/homepage/conditional/AbnormalRingerConditionController.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/AbnormalRingerConditionController.java
rename to src/com/android/settings/homepage/conditional/AbnormalRingerConditionController.java
index 1cc62b9..c6fa21d 100644
--- a/src/com/android/settings/homepage/conditional/v2/AbnormalRingerConditionController.java
+++ b/src/com/android/settings/homepage/conditional/AbnormalRingerConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.BroadcastReceiver;
import android.content.Context;
@@ -23,8 +23,6 @@
import android.media.AudioManager;
import android.provider.Settings;
-import java.util.Objects;
-
public abstract class AbnormalRingerConditionController implements ConditionalCardController {
private static final IntentFilter FILTER =
diff --git a/src/com/android/settings/homepage/conditional/AirplaneModeCondition.java b/src/com/android/settings/homepage/conditional/AirplaneModeCondition.java
deleted file mode 100644
index 069d316..0000000
--- a/src/com/android/settings/homepage/conditional/AirplaneModeCondition.java
+++ /dev/null
@@ -1,121 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.Intent;
-import android.content.IntentFilter;
-import android.graphics.drawable.Drawable;
-import android.net.ConnectivityManager;
-import android.provider.Settings;
-import android.util.Log;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settingslib.WirelessUtils;
-
-/**
- * @deprecated in favor of {@link AirplaneModeConditionCard}.
- */
-@Deprecated
-public class AirplaneModeCondition extends Condition {
- public static String TAG = "APM_Condition";
-
- private final Receiver mReceiver;
-
- private static final IntentFilter AIRPLANE_MODE_FILTER =
- new IntentFilter(Intent.ACTION_AIRPLANE_MODE_CHANGED);
-
- public AirplaneModeCondition(ConditionManager conditionManager) {
- super(conditionManager);
- mReceiver = new Receiver();
- }
-
- @Override
- public void refreshState() {
- Log.d(TAG, "APM condition refreshed");
- setActive(WirelessUtils.isAirplaneModeOn(mManager.getContext()));
- }
-
- @Override
- protected BroadcastReceiver getReceiver() {
- return mReceiver;
- }
-
- @Override
- protected IntentFilter getIntentFilter() {
- return AIRPLANE_MODE_FILTER;
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_airplane);
- }
-
- @Override
- protected void setActive(boolean active) {
- super.setActive(active);
- Log.d(TAG, "setActive was called with " + active);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_airplane_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_airplane_summary);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[] {mManager.getContext().getString(R.string.condition_turn_off)};
- }
-
- @Override
- public void onPrimaryClick() {
- mManager.getContext().startActivity(
- new Intent(Settings.ACTION_WIRELESS_SETTINGS)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK));
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- ConnectivityManager.from(mManager.getContext()).setAirplaneMode(false);
- setActive(false);
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_AIRPLANE_MODE;
- }
-
- public static class Receiver extends BroadcastReceiver {
- @Override
- public void onReceive(Context context, Intent intent) {
- if (Intent.ACTION_AIRPLANE_MODE_CHANGED.equals(intent.getAction())) {
- ConditionManager.get(context).getCondition(AirplaneModeCondition.class)
- .refreshState();
- }
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/AirplaneModeConditionCard.java b/src/com/android/settings/homepage/conditional/AirplaneModeConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/AirplaneModeConditionCard.java
rename to src/com/android/settings/homepage/conditional/AirplaneModeConditionCard.java
index 99055a7..f56fcc3 100644
--- a/src/com/android/settings/homepage/conditional/v2/AirplaneModeConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/AirplaneModeConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/AirplaneModeConditionController.java b/src/com/android/settings/homepage/conditional/AirplaneModeConditionController.java
similarity index 97%
rename from src/com/android/settings/homepage/conditional/v2/AirplaneModeConditionController.java
rename to src/com/android/settings/homepage/conditional/AirplaneModeConditionController.java
index 2d200ec..db3bc71 100644
--- a/src/com/android/settings/homepage/conditional/v2/AirplaneModeConditionController.java
+++ b/src/com/android/settings/homepage/conditional/AirplaneModeConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.BroadcastReceiver;
import android.content.Context;
diff --git a/src/com/android/settings/homepage/conditional/BackgroundDataCondition.java b/src/com/android/settings/homepage/conditional/BackgroundDataCondition.java
deleted file mode 100644
index 4e374ef..0000000
--- a/src/com/android/settings/homepage/conditional/BackgroundDataCondition.java
+++ /dev/null
@@ -1,79 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.content.Intent;
-import android.graphics.drawable.Drawable;
-import android.net.NetworkPolicyManager;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settings.Settings;
-
-@Deprecated
-public class BackgroundDataCondition extends Condition {
-
- public BackgroundDataCondition(ConditionManager manager) {
- super(manager);
- }
-
- @Override
- public void refreshState() {
- setActive(NetworkPolicyManager.from(mManager.getContext()).getRestrictBackground());
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_data_saver);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_bg_data_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_bg_data_summary);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[] {mManager.getContext().getString(R.string.condition_turn_off)};
- }
-
- @Override
- public void onPrimaryClick() {
- mManager.getContext().startActivity(new Intent(mManager.getContext(),
- Settings.DataUsageSummaryActivity.class)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK));
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_BACKGROUND_DATA;
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- NetworkPolicyManager.from(mManager.getContext()).setRestrictBackground(false);
- setActive(false);
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionCard.java b/src/com/android/settings/homepage/conditional/BackgroundDataConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionCard.java
rename to src/com/android/settings/homepage/conditional/BackgroundDataConditionCard.java
index 3ea3113..13792a9 100644
--- a/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/BackgroundDataConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionController.java b/src/com/android/settings/homepage/conditional/BackgroundDataConditionController.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionController.java
rename to src/com/android/settings/homepage/conditional/BackgroundDataConditionController.java
index 3649607..d7c0698 100644
--- a/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionController.java
+++ b/src/com/android/settings/homepage/conditional/BackgroundDataConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.content.Intent;
diff --git a/src/com/android/settings/homepage/conditional/BatterySaverCondition.java b/src/com/android/settings/homepage/conditional/BatterySaverCondition.java
deleted file mode 100644
index f36dcb7..0000000
--- a/src/com/android/settings/homepage/conditional/BatterySaverCondition.java
+++ /dev/null
@@ -1,117 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.content.Intent;
-import android.graphics.drawable.Drawable;
-import android.os.PowerManager;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settings.core.SubSettingLauncher;
-import com.android.settings.fuelgauge.BatterySaverReceiver;
-import com.android.settings.fuelgauge.batterysaver.BatterySaverSettings;
-import com.android.settingslib.fuelgauge.BatterySaverUtils;
-
-/**
- * @deprecated in favor of {@link BatterySaverConditionCard}
- */
-@Deprecated
-public class BatterySaverCondition extends Condition implements
- BatterySaverReceiver.BatterySaverListener {
-
- private final BatterySaverReceiver mReceiver;
-
- public BatterySaverCondition(ConditionManager manager) {
- super(manager);
-
- mReceiver = new BatterySaverReceiver(manager.getContext());
- mReceiver.setBatterySaverListener(this);
- }
-
- @Override
- public void refreshState() {
- PowerManager powerManager = mManager.getContext().getSystemService(PowerManager.class);
- setActive(powerManager.isPowerSaveMode());
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_battery_saver_accent_24dp);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_battery_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_battery_summary);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[]{mManager.getContext().getString(R.string.condition_turn_off)};
- }
-
- @Override
- public void onPrimaryClick() {
- new SubSettingLauncher(mManager.getContext())
- .setDestination(BatterySaverSettings.class.getName())
- .setSourceMetricsCategory(MetricsEvent.DASHBOARD_SUMMARY)
- .setTitleRes(R.string.battery_saver)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
- .launch();
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- BatterySaverUtils.setPowerSaveMode(mManager.getContext(), false,
- /*needFirstTimeWarning*/ false);
- refreshState();
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_BATTERY_SAVER;
- }
-
- @Override
- public void onResume() {
- mReceiver.setListening(true);
- }
-
- @Override
- public void onPause() {
- mReceiver.setListening(false);
- }
-
- @Override
- public void onPowerSaveModeChanged() {
- ConditionManager.get(mManager.getContext()).getCondition(BatterySaverCondition.class)
- .refreshState();
- }
-
- @Override
- public void onBatteryChanged(boolean pluggedIn) {
- // do nothing
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionCard.java b/src/com/android/settings/homepage/conditional/BatterySaverConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/BatterySaverConditionCard.java
rename to src/com/android/settings/homepage/conditional/BatterySaverConditionCard.java
index c30715f..ea84cb2 100644
--- a/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/BatterySaverConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionController.java b/src/com/android/settings/homepage/conditional/BatterySaverConditionController.java
similarity index 97%
rename from src/com/android/settings/homepage/conditional/v2/BatterySaverConditionController.java
rename to src/com/android/settings/homepage/conditional/BatterySaverConditionController.java
index 167dc12..fdbe698 100644
--- a/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionController.java
+++ b/src/com/android/settings/homepage/conditional/BatterySaverConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.os.PowerManager;
diff --git a/src/com/android/settings/homepage/conditional/CellularDataCondition.java b/src/com/android/settings/homepage/conditional/CellularDataCondition.java
deleted file mode 100644
index 4072912..0000000
--- a/src/com/android/settings/homepage/conditional/CellularDataCondition.java
+++ /dev/null
@@ -1,118 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.Intent;
-import android.content.IntentFilter;
-import android.graphics.drawable.Drawable;
-import android.net.ConnectivityManager;
-import android.telephony.TelephonyManager;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.internal.telephony.TelephonyIntents;
-import com.android.settings.R;
-import com.android.settings.Settings;
-
-@Deprecated
-public class CellularDataCondition extends Condition {
-
- private final Receiver mReceiver;
-
- private static final IntentFilter DATA_CONNECTION_FILTER =
- new IntentFilter(TelephonyIntents.ACTION_ANY_DATA_CONNECTION_STATE_CHANGED);
-
- public CellularDataCondition(ConditionManager manager) {
- super(manager);
- mReceiver = new Receiver();
- }
-
- @Override
- public void refreshState() {
- ConnectivityManager connectivity = mManager.getContext().getSystemService(
- ConnectivityManager.class);
- TelephonyManager telephony = mManager.getContext().getSystemService(TelephonyManager.class);
- if (!connectivity.isNetworkSupported(ConnectivityManager.TYPE_MOBILE)
- || telephony.getSimState() != TelephonyManager.SIM_STATE_READY) {
- setActive(false);
- return;
- }
- setActive(!telephony.isDataEnabled());
- }
-
- @Override
- protected BroadcastReceiver getReceiver() {
- return mReceiver;
- }
-
- @Override
- protected IntentFilter getIntentFilter() {
- return DATA_CONNECTION_FILTER;
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_cellular_off);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_cellular_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_cellular_summary);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[] { mManager.getContext().getString(R.string.condition_turn_on) };
- }
-
- @Override
- public void onPrimaryClick() {
- mManager.getContext().startActivity(new Intent(mManager.getContext(),
- Settings.DataUsageSummaryActivity.class).addFlags(Intent.FLAG_ACTIVITY_NEW_TASK));
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- TelephonyManager telephony = mManager.getContext().getSystemService(
- TelephonyManager.class);
- telephony.setDataEnabled(true);
- setActive(false);
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_CELLULAR_DATA;
- }
-
- public static class Receiver extends BroadcastReceiver {
- @Override
- public void onReceive(Context context, Intent intent) {
- if (TelephonyIntents.ACTION_ANY_DATA_CONNECTION_STATE_CHANGED.equals(
- intent.getAction())) {
- CellularDataCondition condition = ConditionManager.get(context).getCondition(
- CellularDataCondition.class);
- if (condition != null) {
- condition.refreshState();
- }
- }
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/CellularDataConditionCard.java b/src/com/android/settings/homepage/conditional/CellularDataConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/CellularDataConditionCard.java
rename to src/com/android/settings/homepage/conditional/CellularDataConditionCard.java
index c09e75a..f135c4f 100644
--- a/src/com/android/settings/homepage/conditional/v2/CellularDataConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/CellularDataConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/CellularDataConditionController.java b/src/com/android/settings/homepage/conditional/CellularDataConditionController.java
similarity index 98%
rename from src/com/android/settings/homepage/conditional/v2/CellularDataConditionController.java
rename to src/com/android/settings/homepage/conditional/CellularDataConditionController.java
index d23dc34..c5c92c2 100644
--- a/src/com/android/settings/homepage/conditional/v2/CellularDataConditionController.java
+++ b/src/com/android/settings/homepage/conditional/CellularDataConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.BroadcastReceiver;
import android.content.Context;
diff --git a/src/com/android/settings/homepage/conditional/Condition.java b/src/com/android/settings/homepage/conditional/Condition.java
deleted file mode 100644
index f91e422..0000000
--- a/src/com/android/settings/homepage/conditional/Condition.java
+++ /dev/null
@@ -1,162 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.IntentFilter;
-import android.graphics.drawable.Drawable;
-import android.os.PersistableBundle;
-
-import androidx.annotation.VisibleForTesting;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.overlay.FeatureFactory;
-import com.android.settingslib.core.instrumentation.MetricsFeatureProvider;
-
-public abstract class Condition {
-
- private static final String KEY_SILENCE = "silence";
- private static final String KEY_ACTIVE = "active";
- private static final String KEY_LAST_STATE = "last_state";
-
- protected final ConditionManager mManager;
- protected final MetricsFeatureProvider mMetricsFeatureProvider;
- protected boolean mReceiverRegistered;
-
- private boolean mIsSilenced;
- private boolean mIsActive;
- private long mLastStateChange;
-
- // All conditions must live in this package.
- Condition(ConditionManager manager) {
- this(manager, FeatureFactory.getFactory(manager.getContext()).getMetricsFeatureProvider());
- }
-
- Condition(ConditionManager manager, MetricsFeatureProvider metricsFeatureProvider) {
- mManager = manager;
- mMetricsFeatureProvider = metricsFeatureProvider;
- }
-
- void restoreState(PersistableBundle bundle) {
- mIsSilenced = bundle.getBoolean(KEY_SILENCE);
- mIsActive = bundle.getBoolean(KEY_ACTIVE);
- mLastStateChange = bundle.getLong(KEY_LAST_STATE);
- }
-
- boolean saveState(PersistableBundle bundle) {
- if (mIsSilenced) {
- bundle.putBoolean(KEY_SILENCE, mIsSilenced);
- }
- if (mIsActive) {
- bundle.putBoolean(KEY_ACTIVE, mIsActive);
- bundle.putLong(KEY_LAST_STATE, mLastStateChange);
- }
- return mIsSilenced || mIsActive;
- }
-
- protected void notifyChanged() {
- mManager.notifyChanged(this);
- }
-
- public boolean isSilenced() {
- return mIsSilenced;
- }
-
- public boolean isActive() {
- return mIsActive;
- }
-
- protected void setActive(boolean active) {
- if (mIsActive == active) {
- return;
- }
- mIsActive = active;
- mLastStateChange = System.currentTimeMillis();
- if (mIsSilenced && !active) {
- mIsSilenced = false;
- onSilenceChanged(mIsSilenced);
- }
- notifyChanged();
- }
-
- public void silence() {
- if (!mIsSilenced) {
- mIsSilenced = true;
- Context context = mManager.getContext();
- mMetricsFeatureProvider.action(context, MetricsEvent.ACTION_SETTINGS_CONDITION_DISMISS,
- getMetricsConstant());
- onSilenceChanged(mIsSilenced);
- notifyChanged();
- }
- }
-
- @VisibleForTesting
- void onSilenceChanged(boolean silenced) {
- final BroadcastReceiver receiver = getReceiver();
- if (receiver == null) {
- return;
- }
- if (silenced) {
- if (!mReceiverRegistered) {
- mManager.getContext().registerReceiver(receiver, getIntentFilter());
- mReceiverRegistered = true;
- }
- } else {
- if (mReceiverRegistered) {
- mManager.getContext().unregisterReceiver(receiver);
- mReceiverRegistered = false;
- }
- }
- }
-
- protected BroadcastReceiver getReceiver() {
- return null;
- }
-
- protected IntentFilter getIntentFilter() {
- return null;
- }
-
- public boolean shouldShow() {
- return isActive() && !isSilenced();
- }
-
- long getLastChange() {
- return mLastStateChange;
- }
-
- public void onResume() {
- }
-
- public void onPause() {
- }
-
- // State.
- public abstract void refreshState();
-
- public abstract int getMetricsConstant();
-
- // UI.
- public abstract Drawable getIcon();
- public abstract CharSequence getTitle();
- public abstract CharSequence getSummary();
- public abstract CharSequence[] getActions();
-
- public abstract void onPrimaryClick();
- public abstract void onActionClick(int index);
-}
diff --git a/src/com/android/settings/homepage/conditional/ConditionAdapter.java b/src/com/android/settings/homepage/conditional/ConditionAdapter.java
index e975146..a87e4bc 100644
--- a/src/com/android/settings/homepage/conditional/ConditionAdapter.java
+++ b/src/com/android/settings/homepage/conditional/ConditionAdapter.java
@@ -1,5 +1,5 @@
/*
- * Copyright (C) 2017 The Android Open Source Project
+ * Copyright (C) 2018 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.
@@ -13,77 +13,38 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
+
package com.android.settings.homepage.conditional;
import android.content.Context;
-import android.util.Log;
+import android.text.TextUtils;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;
-import androidx.annotation.VisibleForTesting;
-import androidx.recyclerview.widget.ItemTouchHelper;
import androidx.recyclerview.widget.RecyclerView;
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
+import com.android.internal.logging.nano.MetricsProto;
import com.android.settings.R;
-import com.android.settings.dashboard.DashboardAdapter.DashboardItemHolder;
+import com.android.settings.dashboard.DashboardAdapter;
import com.android.settings.overlay.FeatureFactory;
-import com.android.settingslib.WirelessUtils;
import com.android.settingslib.core.instrumentation.MetricsFeatureProvider;
import java.util.List;
-import java.util.Objects;
-public class ConditionAdapter extends RecyclerView.Adapter<DashboardItemHolder> {
- public static final String TAG = "ConditionAdapter";
+public class ConditionAdapter extends RecyclerView.Adapter<DashboardAdapter.DashboardItemHolder> {
private final Context mContext;
private final MetricsFeatureProvider mMetricsFeatureProvider;
- private List<Condition> mConditions;
- private boolean mExpanded;
+ private final ConditionManager mConditionManager;
+ private final List<ConditionalCard> mConditions;
+ private final boolean mExpanded;
- private View.OnClickListener mConditionClickListener = new View.OnClickListener() {
-
- @Override
- public void onClick(View v) {
- //TODO: get rid of setTag/getTag
- Condition condition = (Condition) v.getTag();
- mMetricsFeatureProvider.action(mContext,
- MetricsEvent.ACTION_SETTINGS_CONDITION_CLICK,
- condition.getMetricsConstant());
- condition.onPrimaryClick();
- }
- };
-
- @VisibleForTesting
- ItemTouchHelper.SimpleCallback mSwipeCallback = new ItemTouchHelper.SimpleCallback(0,
- ItemTouchHelper.START | ItemTouchHelper.END) {
- @Override
- public boolean onMove(RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder,
- RecyclerView.ViewHolder target) {
- return true;
- }
-
- @Override
- public int getSwipeDirs(RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder) {
- return viewHolder.getItemViewType() == R.layout.condition_tile
- ? super.getSwipeDirs(recyclerView, viewHolder) : 0;
- }
-
- @Override
- public void onSwiped(RecyclerView.ViewHolder viewHolder, int direction) {
- Object item = getItem(viewHolder.getItemId());
- // item can become null when running monkey
- if (item != null) {
- ((Condition) item).silence();
- }
- }
- };
-
- public ConditionAdapter(Context context, List<Condition> conditions, boolean expanded) {
+ public ConditionAdapter(Context context, ConditionManager conditionManager,
+ List<ConditionalCard> conditions, boolean expanded) {
mContext = context;
+ mConditionManager = conditionManager;
mConditions = conditions;
mExpanded = expanded;
mMetricsFeatureProvider = FeatureFactory.getFactory(context).getMetricsFeatureProvider();
@@ -91,30 +52,22 @@
setHasStableIds(true);
}
- public Object getItem(long itemId) {
- for (Condition condition : mConditions) {
- if (Objects.hash(condition.getTitle()) == itemId) {
- return condition;
- }
- }
- return null;
+ @Override
+ public DashboardAdapter.DashboardItemHolder onCreateViewHolder(ViewGroup parent, int viewType) {
+ return new DashboardAdapter.DashboardItemHolder(LayoutInflater.from(parent.getContext())
+ .inflate(viewType, parent, false));
}
@Override
- public DashboardItemHolder onCreateViewHolder(ViewGroup parent, int viewType) {
- return new DashboardItemHolder(LayoutInflater.from(parent.getContext()).inflate(
- viewType, parent, false));
- }
-
- @Override
- public void onBindViewHolder(DashboardItemHolder holder, int position) {
- bindViews(mConditions.get(position), holder,
- position == mConditions.size() - 1, mConditionClickListener);
+ public void onBindViewHolder(DashboardAdapter.DashboardItemHolder holder, int position) {
+ final ConditionalCard condition = mConditions.get(position);
+ final boolean isLastItem = position == mConditions.size() - 1;
+ bindViews(condition, holder, isLastItem);
}
@Override
public long getItemId(int position) {
- return Objects.hash(mConditions.get(position).getTitle());
+ return mConditions.get(position).getId();
}
@Override
@@ -130,52 +83,42 @@
return 0;
}
- public void addDismissHandling(final RecyclerView recyclerView) {
- final ItemTouchHelper itemTouchHelper = new ItemTouchHelper(mSwipeCallback);
- itemTouchHelper.attachToRecyclerView(recyclerView);
- }
-
- private void bindViews(final Condition condition,
- DashboardItemHolder view, boolean isLastItem,
- View.OnClickListener onClickListener) {
- if (condition instanceof AirplaneModeCondition) {
- Log.d(TAG, "Airplane mode condition has been bound with "
- + "isActive=" + condition.isActive() + ". Airplane mode is currently " +
- WirelessUtils.isAirplaneModeOn(condition.mManager.getContext()));
- }
- View card = view.itemView.findViewById(R.id.content);
- card.setTag(condition);
- card.setOnClickListener(onClickListener);
+ private void bindViews(final ConditionalCard condition,
+ DashboardAdapter.DashboardItemHolder view, boolean isLastItem) {
+ mMetricsFeatureProvider.visible(mContext, MetricsProto.MetricsEvent.DASHBOARD_SUMMARY,
+ condition.getMetricsConstant());
+ view.itemView.findViewById(R.id.content).setOnClickListener(
+ v -> {
+ mMetricsFeatureProvider.action(mContext,
+ MetricsProto.MetricsEvent.ACTION_SETTINGS_CONDITION_CLICK,
+ condition.getMetricsConstant());
+ mConditionManager.onPrimaryClick(mContext, condition.getId());
+ });
view.icon.setImageDrawable(condition.getIcon());
view.title.setText(condition.getTitle());
+ view.summary.setText(condition.getSummary());
- CharSequence[] actions = condition.getActions();
- final boolean hasButtons = actions.length > 0;
+ setViewVisibility(view.itemView, R.id.divider, !isLastItem);
+
+ final CharSequence action = condition.getActionText();
+ final boolean hasButtons = !TextUtils.isEmpty(action);
setViewVisibility(view.itemView, R.id.buttonBar, hasButtons);
- view.summary.setText(condition.getSummary());
- for (int i = 0; i < 2; i++) {
- Button button = (Button) view.itemView.findViewById(i == 0
- ? R.id.first_action : R.id.second_action);
- if (actions.length > i) {
- button.setVisibility(View.VISIBLE);
- button.setText(actions[i]);
- final int index = i;
- button.setOnClickListener(new View.OnClickListener() {
- @Override
- public void onClick(View v) {
- Context context = v.getContext();
- FeatureFactory.getFactory(context).getMetricsFeatureProvider()
- .action(context, MetricsEvent.ACTION_SETTINGS_CONDITION_BUTTON,
- condition.getMetricsConstant());
- condition.onActionClick(index);
- }
- });
- } else {
- button.setVisibility(View.GONE);
- }
+ final Button button = view.itemView.findViewById(R.id.first_action);
+ if (hasButtons) {
+ button.setVisibility(View.VISIBLE);
+ button.setText(action);
+ button.setOnClickListener(v -> {
+ final Context context = v.getContext();
+ mMetricsFeatureProvider.action(
+ context, MetricsProto.MetricsEvent.ACTION_SETTINGS_CONDITION_BUTTON,
+ condition.getMetricsConstant());
+ mConditionManager.onActionClick(condition.getId());
+ });
+ } else {
+ button.setVisibility(View.GONE);
}
- setViewVisibility(view.itemView, R.id.divider, !isLastItem);
+
}
private void setViewVisibility(View containerView, int viewId, boolean visible) {
diff --git a/src/com/android/settings/homepage/conditional/ConditionManager.java b/src/com/android/settings/homepage/conditional/ConditionManager.java
index dbb04ae..d6e5012 100644
--- a/src/com/android/settings/homepage/conditional/ConditionManager.java
+++ b/src/com/android/settings/homepage/conditional/ConditionManager.java
@@ -1,5 +1,5 @@
/*
- * Copyright (C) 2015 The Android Open Source Project
+ * Copyright (C) 2018 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.
@@ -13,279 +13,187 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
+
package com.android.settings.homepage.conditional;
import android.content.Context;
-import android.os.AsyncTask;
-import android.os.PersistableBundle;
import android.util.Log;
-import android.util.Xml;
-import com.android.settingslib.core.lifecycle.LifecycleObserver;
-import com.android.settingslib.core.lifecycle.events.OnPause;
-import com.android.settingslib.core.lifecycle.events.OnResume;
+import androidx.annotation.NonNull;
+import androidx.annotation.VisibleForTesting;
-import org.xmlpull.v1.XmlPullParser;
-import org.xmlpull.v1.XmlPullParserException;
-import org.xmlpull.v1.XmlSerializer;
-
-import java.io.File;
-import java.io.FileReader;
-import java.io.FileWriter;
-import java.io.IOException;
import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Comparator;
import java.util.List;
+import java.util.concurrent.Callable;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.Future;
+import java.util.concurrent.TimeUnit;
+import java.util.concurrent.TimeoutException;
-public class ConditionManager implements LifecycleObserver, OnResume, OnPause {
-
+public class ConditionManager {
private static final String TAG = "ConditionManager";
- private static final boolean DEBUG = false;
+ @VisibleForTesting
+ final List<ConditionalCard> mCandidates;
+ @VisibleForTesting
+ final List<ConditionalCardController> mCardControllers;
- private static final String PKG = "com.android.settings.homepage.conditional.";
+ private static final long DISPLAYABLE_CHECKER_TIMEOUT_MS = 20;
- private static final String FILE_NAME = "condition_state.xml";
- private static final String TAG_CONDITIONS = "cs";
- private static final String TAG_CONDITION = "c";
- private static final String ATTR_CLASS = "cls";
+ private final ExecutorService mExecutorService;
+ private final Context mAppContext;
+ private final ConditionListener mListener;
- private static ConditionManager sInstance;
+ private boolean mIsListeningToStateChange;
- private final Context mContext;
- private final ArrayList<Condition> mConditions;
- private File mXmlFile;
-
- private final ArrayList<ConditionListener> mListeners = new ArrayList<>();
-
- private ConditionManager(Context context, boolean loadConditionsNow) {
- mContext = context;
- mConditions = new ArrayList<>();
- if (loadConditionsNow) {
- Log.d(TAG, "conditions loading synchronously");
- ConditionLoader loader = new ConditionLoader();
- loader.onPostExecute(loader.doInBackground());
- } else {
- Log.d(TAG, "conditions loading asychronously");
- new ConditionLoader().execute();
- }
+ public ConditionManager(Context context, ConditionListener listener) {
+ mAppContext = context.getApplicationContext();
+ mExecutorService = Executors.newCachedThreadPool();
+ mCandidates = new ArrayList<>();
+ mCardControllers = new ArrayList<>();
+ mListener = listener;
+ initCandidates();
}
- public void refreshAll() {
- final int N = mConditions.size();
- for (int i = 0; i < N; i++) {
- mConditions.get(i).refreshState();
+ /**
+ * Returns a list of {@link ConditionalCard}s eligible for display.
+ */
+ public List<ConditionalCard> getDisplayableCards() {
+ final List<ConditionalCard> cards = new ArrayList<>();
+ final List<Future<ConditionalCard>> displayableCards = new ArrayList<>();
+ // Check displayable future
+ for (ConditionalCard card : mCandidates) {
+ final DisplayableChecker future = new DisplayableChecker(
+ card, getController(card.getId()));
+ displayableCards.add(mExecutorService.submit(future));
}
- }
-
- private void readFromXml(File xmlFile, ArrayList<Condition> conditions) {
- if (DEBUG) Log.d(TAG, "Reading from " + xmlFile.toString());
- try {
- XmlPullParser parser = Xml.newPullParser();
- FileReader in = new FileReader(xmlFile);
- parser.setInput(in);
- int state = parser.getEventType();
-
- while (state != XmlPullParser.END_DOCUMENT) {
- if (TAG_CONDITION.equals(parser.getName())) {
- int depth = parser.getDepth();
- String clz = parser.getAttributeValue("", ATTR_CLASS);
- if (!clz.startsWith(PKG)) {
- clz = PKG + clz;
- }
- Condition condition = createCondition(Class.forName(clz));
- PersistableBundle bundle = PersistableBundle.restoreFromXml(parser);
- if (DEBUG) Log.d(TAG, "Reading " + clz + " -- " + bundle);
- if (condition != null) {
- condition.restoreState(bundle);
- conditions.add(condition);
- } else {
- Log.e(TAG, "failed to add condition: " + clz);
- }
- while (parser.getDepth() > depth) {
- parser.next();
- }
+ // Collect future and add displayable cards
+ for (Future<ConditionalCard> cardFuture : displayableCards) {
+ try {
+ final ConditionalCard card = cardFuture.get(DISPLAYABLE_CHECKER_TIMEOUT_MS,
+ TimeUnit.MILLISECONDS);
+ if (card != null) {
+ cards.add(card);
}
- state = parser.next();
- }
- in.close();
- } catch (XmlPullParserException | IOException | ClassNotFoundException e) {
- Log.w(TAG, "Problem reading " + FILE_NAME, e);
- }
- }
-
- private void saveToXml() {
- if (DEBUG) Log.d(TAG, "Writing to " + mXmlFile.toString());
- try {
- XmlSerializer serializer = Xml.newSerializer();
- FileWriter writer = new FileWriter(mXmlFile);
- serializer.setOutput(writer);
-
- serializer.startDocument("UTF-8", true);
- serializer.startTag("", TAG_CONDITIONS);
-
- final int N = mConditions.size();
- for (int i = 0; i < N; i++) {
- PersistableBundle bundle = new PersistableBundle();
- if (mConditions.get(i).saveState(bundle)) {
- serializer.startTag("", TAG_CONDITION);
- final String clz = mConditions.get(i).getClass().getSimpleName();
- serializer.attribute("", ATTR_CLASS, clz);
- bundle.saveToXml(serializer);
- serializer.endTag("", TAG_CONDITION);
- }
- }
-
- serializer.endTag("", TAG_CONDITIONS);
- serializer.flush();
- writer.close();
- } catch (XmlPullParserException | IOException e) {
- Log.w(TAG, "Problem writing " + FILE_NAME, e);
- }
- }
-
- private void addMissingConditions(ArrayList<Condition> conditions) {
- addIfMissing(AirplaneModeCondition.class, conditions);
- addIfMissing(HotspotCondition.class, conditions);
- addIfMissing(DndCondition.class, conditions);
- addIfMissing(BatterySaverCondition.class, conditions);
- addIfMissing(CellularDataCondition.class, conditions);
- addIfMissing(BackgroundDataCondition.class, conditions);
- addIfMissing(WorkModeCondition.class, conditions);
- addIfMissing(NightDisplayCondition.class, conditions);
- addIfMissing(RingerMutedCondition.class, conditions);
- addIfMissing(RingerVibrateCondition.class, conditions);
- Collections.sort(conditions, CONDITION_COMPARATOR);
- }
-
- private void addIfMissing(Class<? extends Condition> clz, ArrayList<Condition> conditions) {
- if (getCondition(clz, conditions) == null) {
- if (DEBUG) Log.d(TAG, "Adding missing " + clz.getName());
- Condition condition = createCondition(clz);
- if (condition != null) {
- conditions.add(condition);
+ } catch (InterruptedException | ExecutionException | TimeoutException e) {
+ Log.w(TAG, "Failed to get displayable state for card, likely timeout. Skipping", e);
}
}
+ return cards;
}
- private Condition createCondition(Class<?> clz) {
- if (AirplaneModeCondition.class == clz) {
- return new AirplaneModeCondition(this);
- } else if (HotspotCondition.class == clz) {
- return new HotspotCondition(this);
- } else if (DndCondition.class == clz) {
- return new DndCondition(this);
- } else if (BatterySaverCondition.class == clz) {
- return new BatterySaverCondition(this);
- } else if (CellularDataCondition.class == clz) {
- return new CellularDataCondition(this);
- } else if (BackgroundDataCondition.class == clz) {
- return new BackgroundDataCondition(this);
- } else if (WorkModeCondition.class == clz) {
- return new WorkModeCondition(this);
- } else if (NightDisplayCondition.class == clz) {
- return new NightDisplayCondition(this);
- } else if (RingerMutedCondition.class == clz) {
- return new RingerMutedCondition(this);
- } else if (RingerVibrateCondition.class == clz) {
- return new RingerVibrateCondition(this);
+ /**
+ * Handler when the card is clicked.
+ *
+ * @see {@link ConditionalCardController#onPrimaryClick(Context)}
+ */
+ public void onPrimaryClick(Context context, long id) {
+ getController(id).onPrimaryClick(context);
+ }
+
+ /**
+ * Handler when the card action is clicked.
+ *
+ * @see {@link ConditionalCardController#onActionClick()}
+ */
+ public void onActionClick(long id) {
+ getController(id).onActionClick();
+ onConditionChanged();
+ }
+
+ /**
+ * Start monitoring state change for all conditions
+ */
+ public void startMonitoringStateChange() {
+ if (mIsListeningToStateChange) {
+ Log.d(TAG, "Already listening to condition state changes, skipping");
+ return;
}
- Log.e(TAG, "unknown condition class: " + clz.getSimpleName());
- return null;
+ mIsListeningToStateChange = true;
+ for (ConditionalCardController controller : mCardControllers) {
+ controller.startMonitoringStateChange();
+ }
+ // Force a refresh on listener
+ onConditionChanged();
}
- Context getContext() {
- return mContext;
+ /**
+ * Stop monitoring state change for all conditions
+ */
+ public void stopMonitoringStateChange() {
+ if (!mIsListeningToStateChange) {
+ Log.d(TAG, "Not listening to condition state changes, skipping");
+ return;
+ }
+ for (ConditionalCardController controller : mCardControllers) {
+ controller.stopMonitoringStateChange();
+ }
+ mIsListeningToStateChange = false;
}
- public <T extends Condition> T getCondition(Class<T> clz) {
- return getCondition(clz, mConditions);
+ /**
+ * Called when some conditional card's state has changed
+ */
+ void onConditionChanged() {
+ if (mListener != null) {
+ mListener.onConditionsChanged();
+ }
}
- private <T extends Condition> T getCondition(Class<T> clz, List<Condition> conditions) {
- final int N = conditions.size();
- for (int i = 0; i < N; i++) {
- if (clz.equals(conditions.get(i).getClass())) {
- return (T) conditions.get(i);
+ @NonNull
+ <T extends ConditionalCardController> T getController(long id) {
+ for (ConditionalCardController controller : mCardControllers) {
+ if (controller.getId() == id) {
+ return (T) controller;
}
}
- return null;
+ throw new IllegalStateException("Cannot find controller for " + id);
}
- public List<Condition> getConditions() {
- return mConditions;
+ private void initCandidates() {
+ // Initialize controllers first.
+ mCardControllers.add(new AirplaneModeConditionController(mAppContext, this /* manager */));
+ mCardControllers.add(new BackgroundDataConditionController(mAppContext));
+ mCardControllers.add(new BatterySaverConditionController(mAppContext, this /* manager */));
+ mCardControllers.add(new CellularDataConditionController(mAppContext, this /* manager */));
+ mCardControllers.add(new DndConditionCardController(mAppContext, this /* manager */));
+ mCardControllers.add(new HotspotConditionController(mAppContext, this /* manager */));
+ mCardControllers.add(new NightDisplayConditionController(mAppContext));
+ mCardControllers.add(new RingerVibrateConditionController(mAppContext, this /* manager */));
+ mCardControllers.add(new RingerMutedConditionController(mAppContext, this /* manager */));
+ mCardControllers.add(new WorkModeConditionController(mAppContext));
+
+ // Initialize ui model later. UI model depends on controller.
+ mCandidates.add(new AirplaneModeConditionCard(mAppContext));
+ mCandidates.add(new BackgroundDataConditionCard(mAppContext));
+ mCandidates.add(new BatterySaverConditionCard(mAppContext));
+ mCandidates.add(new CellularDataConditionCard(mAppContext));
+ mCandidates.add(new DndConditionCard(mAppContext, this /* manager */));
+ mCandidates.add(new HotspotConditionCard(mAppContext, this /* manager */));
+ mCandidates.add(new NightDisplayConditionCard(mAppContext));
+ mCandidates.add(new RingerMutedConditionCard(mAppContext));
+ mCandidates.add(new RingerVibrateConditionCard(mAppContext));
+ mCandidates.add(new WorkModeConditionCard(mAppContext));
}
- public void notifyChanged(Condition condition) {
- saveToXml();
- Collections.sort(mConditions, CONDITION_COMPARATOR);
- final int N = mListeners.size();
- for (int i = 0; i < N; i++) {
- mListeners.get(i).onConditionsChanged();
- }
- }
+ /**
+ * Returns card if controller says it's displayable. Otherwise returns null.
+ */
+ public static class DisplayableChecker implements Callable<ConditionalCard> {
- public void addListener(ConditionListener listener) {
- mListeners.add(listener);
- listener.onConditionsChanged();
- }
+ private final ConditionalCard mCard;
+ private final ConditionalCardController mController;
- public void remListener(ConditionListener listener) {
- mListeners.remove(listener);
- }
-
- @Override
- public void onResume() {
- for (int i = 0, size = mConditions.size(); i < size; i++) {
- mConditions.get(i).onResume();
- }
- }
-
- @Override
- public void onPause() {
- for (int i = 0, size = mConditions.size(); i < size; i++) {
- mConditions.get(i).onPause();
- }
- }
-
- private class ConditionLoader extends AsyncTask<Void, Void, ArrayList<Condition>> {
- @Override
- protected ArrayList<Condition> doInBackground(Void... params) {
- Log.d(TAG, "loading conditions from xml");
- ArrayList<Condition> conditions = new ArrayList<>();
- mXmlFile = new File(mContext.getFilesDir(), FILE_NAME);
- if (mXmlFile.exists()) {
- readFromXml(mXmlFile, conditions);
- }
- addMissingConditions(conditions);
- return conditions;
+ private DisplayableChecker(ConditionalCard card, ConditionalCardController controller) {
+ mCard = card;
+ mController = controller;
}
@Override
- protected void onPostExecute(ArrayList<Condition> conditions) {
- Log.d(TAG, "conditions loaded from xml, refreshing conditions");
- mConditions.clear();
- mConditions.addAll(conditions);
- refreshAll();
+ public ConditionalCard call() throws Exception {
+ return mController.isDisplayable() ? mCard : null;
}
}
-
- public static ConditionManager get(Context context) {
- return get(context, true);
- }
-
- public static ConditionManager get(Context context, boolean loadConditionsNow) {
- if (sInstance == null) {
- sInstance = new ConditionManager(context.getApplicationContext(), loadConditionsNow);
- }
- return sInstance;
- }
-
- private static final Comparator<Condition> CONDITION_COMPARATOR = new Comparator<Condition>() {
- @Override
- public int compare(Condition lhs, Condition rhs) {
- return Long.compare(lhs.getLastChange(), rhs.getLastChange());
- }
- };
}
diff --git a/src/com/android/settings/homepage/conditional/v2/ConditionalCard.java b/src/com/android/settings/homepage/conditional/ConditionalCard.java
similarity index 95%
rename from src/com/android/settings/homepage/conditional/v2/ConditionalCard.java
rename to src/com/android/settings/homepage/conditional/ConditionalCard.java
index da832b7..f4c4547 100644
--- a/src/com/android/settings/homepage/conditional/v2/ConditionalCard.java
+++ b/src/com/android/settings/homepage/conditional/ConditionalCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/ConditionalCardController.java b/src/com/android/settings/homepage/conditional/ConditionalCardController.java
similarity index 95%
rename from src/com/android/settings/homepage/conditional/v2/ConditionalCardController.java
rename to src/com/android/settings/homepage/conditional/ConditionalCardController.java
index 7919d73..9993f34 100644
--- a/src/com/android/settings/homepage/conditional/v2/ConditionalCardController.java
+++ b/src/com/android/settings/homepage/conditional/ConditionalCardController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
diff --git a/src/com/android/settings/homepage/conditional/DndCondition.java b/src/com/android/settings/homepage/conditional/DndCondition.java
deleted file mode 100644
index d238613..0000000
--- a/src/com/android/settings/homepage/conditional/DndCondition.java
+++ /dev/null
@@ -1,166 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.app.NotificationManager;
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.Intent;
-import android.content.IntentFilter;
-import android.graphics.drawable.Drawable;
-import android.os.PersistableBundle;
-import android.provider.Settings;
-import android.provider.Settings.Global;
-import android.service.notification.ZenModeConfig;
-
-import androidx.annotation.VisibleForTesting;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settings.core.SubSettingLauncher;
-import com.android.settings.notification.ZenModeSettings;
-
-/**
- * @deprecated in favor of {@link com.android.settings.homepage.conditional.v2.DndConditionCard}
- */
-@Deprecated
-public class DndCondition extends Condition {
-
- private static final String TAG = "DndCondition";
- private static final String KEY_STATE = "state";
-
- private boolean mRegistered;
-
- @VisibleForTesting
- static final IntentFilter DND_FILTER =
- new IntentFilter(NotificationManager.ACTION_INTERRUPTION_FILTER_CHANGED_INTERNAL);
- @VisibleForTesting
- protected ZenModeConfig mConfig;
-
- private int mZen;
- private final Receiver mReceiver;
-
- public DndCondition(ConditionManager manager) {
- super(manager);
- mReceiver = new Receiver();
- mManager.getContext().registerReceiver(mReceiver, DND_FILTER);
- mRegistered = true;
- }
-
- @Override
- public void refreshState() {
- NotificationManager notificationManager =
- mManager.getContext().getSystemService(NotificationManager.class);
- mZen = notificationManager.getZenMode();
- boolean zenModeEnabled = mZen != Settings.Global.ZEN_MODE_OFF;
- if (zenModeEnabled) {
- mConfig = notificationManager.getZenModeConfig();
- } else {
- mConfig = null;
- }
- setActive(zenModeEnabled);
- }
-
- @Override
- boolean saveState(PersistableBundle bundle) {
- bundle.putInt(KEY_STATE, mZen);
- return super.saveState(bundle);
- }
-
- @Override
- void restoreState(PersistableBundle bundle) {
- super.restoreState(bundle);
- mZen = bundle.getInt(KEY_STATE, Global.ZEN_MODE_OFF);
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_do_not_disturb_on_24dp);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_zen_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return ZenModeConfig.getDescription(mManager.getContext(), mZen != Global.ZEN_MODE_OFF,
- mConfig, true);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[] { mManager.getContext().getString(R.string.condition_turn_off) };
- }
-
- @Override
- public void onPrimaryClick() {
- new SubSettingLauncher(mManager.getContext())
- .setDestination(ZenModeSettings.class.getName())
- .setSourceMetricsCategory(MetricsEvent.DASHBOARD_SUMMARY)
- .setTitleRes(R.string.zen_mode_settings_title)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
- .launch();
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- NotificationManager notificationManager = mManager.getContext().getSystemService(
- NotificationManager.class);
- notificationManager.setZenMode(Settings.Global.ZEN_MODE_OFF, null, TAG);
- setActive(false);
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_DND;
- }
-
- public static class Receiver extends BroadcastReceiver {
- @Override
- public void onReceive(Context context, Intent intent) {
- if (NotificationManager.ACTION_INTERRUPTION_FILTER_CHANGED_INTERNAL
- .equals(intent.getAction())) {
- final Condition condition =
- ConditionManager.get(context).getCondition(DndCondition.class);
- if (condition != null) {
- condition.refreshState();
- }
- }
- }
- }
-
- @Override
- public void onResume() {
- if (!mRegistered) {
- mManager.getContext().registerReceiver(mReceiver, DND_FILTER);
- mRegistered = true;
- }
- }
-
- @Override
- public void onPause() {
- if (mRegistered) {
- mManager.getContext().unregisterReceiver(mReceiver);
- mRegistered = false;
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/DndConditionCard.java b/src/com/android/settings/homepage/conditional/DndConditionCard.java
similarity index 93%
rename from src/com/android/settings/homepage/conditional/v2/DndConditionCard.java
rename to src/com/android/settings/homepage/conditional/DndConditionCard.java
index 8e0fdd5..dbc0d83 100644
--- a/src/com/android/settings/homepage/conditional/v2/DndConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/DndConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
@@ -25,12 +25,10 @@
public class DndConditionCard implements ConditionalCard {
private final Context mAppContext;
- private final ConditionManager mManager;
private final DndConditionCardController mController;
public DndConditionCard(Context appContext, ConditionManager manager) {
mAppContext = appContext;
- mManager = manager;
mController = manager.getController(getId());
}
diff --git a/src/com/android/settings/homepage/conditional/v2/DndConditionCardController.java b/src/com/android/settings/homepage/conditional/DndConditionCardController.java
similarity index 98%
rename from src/com/android/settings/homepage/conditional/v2/DndConditionCardController.java
rename to src/com/android/settings/homepage/conditional/DndConditionCardController.java
index 9e51ef9..1729815 100644
--- a/src/com/android/settings/homepage/conditional/v2/DndConditionCardController.java
+++ b/src/com/android/settings/homepage/conditional/DndConditionCardController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.app.NotificationManager;
import android.content.BroadcastReceiver;
diff --git a/src/com/android/settings/homepage/conditional/HotspotCondition.java b/src/com/android/settings/homepage/conditional/HotspotCondition.java
deleted file mode 100644
index 6934d0f..0000000
--- a/src/com/android/settings/homepage/conditional/HotspotCondition.java
+++ /dev/null
@@ -1,145 +0,0 @@
-/*
- * Copyright (C) 2015 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.homepage.conditional;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.Intent;
-import android.content.IntentFilter;
-import android.graphics.drawable.Drawable;
-import android.net.ConnectivityManager;
-import android.net.wifi.WifiConfiguration;
-import android.net.wifi.WifiManager;
-import android.os.UserHandle;
-import android.os.UserManager;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settings.TetherSettings;
-import com.android.settings.core.SubSettingLauncher;
-import com.android.settingslib.RestrictedLockUtils;
-import com.android.settingslib.RestrictedLockUtils.EnforcedAdmin;
-
-@Deprecated
-public class HotspotCondition extends Condition {
-
- private final WifiManager mWifiManager;
- private final Receiver mReceiver;
-
- private static final IntentFilter WIFI_AP_STATE_FILTER =
- new IntentFilter(WifiManager.WIFI_AP_STATE_CHANGED_ACTION);
-
- public HotspotCondition(ConditionManager manager) {
- super(manager);
- mWifiManager = mManager.getContext().getSystemService(WifiManager.class);
- mReceiver = new Receiver();
- }
-
- @Override
- public void refreshState() {
- boolean wifiTetherEnabled = mWifiManager.isWifiApEnabled();
- setActive(wifiTetherEnabled);
- }
-
- @Override
- protected BroadcastReceiver getReceiver() {
- return mReceiver;
- }
-
- @Override
- protected IntentFilter getIntentFilter() {
- return WIFI_AP_STATE_FILTER;
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_hotspot);
- }
-
- private String getSsid() {
- WifiConfiguration wifiConfig = mWifiManager.getWifiApConfiguration();
- if (wifiConfig == null) {
- return mManager.getContext().getString(
- com.android.internal.R.string.wifi_tether_configure_ssid_default);
- } else {
- return wifiConfig.SSID;
- }
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_hotspot_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_hotspot_summary, getSsid());
- }
-
- @Override
- public CharSequence[] getActions() {
- final Context context = mManager.getContext();
- if (RestrictedLockUtils.hasBaseUserRestriction(context,
- UserManager.DISALLOW_CONFIG_TETHERING, UserHandle.myUserId())) {
- return new CharSequence[0];
- }
- return new CharSequence[]{context.getString(R.string.condition_turn_off)};
- }
-
- @Override
- public void onPrimaryClick() {
- new SubSettingLauncher(mManager.getContext())
- .setDestination(TetherSettings.class.getName())
- .setSourceMetricsCategory(MetricsEvent.DASHBOARD_SUMMARY)
- .setTitleRes(R.string.tether_settings_title_all)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
- .launch();
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- final Context context = mManager.getContext();
- final EnforcedAdmin admin = RestrictedLockUtils.checkIfRestrictionEnforced(context,
- UserManager.DISALLOW_CONFIG_TETHERING, UserHandle.myUserId());
- if (admin != null) {
- RestrictedLockUtils.sendShowAdminSupportDetailsIntent(context, admin);
- } else {
- ConnectivityManager cm = (ConnectivityManager) context.getSystemService(
- Context.CONNECTIVITY_SERVICE);
- cm.stopTethering(ConnectivityManager.TETHERING_WIFI);
- setActive(false);
- }
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_HOTSPOT;
- }
-
- public static class Receiver extends BroadcastReceiver {
- @Override
- public void onReceive(Context context, Intent intent) {
- if (WifiManager.WIFI_AP_STATE_CHANGED_ACTION.equals(intent.getAction())) {
- ConditionManager.get(context).getCondition(HotspotCondition.class)
- .refreshState();
- }
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/HotspotConditionCard.java b/src/com/android/settings/homepage/conditional/HotspotConditionCard.java
similarity index 97%
rename from src/com/android/settings/homepage/conditional/v2/HotspotConditionCard.java
rename to src/com/android/settings/homepage/conditional/HotspotConditionCard.java
index 63b5034..a6ff224 100644
--- a/src/com/android/settings/homepage/conditional/v2/HotspotConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/HotspotConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/HotspotConditionController.java b/src/com/android/settings/homepage/conditional/HotspotConditionController.java
similarity index 98%
rename from src/com/android/settings/homepage/conditional/v2/HotspotConditionController.java
rename to src/com/android/settings/homepage/conditional/HotspotConditionController.java
index 56e95d1..4271136 100644
--- a/src/com/android/settings/homepage/conditional/v2/HotspotConditionController.java
+++ b/src/com/android/settings/homepage/conditional/HotspotConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.BroadcastReceiver;
import android.content.Context;
diff --git a/src/com/android/settings/homepage/conditional/NightDisplayCondition.java b/src/com/android/settings/homepage/conditional/NightDisplayCondition.java
deleted file mode 100644
index 384f593..0000000
--- a/src/com/android/settings/homepage/conditional/NightDisplayCondition.java
+++ /dev/null
@@ -1,96 +0,0 @@
-/*
- * Copyright (C) 2016 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.homepage.conditional;
-
-import android.content.Intent;
-import android.graphics.drawable.Drawable;
-
-import com.android.internal.app.ColorDisplayController;
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settings.core.SubSettingLauncher;
-import com.android.settings.display.NightDisplaySettings;
-
-/**
- * @deprecated in favor of {@link NightDisplayConditionController}
- */
-@Deprecated
-public final class NightDisplayCondition extends Condition
- implements ColorDisplayController.Callback {
-
- private ColorDisplayController mController;
-
- NightDisplayCondition(ConditionManager manager) {
- super(manager);
- mController = new ColorDisplayController(manager.getContext());
- mController.setListener(this);
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_NIGHT_DISPLAY;
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_settings_night_display);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_night_display_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_night_display_summary);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[] {mManager.getContext().getString(R.string.condition_turn_off)};
- }
-
- @Override
- public void onPrimaryClick() {
- new SubSettingLauncher(mManager.getContext())
- .setDestination(NightDisplaySettings.class.getName())
- .setSourceMetricsCategory(MetricsEvent.DASHBOARD_SUMMARY)
- .setTitleRes(R.string.night_display_title)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
- .launch();
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- mController.setActivated(false);
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public void refreshState() {
- setActive(mController.isActivated());
- }
-
- @Override
- public void onActivated(boolean activated) {
- refreshState();
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/NightDisplayConditionCard.java b/src/com/android/settings/homepage/conditional/NightDisplayConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/NightDisplayConditionCard.java
rename to src/com/android/settings/homepage/conditional/NightDisplayConditionCard.java
index b08f8e7..e0c6ee0 100644
--- a/src/com/android/settings/homepage/conditional/v2/NightDisplayConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/NightDisplayConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/NightDisplayConditionController.java b/src/com/android/settings/homepage/conditional/NightDisplayConditionController.java
similarity index 97%
rename from src/com/android/settings/homepage/conditional/v2/NightDisplayConditionController.java
rename to src/com/android/settings/homepage/conditional/NightDisplayConditionController.java
index 0d61218..428fe48 100644
--- a/src/com/android/settings/homepage/conditional/v2/NightDisplayConditionController.java
+++ b/src/com/android/settings/homepage/conditional/NightDisplayConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
diff --git a/src/com/android/settings/homepage/conditional/RingerMutedCondition.java b/src/com/android/settings/homepage/conditional/RingerMutedCondition.java
deleted file mode 100644
index d3e618e..0000000
--- a/src/com/android/settings/homepage/conditional/RingerMutedCondition.java
+++ /dev/null
@@ -1,71 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional;
-
-import static android.content.Context.NOTIFICATION_SERVICE;
-
-import android.app.NotificationManager;
-import android.graphics.drawable.Drawable;
-import android.media.AudioManager;
-import android.provider.Settings;
-
-import com.android.internal.logging.nano.MetricsProto;
-import com.android.settings.R;
-
-@Deprecated
-public class RingerMutedCondition extends AbnormalRingerConditionBase {
-
- private final NotificationManager mNotificationManager;
-
- RingerMutedCondition(ConditionManager manager) {
- super(manager);
- mNotificationManager =
- (NotificationManager) mManager.getContext().getSystemService(NOTIFICATION_SERVICE);
- }
-
- @Override
- public void refreshState() {
- int zen = Settings.Global.ZEN_MODE_OFF;
- if (mNotificationManager != null) {
- zen = mNotificationManager.getZenMode();
- }
- final boolean zenModeEnabled = zen != Settings.Global.ZEN_MODE_OFF;
- final boolean isSilent =
- mAudioManager.getRingerModeInternal() == AudioManager.RINGER_MODE_SILENT;
- setActive(isSilent && !zenModeEnabled);
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsProto.MetricsEvent.SETTINGS_CONDITION_DEVICE_MUTED;
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_notifications_off_24dp);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getText(R.string.condition_device_muted_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getText(R.string.condition_device_muted_summary);
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/RingerMutedConditionCard.java b/src/com/android/settings/homepage/conditional/RingerMutedConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/RingerMutedConditionCard.java
rename to src/com/android/settings/homepage/conditional/RingerMutedConditionCard.java
index c9a1ba0..47d6407 100644
--- a/src/com/android/settings/homepage/conditional/v2/RingerMutedConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/RingerMutedConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/RingerMutedConditionController.java b/src/com/android/settings/homepage/conditional/RingerMutedConditionController.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/RingerMutedConditionController.java
rename to src/com/android/settings/homepage/conditional/RingerMutedConditionController.java
index 19850ae..4fce1ec 100644
--- a/src/com/android/settings/homepage/conditional/v2/RingerMutedConditionController.java
+++ b/src/com/android/settings/homepage/conditional/RingerMutedConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static android.content.Context.NOTIFICATION_SERVICE;
diff --git a/src/com/android/settings/homepage/conditional/RingerVibrateCondition.java b/src/com/android/settings/homepage/conditional/RingerVibrateCondition.java
deleted file mode 100644
index 92667c2..0000000
--- a/src/com/android/settings/homepage/conditional/RingerVibrateCondition.java
+++ /dev/null
@@ -1,56 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional;
-
-import android.graphics.drawable.Drawable;
-import android.media.AudioManager;
-
-import com.android.internal.logging.nano.MetricsProto;
-import com.android.settings.R;
-
-@Deprecated
-public class RingerVibrateCondition extends AbnormalRingerConditionBase {
-
- RingerVibrateCondition(ConditionManager manager) {
- super(manager);
- }
-
- @Override
- public void refreshState() {
- setActive(mAudioManager.getRingerModeInternal() == AudioManager.RINGER_MODE_VIBRATE);
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsProto.MetricsEvent.SETTINGS_CONDITION_DEVICE_VIBRATE;
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_volume_ringer_vibrate);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getText(R.string.condition_device_vibrate_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getText(R.string.condition_device_vibrate_summary);
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionCard.java b/src/com/android/settings/homepage/conditional/RingerVibrateConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionCard.java
rename to src/com/android/settings/homepage/conditional/RingerVibrateConditionCard.java
index 3ca3a4d..229cc83 100644
--- a/src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/RingerVibrateConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionController.java b/src/com/android/settings/homepage/conditional/RingerVibrateConditionController.java
similarity index 95%
rename from src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionController.java
rename to src/com/android/settings/homepage/conditional/RingerVibrateConditionController.java
index d2d3244..43de080 100644
--- a/src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionController.java
+++ b/src/com/android/settings/homepage/conditional/RingerVibrateConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.media.AudioManager;
diff --git a/src/com/android/settings/homepage/conditional/WorkModeCondition.java b/src/com/android/settings/homepage/conditional/WorkModeCondition.java
deleted file mode 100644
index 31266a4..0000000
--- a/src/com/android/settings/homepage/conditional/WorkModeCondition.java
+++ /dev/null
@@ -1,108 +0,0 @@
-/*
- * Copyright (C) 2016 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.homepage.conditional;
-
-import android.content.Context;
-import android.content.Intent;
-import android.content.pm.UserInfo;
-import android.graphics.drawable.Drawable;
-import android.os.UserHandle;
-import android.os.UserManager;
-
-import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.R;
-import com.android.settings.Settings;
-
-import java.util.List;
-
-@Deprecated
-public class WorkModeCondition extends Condition {
-
- private UserManager mUm;
- private UserHandle mUserHandle;
-
- public WorkModeCondition(ConditionManager conditionManager) {
- super(conditionManager);
- mUm = (UserManager) mManager.getContext().getSystemService(Context.USER_SERVICE);
- }
-
- private void updateUserHandle() {
- List<UserInfo> profiles = mUm.getProfiles(UserHandle.myUserId());
- final int profilesCount = profiles.size();
- mUserHandle = null;
- for (int i = 0; i < profilesCount; i++) {
- UserInfo userInfo = profiles.get(i);
- if (userInfo.isManagedProfile()) {
- // We assume there's only one managed profile, otherwise UI needs to change.
- mUserHandle = userInfo.getUserHandle();
- break;
- }
- }
- }
-
- @Override
- public void refreshState() {
- updateUserHandle();
- setActive(mUserHandle != null && mUm.isQuietModeEnabled(mUserHandle));
- }
-
- @Override
- public Drawable getIcon() {
- return mManager.getContext().getDrawable(R.drawable.ic_signal_workmode_enable);
- }
-
- @Override
- public CharSequence getTitle() {
- return mManager.getContext().getString(R.string.condition_work_title);
- }
-
- @Override
- public CharSequence getSummary() {
- return mManager.getContext().getString(R.string.condition_work_summary);
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[] {
- mManager.getContext().getString(R.string.condition_turn_on)
- };
- }
-
- @Override
- public void onPrimaryClick() {
- mManager.getContext().startActivity(new Intent(mManager.getContext(),
- Settings.AccountDashboardActivity.class)
- .addFlags(Intent.FLAG_ACTIVITY_NEW_TASK));
- }
-
- @Override
- public void onActionClick(int index) {
- if (index == 0) {
- if (mUserHandle != null) {
- mUm.requestQuietModeEnabled(false, mUserHandle);
- }
- setActive(false);
- } else {
- throw new IllegalArgumentException("Unexpected index " + index);
- }
- }
-
- @Override
- public int getMetricsConstant() {
- return MetricsEvent.SETTINGS_CONDITION_WORK_MODE;
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/WorkModeConditionCard.java b/src/com/android/settings/homepage/conditional/WorkModeConditionCard.java
similarity index 96%
rename from src/com/android/settings/homepage/conditional/v2/WorkModeConditionCard.java
rename to src/com/android/settings/homepage/conditional/WorkModeConditionCard.java
index 10bcd6d..c283bec 100644
--- a/src/com/android/settings/homepage/conditional/v2/WorkModeConditionCard.java
+++ b/src/com/android/settings/homepage/conditional/WorkModeConditionCard.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.graphics.drawable.Drawable;
diff --git a/src/com/android/settings/homepage/conditional/v2/WorkModeConditionController.java b/src/com/android/settings/homepage/conditional/WorkModeConditionController.java
similarity index 97%
rename from src/com/android/settings/homepage/conditional/v2/WorkModeConditionController.java
rename to src/com/android/settings/homepage/conditional/WorkModeConditionController.java
index 73689c7..1bd227a 100644
--- a/src/com/android/settings/homepage/conditional/v2/WorkModeConditionController.java
+++ b/src/com/android/settings/homepage/conditional/WorkModeConditionController.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import android.content.Context;
import android.content.Intent;
diff --git a/src/com/android/settings/homepage/conditional/v2/ConditionAdapter.java b/src/com/android/settings/homepage/conditional/v2/ConditionAdapter.java
deleted file mode 100644
index 51d8c47..0000000
--- a/src/com/android/settings/homepage/conditional/v2/ConditionAdapter.java
+++ /dev/null
@@ -1,130 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional.v2;
-
-import android.content.Context;
-import android.text.TextUtils;
-import android.view.LayoutInflater;
-import android.view.View;
-import android.view.ViewGroup;
-import android.widget.Button;
-
-import androidx.recyclerview.widget.RecyclerView;
-
-import com.android.internal.logging.nano.MetricsProto;
-import com.android.settings.R;
-import com.android.settings.dashboard.DashboardAdapter;
-import com.android.settings.overlay.FeatureFactory;
-import com.android.settingslib.core.instrumentation.MetricsFeatureProvider;
-
-import java.util.List;
-
-public class ConditionAdapter extends RecyclerView.Adapter<DashboardAdapter.DashboardItemHolder> {
-
- private final Context mContext;
- private final MetricsFeatureProvider mMetricsFeatureProvider;
- private final ConditionManager mConditionManager;
- private final List<ConditionalCard> mConditions;
- private final boolean mExpanded;
-
- public ConditionAdapter(Context context, ConditionManager conditionManager,
- List<ConditionalCard> conditions, boolean expanded) {
- mContext = context;
- mConditionManager = conditionManager;
- mConditions = conditions;
- mExpanded = expanded;
- mMetricsFeatureProvider = FeatureFactory.getFactory(context).getMetricsFeatureProvider();
-
- setHasStableIds(true);
- }
-
- @Override
- public DashboardAdapter.DashboardItemHolder onCreateViewHolder(ViewGroup parent, int viewType) {
- return new DashboardAdapter.DashboardItemHolder(LayoutInflater.from(parent.getContext())
- .inflate(viewType, parent, false));
- }
-
- @Override
- public void onBindViewHolder(DashboardAdapter.DashboardItemHolder holder, int position) {
- final ConditionalCard condition = mConditions.get(position);
- final boolean isLastItem = position == mConditions.size() - 1;
- bindViews(condition, holder, isLastItem);
- }
-
- @Override
- public long getItemId(int position) {
- return mConditions.get(position).getId();
- }
-
- @Override
- public int getItemViewType(int position) {
- return R.layout.condition_tile;
- }
-
- @Override
- public int getItemCount() {
- if (mExpanded) {
- return mConditions.size();
- }
- return 0;
- }
-
- private void bindViews(final ConditionalCard condition,
- DashboardAdapter.DashboardItemHolder view, boolean isLastItem) {
- mMetricsFeatureProvider.visible(mContext, MetricsProto.MetricsEvent.DASHBOARD_SUMMARY,
- condition.getMetricsConstant());
- view.itemView.findViewById(R.id.content).setOnClickListener(
- v -> {
- mMetricsFeatureProvider.action(mContext,
- MetricsProto.MetricsEvent.ACTION_SETTINGS_CONDITION_CLICK,
- condition.getMetricsConstant());
- mConditionManager.onPrimaryClick(mContext, condition.getId());
- });
- view.icon.setImageDrawable(condition.getIcon());
- view.title.setText(condition.getTitle());
- view.summary.setText(condition.getSummary());
-
- setViewVisibility(view.itemView, R.id.divider, !isLastItem);
-
- final CharSequence action = condition.getActionText();
- final boolean hasButtons = !TextUtils.isEmpty(action);
- setViewVisibility(view.itemView, R.id.buttonBar, hasButtons);
-
- final Button button = view.itemView.findViewById(R.id.first_action);
- if (hasButtons) {
- button.setVisibility(View.VISIBLE);
- button.setText(action);
- button.setOnClickListener(v -> {
- final Context context = v.getContext();
- mMetricsFeatureProvider.action(
- context, MetricsProto.MetricsEvent.ACTION_SETTINGS_CONDITION_BUTTON,
- condition.getMetricsConstant());
- mConditionManager.onActionClick(condition.getId());
- });
- } else {
- button.setVisibility(View.GONE);
- }
-
- }
-
- private void setViewVisibility(View containerView, int viewId, boolean visible) {
- View view = containerView.findViewById(viewId);
- if (view != null) {
- view.setVisibility(visible ? View.VISIBLE : View.GONE);
- }
- }
-}
diff --git a/src/com/android/settings/homepage/conditional/v2/ConditionManager.java b/src/com/android/settings/homepage/conditional/v2/ConditionManager.java
deleted file mode 100644
index c67b255..0000000
--- a/src/com/android/settings/homepage/conditional/v2/ConditionManager.java
+++ /dev/null
@@ -1,210 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional.v2;
-
-import android.content.Context;
-import android.util.FeatureFlagUtils;
-import android.util.Log;
-
-import androidx.annotation.NonNull;
-import androidx.annotation.VisibleForTesting;
-
-import com.android.settings.core.FeatureFlags;
-import com.android.settings.homepage.conditional.ConditionListener;
-
-import java.util.ArrayList;
-import java.util.List;
-import java.util.concurrent.Callable;
-import java.util.concurrent.ExecutionException;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
-import java.util.concurrent.Future;
-import java.util.concurrent.TimeUnit;
-import java.util.concurrent.TimeoutException;
-
-public class ConditionManager {
- private static final String TAG = "ConditionManager";
-
- @VisibleForTesting
- final List<ConditionalCard> mCandidates;
- @VisibleForTesting
- final List<ConditionalCardController> mCardControllers;
-
- private static final long DISPLAYABLE_CHECKER_TIMEOUT_MS = 20;
-
- private final ExecutorService mExecutorService;
- private final Context mAppContext;
- private final ConditionListener mListener;
-
- private boolean mIsListeningToStateChange;
-
- /**
- * Whether or not the new condition manager is should be used.
- */
- public static boolean isEnabled(Context context) {
- return FeatureFlagUtils.isEnabled(context, FeatureFlags.CONDITION_MANAGER_V2);
- }
-
- public ConditionManager(Context context, ConditionListener listener) {
- mAppContext = context.getApplicationContext();
- mExecutorService = Executors.newCachedThreadPool();
- mCandidates = new ArrayList<>();
- mCardControllers = new ArrayList<>();
- mListener = listener;
- initCandidates();
- }
-
- /**
- * Returns a list of {@link ConditionalCard}s eligible for display.
- */
- public List<ConditionalCard> getDisplayableCards() {
- final List<ConditionalCard> cards = new ArrayList<>();
- final List<Future<ConditionalCard>> displayableCards = new ArrayList<>();
- // Check displayable future
- for (ConditionalCard card : mCandidates) {
- final DisplayableChecker future = new DisplayableChecker(
- card, getController(card.getId()));
- displayableCards.add(mExecutorService.submit(future));
- }
- // Collect future and add displayable cards
- for (Future<ConditionalCard> cardFuture : displayableCards) {
- try {
- final ConditionalCard card = cardFuture.get(DISPLAYABLE_CHECKER_TIMEOUT_MS,
- TimeUnit.MILLISECONDS);
- if (card != null) {
- cards.add(card);
- }
- } catch (InterruptedException | ExecutionException | TimeoutException e) {
- Log.w(TAG, "Failed to get displayable state for card, likely timeout. Skipping", e);
- }
- }
- return cards;
- }
-
- /**
- * Handler when the card is clicked.
- *
- * @see {@link ConditionalCardController#onPrimaryClick(Context)}
- */
- public void onPrimaryClick(Context context, long id) {
- getController(id).onPrimaryClick(context);
- }
-
- /**
- * Handler when the card action is clicked.
- *
- * @see {@link ConditionalCardController#onActionClick()}
- */
- public void onActionClick(long id) {
- getController(id).onActionClick();
- onConditionChanged();
- }
-
- /**
- * Start monitoring state change for all conditions
- */
- public void startMonitoringStateChange() {
- if (mIsListeningToStateChange) {
- Log.d(TAG, "Already listening to condition state changes, skipping");
- return;
- }
- mIsListeningToStateChange = true;
- for (ConditionalCardController controller : mCardControllers) {
- controller.startMonitoringStateChange();
- }
- // Force a refresh on listener
- onConditionChanged();
- }
-
- /**
- * Stop monitoring state change for all conditions
- */
- public void stopMonitoringStateChange() {
- if (!mIsListeningToStateChange) {
- Log.d(TAG, "Not listening to condition state changes, skipping");
- return;
- }
- for (ConditionalCardController controller : mCardControllers) {
- controller.stopMonitoringStateChange();
- }
- mIsListeningToStateChange = false;
- }
-
- /**
- * Called when some conditional card's state has changed
- */
- void onConditionChanged() {
- if (mListener != null) {
- mListener.onConditionsChanged();
- }
- }
-
- @NonNull
- <T extends ConditionalCardController> T getController(long id) {
- for (ConditionalCardController controller : mCardControllers) {
- if (controller.getId() == id) {
- return (T) controller;
- }
- }
- throw new IllegalStateException("Cannot find controller for " + id);
- }
-
- private void initCandidates() {
- // Initialize controllers first.
- mCardControllers.add(new AirplaneModeConditionController(mAppContext, this /* manager */));
- mCardControllers.add(new BackgroundDataConditionController(mAppContext));
- mCardControllers.add(new BatterySaverConditionController(mAppContext, this /* manager */));
- mCardControllers.add(new CellularDataConditionController(mAppContext, this /* manager */));
- mCardControllers.add(new DndConditionCardController(mAppContext, this /* manager */));
- mCardControllers.add(new HotspotConditionController(mAppContext, this /* manager */));
- mCardControllers.add(new NightDisplayConditionController(mAppContext));
- mCardControllers.add(new RingerVibrateConditionController(mAppContext, this /* manager */));
- mCardControllers.add(new RingerMutedConditionController(mAppContext, this /* manager */));
- mCardControllers.add(new WorkModeConditionController(mAppContext));
-
- // Initialize ui model later. UI model depends on controller.
- mCandidates.add(new AirplaneModeConditionCard(mAppContext));
- mCandidates.add(new BackgroundDataConditionCard(mAppContext));
- mCandidates.add(new BatterySaverConditionCard(mAppContext));
- mCandidates.add(new CellularDataConditionCard(mAppContext));
- mCandidates.add(new DndConditionCard(mAppContext, this /* manager */));
- mCandidates.add(new HotspotConditionCard(mAppContext, this /* manager */));
- mCandidates.add(new NightDisplayConditionCard(mAppContext));
- mCandidates.add(new RingerMutedConditionCard(mAppContext));
- mCandidates.add(new RingerVibrateConditionCard(mAppContext));
- mCandidates.add(new WorkModeConditionCard(mAppContext));
- }
-
- /**
- * Returns card if controller says it's displayable. Otherwise returns null.
- */
- public static class DisplayableChecker implements Callable<ConditionalCard> {
-
- private final ConditionalCard mCard;
- private final ConditionalCardController mController;
-
- private DisplayableChecker(ConditionalCard card, ConditionalCardController controller) {
- mCard = card;
- mController = controller;
- }
-
- @Override
- public ConditionalCard call() throws Exception {
- return mController.isDisplayable() ? mCard : null;
- }
- }
-}
diff --git a/tests/robotests/src/com/android/settings/dashboard/DashboardAdapterTest.java b/tests/robotests/src/com/android/settings/dashboard/DashboardAdapterTest.java
index 58a5cc9..f544f09 100644
--- a/tests/robotests/src/com/android/settings/dashboard/DashboardAdapterTest.java
+++ b/tests/robotests/src/com/android/settings/dashboard/DashboardAdapterTest.java
@@ -47,8 +47,9 @@
import com.android.settings.R;
import com.android.settings.SettingsActivity;
-import com.android.settings.homepage.conditional.Condition;
import com.android.settings.dashboard.suggestions.SuggestionAdapter;
+import com.android.settings.homepage.conditional.ConditionManager;
+import com.android.settings.homepage.conditional.ConditionalCard;
import com.android.settings.testutils.FakeFeatureFactory;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import com.android.settings.testutils.shadow.SettingsShadowResources;
@@ -77,22 +78,22 @@
@Mock(answer = Answers.RETURNS_DEEP_STUBS)
private SettingsActivity mContext;
@Mock
- private View mView;
- @Mock
- private Condition mCondition;
+ private ConditionalCard mCondition;
@Mock
private Resources mResources;
@Mock
private WindowManager mWindowManager;
+ @Mock
+ private ConditionManager mConditionManager;
+
private ActivityInfo mActivityInfo;
- private FakeFeatureFactory mFactory;
private DashboardAdapter mDashboardAdapter;
- private List<Condition> mConditionList;
+ private List<ConditionalCard> mConditionList;
@Before
public void setUp() {
MockitoAnnotations.initMocks(this);
- mFactory = FakeFeatureFactory.setupForTest();
+ FakeFeatureFactory.setupForTest();
mActivityInfo = new ActivityInfo();
mActivityInfo.packageName = "pkg";
mActivityInfo.name = "class";
@@ -105,18 +106,15 @@
mConditionList = new ArrayList<>();
mConditionList.add(mCondition);
- when(mCondition.shouldShow()).thenReturn(true);
mDashboardAdapter = new DashboardAdapter(mContext, null /* savedInstanceState */,
- mConditionList, null /* conditionManager */,
- null /* suggestionControllerMixin */, null /* lifecycle */);
- when(mView.getTag()).thenReturn(mCondition);
+ mConditionManager, null /* suggestionControllerMixin */, null /* lifecycle */);
}
@Test
public void onSuggestionClosed_notOnlySuggestion_updateSuggestionOnly() {
final DashboardAdapter adapter =
spy(new DashboardAdapter(mContext, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
+ mConditionManager,
null /* suggestionControllerMixin */,
null /* lifecycle */));
final List<Suggestion> suggestions = makeSuggestions("pkg1", "pkg2", "pkg3");
@@ -149,7 +147,7 @@
public void onSuggestionClosed_onlySuggestion_updateDashboardData() {
final DashboardAdapter adapter =
spy(new DashboardAdapter(mContext, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
+ mConditionManager,
null /* suggestionControllerMixin */, null /* lifecycle */));
final List<Suggestion> suggestions = makeSuggestions("pkg1");
adapter.setSuggestions(suggestions);
@@ -166,7 +164,7 @@
public void onSuggestionClosed_notInSuggestionList_shouldNotUpdateSuggestionList() {
final DashboardAdapter adapter =
spy(new DashboardAdapter(mContext, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
+ mConditionManager,
null /* suggestionControllerMixin */, null /* lifecycle */));
final List<Suggestion> suggestions = makeSuggestions("pkg1");
adapter.setSuggestions(suggestions);
@@ -181,8 +179,7 @@
@Test
public void onBindSuggestion_shouldSetSuggestionAdapterAndNoCrash() {
mDashboardAdapter = new DashboardAdapter(mContext, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
- null /* suggestionControllerMixin */, null /* lifecycle */);
+ mConditionManager, null /* suggestionControllerMixin */, null /* lifecycle */);
final List<Suggestion> suggestions = makeSuggestions("pkg1");
mDashboardAdapter.setSuggestions(suggestions);
@@ -218,8 +215,7 @@
.thenReturn(context.getDrawable(R.drawable.ic_settings));
mDashboardAdapter = new DashboardAdapter(context, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
- null /* suggestionControllerMixin */, null /* lifecycle */);
+ mConditionManager, null /* suggestionControllerMixin */, null /* lifecycle */);
ReflectionHelpers.setField(mDashboardAdapter, "mCache", iconCache);
mDashboardAdapter.onBindTile(holder, tile);
@@ -239,7 +235,7 @@
final IconCache iconCache = new IconCache(context);
mDashboardAdapter = new DashboardAdapter(context, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
+ mConditionManager,
null /* suggestionControllerMixin */, null /* lifecycle */);
ReflectionHelpers.setField(mDashboardAdapter, "mCache", iconCache);
@@ -264,8 +260,7 @@
when(iconCache.getIcon(tile.getIcon(context))).thenReturn(mock(RoundedHomepageIcon.class));
mDashboardAdapter = new DashboardAdapter(context, null /* savedInstanceState */,
- null /* conditions */, null /* conditionManager */,
- null /* suggestionControllerMixin */, null /* lifecycle */);
+ mConditionManager, null /* suggestionControllerMixin */, null /* lifecycle */);
ReflectionHelpers.setField(mDashboardAdapter, "mCache", iconCache);
mDashboardAdapter.onBindTile(holder, tile);
diff --git a/tests/robotests/src/com/android/settings/dashboard/DashboardDataTest.java b/tests/robotests/src/com/android/settings/dashboard/DashboardDataTest.java
index 406cb3c..73b1734 100644
--- a/tests/robotests/src/com/android/settings/dashboard/DashboardDataTest.java
+++ b/tests/robotests/src/com/android/settings/dashboard/DashboardDataTest.java
@@ -33,8 +33,8 @@
import androidx.recyclerview.widget.DiffUtil;
import androidx.recyclerview.widget.ListUpdateCallback;
-import com.android.settings.homepage.conditional.AirplaneModeCondition;
-import com.android.settings.homepage.conditional.Condition;
+import com.android.settings.homepage.conditional.AirplaneModeConditionCard;
+import com.android.settings.homepage.conditional.ConditionalCard;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import com.android.settingslib.drawer.CategoryKey;
import com.android.settingslib.drawer.DashboardCategory;
@@ -63,9 +63,9 @@
@Mock
private Tile mTestCategoryTile;
@Mock
- private Condition mTestCondition;
+ private ConditionalCard mTestCondition;
@Mock
- private Condition mSecondCondition; // condition used to test insert in DiffUtil
+ private ConditionalCard mSecondCondition; // condition used to test insert in DiffUtil
private Suggestion mTestSuggestion;
@Before
@@ -83,13 +83,11 @@
suggestions.add(mTestSuggestion);
// Build oneItemConditions
- final List<Condition> oneItemConditions = new ArrayList<>();
- when(mTestCondition.shouldShow()).thenReturn(true);
+ final List<ConditionalCard> oneItemConditions = new ArrayList<>();
oneItemConditions.add(mTestCondition);
// Build twoItemConditions
- final List<Condition> twoItemsConditions = new ArrayList<>();
- when(mSecondCondition.shouldShow()).thenReturn(true);
+ final List<ConditionalCard> twoItemsConditions = new ArrayList<>();
twoItemsConditions.add(mTestCondition);
twoItemsConditions.add(mSecondCondition);
@@ -170,7 +168,7 @@
@Test
public void testGetPositionByEntity_notExisted_returnNotFound() {
- final Condition condition = mock(AirplaneModeCondition.class);
+ final ConditionalCard condition = mock(AirplaneModeConditionCard.class);
final int position = mDashboardDataWithOneConditions.getPositionByEntity(condition);
assertThat(position).isEqualTo(DashboardData.POSITION_NOT_FOUND);
}
@@ -227,8 +225,8 @@
testResultData.add(new ListUpdateResult.ResultData(
ListUpdateResult.ResultData.TYPE_OPERATION_CHANGE, 2, 1));
// Build DashboardData
- final List<Condition> oneItemConditions = new ArrayList<>();
- when(mTestCondition.shouldShow()).thenReturn(true);
+ final List<ConditionalCard> oneItemConditions = new ArrayList<>();
+
oneItemConditions.add(mTestCondition);
final List<Suggestion> suggestions = new ArrayList<>();
suggestions.add(mTestSuggestion);
diff --git a/tests/robotests/src/com/android/settings/dashboard/DashboardSummaryTest.java b/tests/robotests/src/com/android/settings/dashboard/DashboardSummaryTest.java
index 4eaeaf2..1759fff 100644
--- a/tests/robotests/src/com/android/settings/dashboard/DashboardSummaryTest.java
+++ b/tests/robotests/src/com/android/settings/dashboard/DashboardSummaryTest.java
@@ -27,7 +27,6 @@
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
-import android.app.Activity;
import android.content.Context;
import com.android.settings.homepage.conditional.ConditionManager;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/AbnormalRingerConditionControllerBaseTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/AbnormalRingerConditionControllerBaseTest.java
similarity index 92%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/AbnormalRingerConditionControllerBaseTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/AbnormalRingerConditionControllerBaseTest.java
index dc06710..0b17728 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/AbnormalRingerConditionControllerBaseTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/AbnormalRingerConditionControllerBaseTest.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.verify;
@@ -23,6 +23,8 @@
import android.content.Intent;
import android.media.AudioManager;
+import com.android.settings.homepage.conditional.AbnormalRingerConditionController;
+import com.android.settings.homepage.conditional.ConditionManager;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionControllerTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/BackgroundDataConditionControllerTest.java
similarity index 93%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionControllerTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/BackgroundDataConditionControllerTest.java
index b12d786..4ad7e6d 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/BackgroundDataConditionControllerTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/BackgroundDataConditionControllerTest.java
@@ -13,7 +13,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static com.google.common.truth.Truth.assertThat;
@@ -24,6 +24,7 @@
import android.content.Intent;
import com.android.settings.Settings;
+import com.android.settings.homepage.conditional.BackgroundDataConditionController;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionControllerTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/BatterySaverConditionControllerTest.java
similarity index 93%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionControllerTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/BatterySaverConditionControllerTest.java
index 8849e2d..9dd6780 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/BatterySaverConditionControllerTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/BatterySaverConditionControllerTest.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static com.google.common.truth.Truth.assertThat;
@@ -27,6 +27,8 @@
import android.os.PowerManager;
import com.android.settings.fuelgauge.BatterySaverReceiver;
+import com.android.settings.homepage.conditional.BatterySaverConditionController;
+import com.android.settings.homepage.conditional.ConditionManager;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/BatterySaverConditionTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/BatterySaverConditionTest.java
deleted file mode 100644
index 323e6c5..0000000
--- a/tests/robotests/src/com/android/settings/homepage/conditional/BatterySaverConditionTest.java
+++ /dev/null
@@ -1,103 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional;
-
-import static com.google.common.truth.Truth.assertThat;
-
-import static org.mockito.Matchers.any;
-import static org.mockito.Mockito.spy;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-import android.content.Context;
-import android.content.IntentFilter;
-import android.os.PowerManager;
-
-import com.android.settings.R;
-import com.android.settings.fuelgauge.BatterySaverReceiver;
-import com.android.settings.testutils.SettingsRobolectricTestRunner;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.MockitoAnnotations;
-import org.robolectric.RuntimeEnvironment;
-import org.robolectric.Shadows;
-import org.robolectric.shadows.ShadowPowerManager;
-
-@RunWith(SettingsRobolectricTestRunner.class)
-public class BatterySaverConditionTest {
- @Mock
- private ConditionManager mConditionManager;
-
- private ShadowPowerManager mPowerManager;
- private Context mContext;
- private BatterySaverCondition mCondition;
-
- @Before
- public void setUp() {
- MockitoAnnotations.initMocks(this);
- mContext = spy(RuntimeEnvironment.application);
- mPowerManager = Shadows.shadowOf(mContext.getSystemService(PowerManager.class));
- when(mConditionManager.getContext()).thenReturn(mContext);
- mCondition = spy(new BatterySaverCondition(mConditionManager));
- }
-
- @Test
- public void verifyText() {
- assertThat(mCondition.getTitle()).isEqualTo(
- mContext.getText(R.string.condition_battery_title));
- assertThat(mCondition.getSummary()).isEqualTo(
- mContext.getText(R.string.condition_battery_summary));
- assertThat(mCondition.getActions()[0]).isEqualTo(
- mContext.getText(R.string.condition_turn_off));
- }
-
- @Test
- public void onResume_shouldRegisterReceiver() {
- mCondition.onResume();
-
- verify(mContext).registerReceiver(any(BatterySaverReceiver.class), any(IntentFilter.class));
- }
-
- @Test
- public void onPause_shouldUnregisterReceiver() {
- mCondition.onResume();
- mCondition.onPause();
-
- verify(mContext).unregisterReceiver(any(BatterySaverReceiver.class));
- }
-
- @Test
- public void refreshState_PowerSaverOn_shouldActivate() {
- mPowerManager.setIsPowerSaveMode(true);
-
- mCondition.refreshState();
-
- assertThat(mCondition.isActive()).isTrue();
- }
-
- @Test
- public void refreshState_PowerSaverOff_shouldNotActivate() {
- mPowerManager.setIsPowerSaveMode(false);
-
- mCondition.refreshState();
-
- assertThat(mCondition.isActive()).isFalse();
- }
-}
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/ConditionAdapterTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/ConditionAdapterTest.java
index 663c630..4d154da 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/ConditionAdapterTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/ConditionAdapterTest.java
@@ -17,6 +17,8 @@
import static com.google.common.truth.Truth.assertThat;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyLong;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
@@ -25,8 +27,6 @@
import android.view.View;
import android.widget.LinearLayout;
-import androidx.recyclerview.widget.RecyclerView;
-
import com.android.settings.R;
import com.android.settings.dashboard.DashboardAdapter;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
@@ -45,22 +45,23 @@
public class ConditionAdapterTest {
@Mock
- private Condition mCondition1;
+ private ConditionalCard mCondition1;
@Mock
- private Condition mCondition2;
+ private ConditionalCard mCondition2;
+ @Mock
+ private ConditionManager mConditionManager;
private Context mContext;
private ConditionAdapter mConditionAdapter;
- private List<Condition> mOneCondition;
- private List<Condition> mTwoConditions;
+ private List<ConditionalCard> mOneCondition;
+ private List<ConditionalCard> mTwoConditions;
@Before
public void setUp() {
MockitoAnnotations.initMocks(this);
mContext = RuntimeEnvironment.application;
- final CharSequence[] actions = new CharSequence[2];
- when(mCondition1.getActions()).thenReturn(actions);
- when(mCondition1.shouldShow()).thenReturn(true);
+ final CharSequence action = "action";
+ when(mCondition1.getActionText()).thenReturn(action);
mOneCondition = new ArrayList<>();
mOneCondition.add(mCondition1);
mTwoConditions = new ArrayList<>();
@@ -70,32 +71,32 @@
@Test
public void getItemCount_notExpanded_shouldReturn0() {
- mConditionAdapter = new ConditionAdapter(mContext, mOneCondition, false);
+ mConditionAdapter = new ConditionAdapter(mContext, mConditionManager, mOneCondition, false);
assertThat(mConditionAdapter.getItemCount()).isEqualTo(0);
}
@Test
public void getItemCount_expanded_shouldReturnListSize() {
- mConditionAdapter = new ConditionAdapter(mContext, mOneCondition, true);
+ mConditionAdapter = new ConditionAdapter(mContext, mConditionManager, mOneCondition, true);
assertThat(mConditionAdapter.getItemCount()).isEqualTo(1);
- mConditionAdapter = new ConditionAdapter(mContext, mTwoConditions, true);
+ mConditionAdapter = new ConditionAdapter(mContext, mConditionManager, mTwoConditions, true);
assertThat(mConditionAdapter.getItemCount()).isEqualTo(2);
}
@Test
public void getItemViewType_shouldReturnConditionTile() {
- mConditionAdapter = new ConditionAdapter(mContext, mTwoConditions, true);
+ mConditionAdapter = new ConditionAdapter(mContext, mConditionManager, mTwoConditions, true);
assertThat(mConditionAdapter.getItemViewType(0)).isEqualTo(R.layout.condition_tile);
}
@Test
public void onBindViewHolder_shouldSetListener() {
final View view = LayoutInflater.from(mContext)
- .inflate(R.layout.condition_tile, new LinearLayout(mContext), true);
+ .inflate(R.layout.condition_tile, new LinearLayout(mContext), true);
final DashboardAdapter.DashboardItemHolder viewHolder =
- new DashboardAdapter.DashboardItemHolder(view);
- mConditionAdapter = new ConditionAdapter(mContext, mOneCondition, true);
+ new DashboardAdapter.DashboardItemHolder(view);
+ mConditionAdapter = new ConditionAdapter(mContext, mConditionManager, mOneCondition, true);
mConditionAdapter.onBindViewHolder(viewHolder, 0);
final View card = view.findViewById(R.id.content);
@@ -106,30 +107,15 @@
@Test
public void viewClick_shouldInvokeConditionPrimaryClick() {
final View view = LayoutInflater.from(mContext)
- .inflate(R.layout.condition_tile, new LinearLayout(mContext), true);
+ .inflate(R.layout.condition_tile, new LinearLayout(mContext), true);
final DashboardAdapter.DashboardItemHolder viewHolder =
- new DashboardAdapter.DashboardItemHolder(view);
- mConditionAdapter = new ConditionAdapter(mContext, mOneCondition, true);
+ new DashboardAdapter.DashboardItemHolder(view);
+ mConditionAdapter = new ConditionAdapter(mContext, mConditionManager, mOneCondition, true);
mConditionAdapter.onBindViewHolder(viewHolder, 0);
final View card = view.findViewById(R.id.content);
assertThat(card).isNotNull();
card.performClick();
- verify(mCondition1).onPrimaryClick();
- }
-
- @Test
- public void onSwiped_nullCondition_shouldNotCrash() {
- final RecyclerView recyclerView = new RecyclerView(mContext);
- final View view = LayoutInflater.from(mContext).inflate(
- R.layout.condition_tile, new LinearLayout(mContext), true);
- final DashboardAdapter.DashboardItemHolder viewHolder =
- new DashboardAdapter.DashboardItemHolder(view);
- mConditionAdapter = new ConditionAdapter(mContext, mOneCondition, true);
- mConditionAdapter.addDismissHandling(recyclerView);
-
- // do not bind viewholder to simulate the null condition scenario
- mConditionAdapter.mSwipeCallback.onSwiped(viewHolder, 0);
- // no crash
+ verify(mConditionManager).onPrimaryClick(any(Context.class), anyLong());
}
}
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/ConditionManagerTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/ConditionManagerTest.java
similarity index 93%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/ConditionManagerTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/ConditionManagerTest.java
index 2f96eaf..f90c448 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/ConditionManagerTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/ConditionManagerTest.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static com.google.common.truth.Truth.assertThat;
@@ -25,6 +25,9 @@
import android.content.Context;
import com.android.settings.homepage.conditional.ConditionListener;
+import com.android.settings.homepage.conditional.ConditionManager;
+import com.android.settings.homepage.conditional.ConditionalCard;
+import com.android.settings.homepage.conditional.ConditionalCardController;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/ConditionTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/ConditionTest.java
deleted file mode 100644
index 981ef66..0000000
--- a/tests/robotests/src/com/android/settings/homepage/conditional/ConditionTest.java
+++ /dev/null
@@ -1,161 +0,0 @@
-/*
- * Copyright (C) 2016 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.homepage.conditional;
-
-import static com.google.common.truth.Truth.assertThat;
-
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.eq;
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.never;
-import static org.mockito.Mockito.spy;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.IntentFilter;
-import android.graphics.drawable.Drawable;
-
-import com.android.internal.logging.nano.MetricsProto;
-import com.android.settings.testutils.SettingsRobolectricTestRunner;
-import com.android.settingslib.core.instrumentation.MetricsFeatureProvider;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.MockitoAnnotations;
-import org.robolectric.RuntimeEnvironment;
-
-@RunWith(SettingsRobolectricTestRunner.class)
-public class ConditionTest {
-
- @Mock
- private ConditionManager mConditionManager;
- @Mock
- private MetricsFeatureProvider mMetricsFeatureProvider;
-
- private Context mContext;
- private TestCondition mCondition;
-
- @Before
- public void setUp() {
- MockitoAnnotations.initMocks(this);
- mContext = spy(RuntimeEnvironment.application);
- mCondition = new TestCondition(mConditionManager, mMetricsFeatureProvider);
- when(mConditionManager.getContext()).thenReturn(mContext);
- }
-
- @Test
- public void initialize_shouldNotBeSilenced() {
- assertThat(mCondition.isSilenced()).isFalse();
- }
-
- @Test
- public void silence_shouldNotifyDataChangeAndLog() {
- mCondition.silence();
-
- assertThat(mCondition.isSilenced()).isTrue();
- verify(mConditionManager).notifyChanged(mCondition);
- verify(mMetricsFeatureProvider).action(any(Context.class),
- eq(MetricsProto.MetricsEvent.ACTION_SETTINGS_CONDITION_DISMISS),
- eq(TestCondition.TEST_METRIC_CONSTANT));
- }
-
- @Test
- public void onSilenceChanged_silenced_shouldRegisterReceiver() {
- mCondition.onSilenceChanged(true);
-
- verify(mContext).registerReceiver(
- TestCondition.mReceiver, TestCondition.TESTS_INTENT_FILTER);
- }
-
- @Test
- public void onSilenceChanged_notSilenced_registered_shouldUnregisterReceiver() {
- mCondition.onSilenceChanged(true);
-
- mCondition.onSilenceChanged(false);
-
- verify(mContext).unregisterReceiver(TestCondition.mReceiver);
- }
-
- @Test
- public void onSilenceChanged_notSilenced_notRegistered_shouldNotCrash() {
- mCondition.onSilenceChanged(false);
-
- verify(mContext, never()).unregisterReceiver(TestCondition.mReceiver);
- // no crash
- }
-
- private static final class TestCondition extends Condition {
-
- private static final int TEST_METRIC_CONSTANT = 1234;
- private static final IntentFilter TESTS_INTENT_FILTER = new IntentFilter("TestIntent");
- private static final BroadcastReceiver mReceiver = mock(BroadcastReceiver.class);
-
- TestCondition(ConditionManager manager, MetricsFeatureProvider metricsFeatureProvider) {
- super(manager, metricsFeatureProvider);
- }
-
- @Override
- public void refreshState() {
- }
-
- @Override
- public int getMetricsConstant() {
- return TEST_METRIC_CONSTANT;
- }
-
- @Override
- public Drawable getIcon() {
- return null;
- }
-
- @Override
- public CharSequence getTitle() {
- return null;
- }
-
- @Override
- public CharSequence getSummary() {
- return null;
- }
-
- @Override
- public CharSequence[] getActions() {
- return new CharSequence[0];
- }
-
- @Override
- public void onPrimaryClick() {
- }
-
- @Override
- public void onActionClick(int index) {
- }
-
- @Override
- public BroadcastReceiver getReceiver() {
- return mReceiver;
- }
-
- @Override
- public IntentFilter getIntentFilter() {
- return TESTS_INTENT_FILTER;
- }
- }
-}
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionTest.java
deleted file mode 100644
index 9da1f9c..0000000
--- a/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionTest.java
+++ /dev/null
@@ -1,118 +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.homepage.conditional;
-
-import static com.google.common.truth.Truth.assertThat;
-
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.eq;
-import static org.mockito.Mockito.never;
-import static org.mockito.Mockito.reset;
-import static org.mockito.Mockito.times;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-import android.content.ComponentName;
-import android.content.Context;
-import android.content.pm.PackageManager;
-
-import com.android.settings.testutils.FakeFeatureFactory;
-import com.android.settings.testutils.SettingsRobolectricTestRunner;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.MockitoAnnotations;
-
-@RunWith(SettingsRobolectricTestRunner.class)
-public class DndConditionTest {
-
- @Mock
- private ConditionManager mConditionManager;
- @Mock
- private PackageManager mPackageManager;
- @Mock
- private Context mContext;
-
- @Before
- public void setUp() {
- MockitoAnnotations.initMocks(this);
- FakeFeatureFactory.setupForTest();
- when(mConditionManager.getContext()).thenReturn(mContext);
- when(mContext.getPackageManager()).thenReturn(mPackageManager);
- }
-
- @Test
- public void constructor_shouldNotDisableReceiver() {
- new DndCondition(mConditionManager);
- verify(mPackageManager, never()).setComponentEnabledSetting(any(ComponentName.class),
- eq(PackageManager.COMPONENT_ENABLED_STATE_DISABLED), eq(PackageManager.DONT_KILL_APP));
- }
-
- @Test
- public void constructor_shouldRegisterReceiver() {
- new DndCondition(mConditionManager);
- verify(mContext).registerReceiver(any(DndCondition.Receiver.class),
- eq(DndCondition.DND_FILTER));
- }
-
- @Test
- public void silence_shouldNotDisableReceiver() {
- new DndCondition(mConditionManager).silence();
-
- verify(mPackageManager, never()).setComponentEnabledSetting(any(ComponentName.class),
- eq(PackageManager.COMPONENT_ENABLED_STATE_DISABLED), eq(PackageManager.DONT_KILL_APP));
- }
-
- @Test
- public void onResume_shouldRegisterReceiver() {
- DndCondition condition = new DndCondition(mConditionManager);
- condition.onPause();
- condition.onResume();
-
- // one from constructor, one from onResume()
- verify(mContext, times(2)).registerReceiver(any(DndCondition.Receiver.class),
- eq(DndCondition.DND_FILTER));
- }
-
- @Test
- public void onPause_shouldUnregisterReceiver() {
- new DndCondition(mConditionManager).onPause();
-
- verify(mContext).unregisterReceiver(any(DndCondition.Receiver.class));
- }
-
- @Test
- public void onPause_noReceiverRegistered_shouldNotUnregisterReceiver() {
- DndCondition condition = new DndCondition(mConditionManager);
- condition.onPause();
- reset(mContext);
-
- condition.onPause();
-
- verify(mContext, never()).unregisterReceiver(any(DndCondition.Receiver.class));
- }
-
- @Test
- public void nullZenConfig_noCrash() {
- DndCondition condition = new DndCondition(mConditionManager);
- assertThat(condition.mConfig).isNull();
-
- // should not crash, instead summary is null
- assertThat(condition.getSummary()).isNull();
- }
-}
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/DndConditionalCardControllerTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionalCardControllerTest.java
similarity index 90%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/DndConditionalCardControllerTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/DndConditionalCardControllerTest.java
index fe4c621..c68f899 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/DndConditionalCardControllerTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionalCardControllerTest.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static org.mockito.ArgumentMatchers.any;
@@ -24,6 +24,8 @@
import android.content.Context;
+import com.android.settings.homepage.conditional.ConditionManager;
+import com.android.settings.homepage.conditional.DndConditionCardController;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/DndConditionalCardTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionalCardTest.java
similarity index 87%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/DndConditionalCardTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/DndConditionalCardTest.java
index 1dfbd29..47a3010 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/DndConditionalCardTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/DndConditionalCardTest.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static com.google.common.truth.Truth.assertThat;
@@ -24,6 +24,9 @@
import android.content.Context;
+import com.android.settings.homepage.conditional.ConditionManager;
+import com.android.settings.homepage.conditional.DndConditionCard;
+import com.android.settings.homepage.conditional.DndConditionCardController;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/RingerMutedConditionTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/RingerMutedConditionTest.java
deleted file mode 100644
index 408aa13..0000000
--- a/tests/robotests/src/com/android/settings/homepage/conditional/RingerMutedConditionTest.java
+++ /dev/null
@@ -1,96 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional;
-
-
-import static com.google.common.truth.Truth.assertThat;
-
-import static org.mockito.Mockito.spy;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-import android.content.Context;
-import android.media.AudioManager;
-import android.provider.Settings;
-
-import com.android.settings.R;
-import com.android.settings.testutils.SettingsRobolectricTestRunner;
-import com.android.settings.testutils.shadow.ShadowAudioManager;
-import com.android.settings.testutils.shadow.ShadowNotificationManager;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.MockitoAnnotations;
-import org.robolectric.RuntimeEnvironment;
-import org.robolectric.annotation.Config;
-import org.robolectric.shadow.api.Shadow;
-
-@RunWith(SettingsRobolectricTestRunner.class)
-@Config(shadows = {ShadowNotificationManager.class, ShadowAudioManager.class})
-public class RingerMutedConditionTest {
- private static final String TAG = "RingerMutedConditionTest";
- @Mock
- private ConditionManager mConditionManager;
-
- private Context mContext;
- private ShadowNotificationManager mNotificationManager;
- private ShadowAudioManager mAudioManager;
- private RingerMutedCondition mCondition;
-
- @Before
- public void setUp() {
- MockitoAnnotations.initMocks(this);
- mContext = RuntimeEnvironment.application;
- mAudioManager = Shadow.extract(mContext.getSystemService(Context.AUDIO_SERVICE));
- mNotificationManager = Shadow.extract(
- mContext.getSystemService(Context.NOTIFICATION_SERVICE));
- when(mConditionManager.getContext()).thenReturn(mContext);
- mCondition = spy(new RingerMutedCondition(mConditionManager));
- }
-
- @Test
- public void verifyText() {
- assertThat(mCondition.getTitle()).isEqualTo(
- mContext.getText(R.string.condition_device_muted_title));
- assertThat(mCondition.getSummary()).isEqualTo(
- mContext.getText(R.string.condition_device_muted_summary));
- assertThat(mCondition.getActions()[0]).isEqualTo(
- mContext.getText(R.string.condition_device_muted_action_turn_on_sound));
- }
-
- @Test
- public void refreshState_zenModeOn_shouldNotActivate() {
- mAudioManager.setRingerModeInternal(AudioManager.RINGER_MODE_SILENT);
- mNotificationManager.setZenMode(Settings.Global.ZEN_MODE_NO_INTERRUPTIONS, null, TAG);
-
- mCondition.refreshState();
-
- verify(mCondition).setActive(false);
- }
-
- @Test
- public void refreshState_zenModeOff_shouldActivate() {
- mAudioManager.setRingerModeInternal(AudioManager.RINGER_MODE_SILENT);
- mNotificationManager.setZenMode(Settings.Global.ZEN_MODE_OFF, null, TAG);
-
- mCondition.refreshState();
-
- verify(mCondition).setActive(true);
- }
-}
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/RingerVibrateConditionTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/RingerVibrateConditionCardTest.java
similarity index 74%
rename from tests/robotests/src/com/android/settings/homepage/conditional/RingerVibrateConditionTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/RingerVibrateConditionCardTest.java
index e886236..1f976dd 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/RingerVibrateConditionTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/RingerVibrateConditionCardTest.java
@@ -19,8 +19,6 @@
import static com.google.common.truth.Truth.assertThat;
-import static org.mockito.Mockito.when;
-
import android.content.Context;
import com.android.settings.R;
@@ -29,33 +27,29 @@
import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;
-import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
import org.robolectric.RuntimeEnvironment;
@RunWith(SettingsRobolectricTestRunner.class)
-public class RingerVibrateConditionTest {
- @Mock
- private ConditionManager mConditionManager;
+public class RingerVibrateConditionCardTest {
private Context mContext;
- private RingerVibrateCondition mCondition;
+ private RingerVibrateConditionCard mCard;
@Before
public void setUp() {
MockitoAnnotations.initMocks(this);
mContext = RuntimeEnvironment.application;
- when(mConditionManager.getContext()).thenReturn(mContext);
- mCondition = new RingerVibrateCondition(mConditionManager);
+ mCard = new RingerVibrateConditionCard(mContext);
}
@Test
public void verifyText() {
- assertThat(mCondition.getTitle()).isEqualTo(
+ assertThat(mCard.getTitle()).isEqualTo(
mContext.getText(R.string.condition_device_vibrate_title));
- assertThat(mCondition.getSummary()).isEqualTo(
+ assertThat(mCard.getSummary()).isEqualTo(
mContext.getText(R.string.condition_device_vibrate_summary));
- assertThat(mCondition.getActions()[0]).isEqualTo(
+ assertThat(mCard.getActionText()).isEqualTo(
mContext.getText(R.string.condition_device_muted_action_turn_on_sound));
}
}
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/v2/WorkModeConditionControllerTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/WorkModeConditionControllerTest.java
similarity index 93%
rename from tests/robotests/src/com/android/settings/homepage/conditional/v2/WorkModeConditionControllerTest.java
rename to tests/robotests/src/com/android/settings/homepage/conditional/WorkModeConditionControllerTest.java
index 04d5841..52c9ffe 100644
--- a/tests/robotests/src/com/android/settings/homepage/conditional/v2/WorkModeConditionControllerTest.java
+++ b/tests/robotests/src/com/android/settings/homepage/conditional/WorkModeConditionControllerTest.java
@@ -14,7 +14,7 @@
* limitations under the License.
*/
-package com.android.settings.homepage.conditional.v2;
+package com.android.settings.homepage.conditional;
import static org.mockito.ArgumentMatchers.argThat;
import static org.mockito.Mockito.spy;
@@ -24,6 +24,7 @@
import android.content.Context;
import com.android.settings.Settings;
+import com.android.settings.homepage.conditional.WorkModeConditionController;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import org.junit.Before;
diff --git a/tests/robotests/src/com/android/settings/homepage/conditional/WorkModeConditionTest.java b/tests/robotests/src/com/android/settings/homepage/conditional/WorkModeConditionTest.java
deleted file mode 100644
index c846be0..0000000
--- a/tests/robotests/src/com/android/settings/homepage/conditional/WorkModeConditionTest.java
+++ /dev/null
@@ -1,65 +0,0 @@
-/*
- * Copyright (C) 2018 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.homepage.conditional;
-
-import static org.mockito.ArgumentMatchers.argThat;
-import static org.mockito.Mockito.spy;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
-import android.content.ComponentName;
-import android.content.Context;
-
-import com.android.settings.Settings;
-import com.android.settings.testutils.FakeFeatureFactory;
-import com.android.settings.testutils.SettingsRobolectricTestRunner;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.MockitoAnnotations;
-import org.robolectric.RuntimeEnvironment;
-
-@RunWith(SettingsRobolectricTestRunner.class)
-public class WorkModeConditionTest {
-
- @Mock
- private ConditionManager mConditionManager;
-
- private Context mContext;
- private WorkModeCondition mCondition;
-
- @Before
- public void setUp() {
- MockitoAnnotations.initMocks(this);
- mContext = spy(RuntimeEnvironment.application);
- FakeFeatureFactory.setupForTest();
- when(mConditionManager.getContext()).thenReturn(mContext);
- mCondition = new WorkModeCondition(mConditionManager);
- }
-
- @Test
- public void onPrimaryClick_shouldLaunchAccountsSetting() {
- final ComponentName componentName =
- new ComponentName(mContext, Settings.AccountDashboardActivity.class);
-
- mCondition.onPrimaryClick();
-
- verify(mContext).startActivity(
- argThat(intent-> intent.getComponent().equals(componentName)));
- }
-}