diff --git a/src/com/android/settings/core/FeatureFlags.java b/src/com/android/settings/core/FeatureFlags.java index c8ae82f4859..c6b9d836828 100644 --- a/src/com/android/settings/core/FeatureFlags.java +++ b/src/com/android/settings/core/FeatureFlags.java @@ -25,5 +25,4 @@ public class FeatureFlags { 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 5a8df911373..5e2e57da18d 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; 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 class DashboardAdapter extends RecyclerView.Adapter conditions, ConditionManager conditionManager, + ConditionManager conditionManager, SuggestionControllerMixinCompat suggestionControllerMixin, Lifecycle lifecycle) { DashboardCategory category = null; @@ -116,8 +115,7 @@ public class DashboardAdapter extends RecyclerView.Adapter conditions) { + public void setConditions(List conditions) { final DashboardData prevData = mDashboardData; Log.d(TAG, "adapter setConditions called"); mDashboardData = new DashboardData.Builder(prevData) @@ -151,15 +149,6 @@ public class DashboardAdapter extends RecyclerView.Adapter 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 list = mDashboardData.getSuggestions(); @@ -301,31 +290,12 @@ public class DashboardAdapter extends RecyclerView.Adapter conditionsV1; - final List 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 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 9feff580e8d..db296ab7470 100644 --- a/src/com/android/settings/dashboard/DashboardData.java +++ b/src/com/android/settings/dashboard/DashboardData.java @@ -24,8 +24,7 @@ import androidx.annotation.VisibleForTesting; 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 @@ public class DashboardData { private final List mItems; private final DashboardCategory mCategory; - private final List mConditions; - private final List mConditionsV2; + private final List mConditions; private final List 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 @@ public class DashboardData { return mCategory; } - public List getConditions() { + public List getConditions() { return mConditions; } @@ -168,7 +165,8 @@ public class DashboardData { /** * 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 @@ public class DashboardData { * and mIsShowingAll, mConditionExpanded flag. */ private void buildItemsData() { - final List conditionsV1 = getConditionsToShow(mConditions); - final boolean hasConditionsV1 = sizeOf(conditionsV1) > 0; - final List conditionsV2 = mConditionsV2; - final boolean hasConditionsV2 = sizeOf(conditionsV2) > 0; - final boolean hasConditions = hasConditionsV1 || hasConditionsV2; + final List conditions = mConditions; + final boolean hasConditions = sizeOf(conditions) > 0; final List suggestions = getSuggestionsToShow(mSuggestions); final boolean hasSuggestions = sizeOf(suggestions) > 0; @@ -203,20 +198,14 @@ public class DashboardData { 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 @@ public class DashboardData { return list == null ? 0 : list.size(); } - private List getConditionsToShow(List conditions) { - if (conditions == null) { - return null; - } - List 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 getSuggestionsToShow(List suggestions) { if (suggestions == null) { return null; @@ -270,8 +244,7 @@ public class DashboardData { */ public static class Builder { private DashboardCategory mCategory; - private List mConditions; - private List mConditionsV2; + private List mConditions; private List mSuggestions; private boolean mConditionExpanded; @@ -281,7 +254,6 @@ public class DashboardData { public Builder(DashboardData dashboardData) { mCategory = dashboardData.mCategory; mConditions = dashboardData.mConditions; - mConditionsV2 = dashboardData.mConditionsV2; mSuggestions = dashboardData.mSuggestions; mConditionExpanded = dashboardData.mConditionExpanded; } @@ -291,16 +263,11 @@ public class DashboardData { return this; } - public Builder setConditions(List conditions) { + public Builder setConditions(List conditions) { this.mConditions = conditions; return this; } - public Builder setConditionsV2(List conditions) { - this.mConditionsV2 = conditions; - return this; - } - public Builder setSuggestions(List suggestions) { this.mSuggestions = suggestions; return this; @@ -374,7 +341,7 @@ public class DashboardData { } /** - * 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 class DashboardData { public final CharSequence title; public final int conditionCount; - public ConditionHeaderData(List conditions, List 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 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 18f6999cf6d..4575482f2d6 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.InstrumentedFragment; 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 @@ public class DashboardSummary extends InstrumentedFragment 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 @@ public class DashboardSummary extends InstrumentedFragment 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 @@ public class DashboardSummary extends InstrumentedFragment ((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 @@ public class DashboardSummary extends InstrumentedFragment ((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 @@ public class DashboardSummary extends InstrumentedFragment 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 @@ public class DashboardSummary extends InstrumentedFragment // 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 b6a67af3015..00000000000 --- 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 1cc62b944c2..c6fa21d38c0 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.content.IntentFilter; 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 069d316d81e..00000000000 --- 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 99055a749fa..f56fcc34d98 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 2d200ecdba6..db3bc71375b 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 4e374ef1c6a..00000000000 --- 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 3ea31130d8d..13792a9634d 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 3649607879b..d7c06987e74 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 f36dcb7cf8d..00000000000 --- 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 97% rename from src/com/android/settings/homepage/conditional/v2/BatterySaverConditionCard.java rename to src/com/android/settings/homepage/conditional/BatterySaverConditionCard.java index c30715f5bd4..ea84cb2f521 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 98% rename from src/com/android/settings/homepage/conditional/v2/BatterySaverConditionController.java rename to src/com/android/settings/homepage/conditional/BatterySaverConditionController.java index 167dc12c614..fdbe69866f4 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 40729128c57..00000000000 --- 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 c09e75a4f62..f135c4f8664 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 d23dc343501..c5c92c22e54 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 f91e422aa6c..00000000000 --- 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 e975146d095..a87e4bc3eca 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 { - public static final String TAG = "ConditionAdapter"; +public class ConditionAdapter extends RecyclerView.Adapter { private final Context mContext; private final MetricsFeatureProvider mMetricsFeatureProvider; - private List mConditions; - private boolean mExpanded; + private final ConditionManager mConditionManager; + private final List 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 conditions, boolean expanded) { + public ConditionAdapter(Context context, ConditionManager conditionManager, + List conditions, boolean expanded) { mContext = context; + mConditionManager = conditionManager; mConditions = conditions; mExpanded = expanded; mMetricsFeatureProvider = FeatureFactory.getFactory(context).getMetricsFeatureProvider(); @@ -91,30 +52,22 @@ public class ConditionAdapter extends RecyclerView.Adapter 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 @@ public class ConditionAdapter extends RecyclerView.Adapter 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 dbb04ae7f37..d6e50123a52 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 mCandidates; + @VisibleForTesting + final List 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 mConditions; - private File mXmlFile; - - private final ArrayList 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 getDisplayableCards() { + final List cards = new ArrayList<>(); + final List> 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 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 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 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 clz, ArrayList 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 getCondition(Class clz) { - return getCondition(clz, mConditions); + /** + * Called when some conditional card's state has changed + */ + void onConditionChanged() { + if (mListener != null) { + mListener.onConditionsChanged(); + } } - private T getCondition(Class clz, List 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 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 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 { - 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> { - @Override - protected ArrayList doInBackground(Void... params) { - Log.d(TAG, "loading conditions from xml"); - ArrayList 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 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_COMPARATOR = new Comparator() { - @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 da832b7db86..f4c4547fe6d 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 7919d7356b4..9993f343731 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 d2386136ebe..00000000000 --- 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 8e0fdd59aaf..dbc0d8394e9 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 @@ import com.android.settings.R; 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 9e51ef95efe..1729815ffd3 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 6934d0f304e..00000000000 --- 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 63b5034729d..a6ff224d6e0 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 56e95d1c4ed..42711366d24 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 384f593ec4f..00000000000 --- 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 97% rename from src/com/android/settings/homepage/conditional/v2/NightDisplayConditionCard.java rename to src/com/android/settings/homepage/conditional/NightDisplayConditionCard.java index b08f8e721e5..e0c6ee0a57d 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 0d612188c5e..428fe488989 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 d3e618e5b59..00000000000 --- 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 97% rename from src/com/android/settings/homepage/conditional/v2/RingerMutedConditionCard.java rename to src/com/android/settings/homepage/conditional/RingerMutedConditionCard.java index c9a1ba062a7..47d64077991 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 97% rename from src/com/android/settings/homepage/conditional/v2/RingerMutedConditionController.java rename to src/com/android/settings/homepage/conditional/RingerMutedConditionController.java index 19850ae1c43..4fce1ec9b03 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 92667c2117c..00000000000 --- 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 97% rename from src/com/android/settings/homepage/conditional/v2/RingerVibrateConditionCard.java rename to src/com/android/settings/homepage/conditional/RingerVibrateConditionCard.java index 3ca3a4de552..229cc83f5ec 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 d2d32447053..43de0803e7e 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 31266a4b512..00000000000 --- 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 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 10bcd6d7a11..c283bece726 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 73689c7f557..1bd227a0912 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 51d8c47e18c..00000000000 --- 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 { - - private final Context mContext; - private final MetricsFeatureProvider mMetricsFeatureProvider; - private final ConditionManager mConditionManager; - private final List mConditions; - private final boolean mExpanded; - - public ConditionAdapter(Context context, ConditionManager conditionManager, - List 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 c67b255d4f0..00000000000 --- 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 mCandidates; - @VisibleForTesting - final List 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 getDisplayableCards() { - final List cards = new ArrayList<>(); - final List> 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 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 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 { - - 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 58a5cc9dc6f..f544f09db97 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 androidx.recyclerview.widget.RecyclerView; 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 @@ public class DashboardAdapterTest { @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 mConditionList; + private List 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 @@ public class DashboardAdapterTest { 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 suggestions = makeSuggestions("pkg1", "pkg2", "pkg3"); @@ -149,7 +147,7 @@ public class DashboardAdapterTest { 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 suggestions = makeSuggestions("pkg1"); adapter.setSuggestions(suggestions); @@ -166,7 +164,7 @@ public class DashboardAdapterTest { 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 suggestions = makeSuggestions("pkg1"); adapter.setSuggestions(suggestions); @@ -181,8 +179,7 @@ public class DashboardAdapterTest { @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 suggestions = makeSuggestions("pkg1"); mDashboardAdapter.setSuggestions(suggestions); @@ -218,8 +215,7 @@ public class DashboardAdapterTest { .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 @@ public class DashboardAdapterTest { 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 @@ public class DashboardAdapterTest { 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 406cb3cd1df..73b1734212b 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.annotation.NonNull; 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 @@ public class DashboardDataTest { @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 @@ public class DashboardDataTest { suggestions.add(mTestSuggestion); // Build oneItemConditions - final List oneItemConditions = new ArrayList<>(); - when(mTestCondition.shouldShow()).thenReturn(true); + final List oneItemConditions = new ArrayList<>(); oneItemConditions.add(mTestCondition); // Build twoItemConditions - final List twoItemsConditions = new ArrayList<>(); - when(mSecondCondition.shouldShow()).thenReturn(true); + final List twoItemsConditions = new ArrayList<>(); twoItemsConditions.add(mTestCondition); twoItemsConditions.add(mSecondCondition); @@ -170,7 +168,7 @@ public class DashboardDataTest { @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 @@ public class DashboardDataTest { testResultData.add(new ListUpdateResult.ResultData( ListUpdateResult.ResultData.TYPE_OPERATION_CHANGE, 2, 1)); // Build DashboardData - final List oneItemConditions = new ArrayList<>(); - when(mTestCondition.shouldShow()).thenReturn(true); + final List oneItemConditions = new ArrayList<>(); + oneItemConditions.add(mTestCondition); final List 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 4eaeaf23e10..1759fffc9ca 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.spy; 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 dc06710335e..0b1772802be 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.Context; 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 b12d786067a..4ad7e6dee7b 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.Context; 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 8849e2d58f3..9dd6780181d 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.content.IntentFilter; 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 323e6c5ff91..00000000000 --- 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 663c630f3db..4d154da682a 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 @@ package com.android.settings.homepage.conditional; 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.LayoutInflater; 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 @@ import java.util.List; 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 mOneCondition; - private List mTwoConditions; + private List mOneCondition; + private List 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 @@ public class ConditionAdapterTest { @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 @@ public class ConditionAdapterTest { @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 2f96eafb05b..f90c4489540 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 static org.mockito.Mockito.when; 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 981ef662ef5..00000000000 --- 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 9da1f9c3bb5..00000000000 --- 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 91% 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 fe4c621cdd4..c68f8999fc9 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 static org.mockito.Mockito.verify; 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 1dfbd294a87..47a3010babc 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 static org.mockito.Mockito.when; 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 408aa137891..00000000000 --- 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 e886236e209..1f976ddd118 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 @@ package com.android.settings.homepage.conditional; 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 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 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 04d58412883..52c9ffedbf0 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.ComponentName; 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 c846be0f854..00000000000 --- 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))); - } -}