Remove DND and all gesture suggestions from Settings

Change-Id: I2f59c426132ca907e556b8bb2fbc869f4e830768
Fix: 64156274
Test: robotests
This commit is contained in:
Fan Zhang
2017-07-28 14:20:23 -07:00
parent c50495ba06
commit 4f04c68dd1
5 changed files with 6 additions and 73 deletions

View File

@@ -772,25 +772,6 @@
android:value="true" /> android:value="true" />
</activity> </activity>
<activity android:name="Settings$ZenModeAutomationSuggestionActivity"
android:label="@string/zen_mode_automation_settings_title"
android:icon="@drawable/ic_settings_notifications"
android:exported="true"
android:taskAffinity="">
<intent-filter>
<action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.FIRST_IMPRESSION" />
</intent-filter>
<meta-data android:name="com.android.settings.FRAGMENT_CLASS"
android:value="com.android.settings.notification.ZenModeSettings" />
<meta-data android:name="com.android.settings.dismiss"
android:value="8,12,30" />
<meta-data android:name="com.android.settings.title"
android:resource="@string/zen_mode_automation_suggestion_title" />
<meta-data android:name="com.android.settings.summary"
android:resource="@string/zen_mode_automation_suggestion_summary" />
</activity>
<activity android:name="Settings$WallpaperSettingsActivity" <activity android:name="Settings$WallpaperSettingsActivity"
android:label="@string/wallpaper_settings_fragment_title" android:label="@string/wallpaper_settings_fragment_title"
android:icon="@drawable/ic_wallpaper" android:icon="@drawable/ic_wallpaper"
@@ -3160,10 +3141,6 @@
<action android:name="android.intent.action.MAIN" /> <action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.GESTURE" /> <category android:name="com.android.settings.suggested.category.GESTURE" />
</intent-filter> </intent-filter>
<intent-filter>
<action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.FIRST_IMPRESSION" />
</intent-filter>
<meta-data android:name="com.android.settings.dismiss" <meta-data android:name="com.android.settings.dismiss"
android:value="4,8,30" /> android:value="4,8,30" />
<meta-data android:name="com.android.settings.FRAGMENT_CLASS" <meta-data android:name="com.android.settings.FRAGMENT_CLASS"
@@ -3182,10 +3159,6 @@
<action android:name="android.intent.action.MAIN" /> <action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.GESTURE" /> <category android:name="com.android.settings.suggested.category.GESTURE" />
</intent-filter> </intent-filter>
<intent-filter>
<action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.FIRST_IMPRESSION" />
</intent-filter>
<meta-data android:name="com.android.settings.dismiss" <meta-data android:name="com.android.settings.dismiss"
android:value="4,8,30" /> android:value="4,8,30" />
<meta-data android:name="com.android.settings.FRAGMENT_CLASS" <meta-data android:name="com.android.settings.FRAGMENT_CLASS"
@@ -3204,10 +3177,6 @@
<action android:name="android.intent.action.MAIN" /> <action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.GESTURE" /> <category android:name="com.android.settings.suggested.category.GESTURE" />
</intent-filter> </intent-filter>
<intent-filter>
<action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.FIRST_IMPRESSION" />
</intent-filter>
<meta-data android:name="com.android.settings.dismiss" <meta-data android:name="com.android.settings.dismiss"
android:value="9,13,30" /> android:value="9,13,30" />
<meta-data android:name="com.android.settings.FRAGMENT_CLASS" <meta-data android:name="com.android.settings.FRAGMENT_CLASS"
@@ -3226,10 +3195,6 @@
<action android:name="android.intent.action.MAIN" /> <action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.GESTURE" /> <category android:name="com.android.settings.suggested.category.GESTURE" />
</intent-filter> </intent-filter>
<intent-filter>
<action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.FIRST_IMPRESSION" />
</intent-filter>
<meta-data android:name="com.android.settings.dismiss" <meta-data android:name="com.android.settings.dismiss"
android:value="9,13,30" /> android:value="9,13,30" />
<meta-data android:name="com.android.settings.FRAGMENT_CLASS" <meta-data android:name="com.android.settings.FRAGMENT_CLASS"
@@ -3248,10 +3213,6 @@
<action android:name="android.intent.action.MAIN" /> <action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.GESTURE" /> <category android:name="com.android.settings.suggested.category.GESTURE" />
</intent-filter> </intent-filter>
<intent-filter>
<action android:name="android.intent.action.MAIN" />
<category android:name="com.android.settings.suggested.category.FIRST_IMPRESSION" />
</intent-filter>
<meta-data android:name="com.android.settings.dismiss" <meta-data android:name="com.android.settings.dismiss"
android:value="9,13,30" /> android:value="9,13,30" />
<meta-data android:name="com.android.settings.FRAGMENT_CLASS" <meta-data android:name="com.android.settings.FRAGMENT_CLASS"

View File

@@ -141,7 +141,6 @@ public class Settings extends SettingsActivity {
public static class ManageAppExternalSourcesActivity extends SettingsActivity { /* empty */ } public static class ManageAppExternalSourcesActivity extends SettingsActivity { /* empty */ }
public static class WifiCallingSuggestionActivity extends SettingsActivity { /* empty */ } public static class WifiCallingSuggestionActivity extends SettingsActivity { /* empty */ }
public static class ZenModeAutomationSuggestionActivity extends SettingsActivity { /* empty */ }
public static class FingerprintEnrollSuggestionActivity extends FingerprintEnrollIntroduction { public static class FingerprintEnrollSuggestionActivity extends FingerprintEnrollIntroduction {
/* empty */ /* empty */
} }

View File

@@ -16,30 +16,24 @@
package com.android.settings.dashboard.suggestions; package com.android.settings.dashboard.suggestions;
import android.app.AutomaticZenRule;
import android.app.KeyguardManager; import android.app.KeyguardManager;
import android.app.NotificationManager;
import android.app.WallpaperManager; import android.app.WallpaperManager;
import android.app.admin.DevicePolicyManager; import android.app.admin.DevicePolicyManager;
import android.content.ComponentName; import android.content.ComponentName;
import android.content.Context; import android.content.Context;
import android.hardware.fingerprint.FingerprintManager; import android.hardware.fingerprint.FingerprintManager;
import android.provider.Settings;
import android.support.annotation.VisibleForTesting; import android.support.annotation.VisibleForTesting;
import com.android.ims.ImsManager; import com.android.ims.ImsManager;
import com.android.settings.Settings.FingerprintEnrollSuggestionActivity; import com.android.settings.Settings.FingerprintEnrollSuggestionActivity;
import com.android.settings.Settings.ScreenLockSuggestionActivity; import com.android.settings.Settings.ScreenLockSuggestionActivity;
import com.android.settings.Settings.WifiCallingSuggestionActivity; import com.android.settings.Settings.WifiCallingSuggestionActivity;
import com.android.settings.Settings.ZenModeAutomationSuggestionActivity;
import com.android.settings.Utils; import com.android.settings.Utils;
import com.android.settings.fingerprint.FingerprintSuggestionActivity; import com.android.settings.fingerprint.FingerprintSuggestionActivity;
import com.android.settings.overlay.FeatureFactory; import com.android.settings.overlay.FeatureFactory;
import com.android.settings.wallpaper.WallpaperSuggestionActivity; import com.android.settings.wallpaper.WallpaperSuggestionActivity;
import com.android.settingslib.drawer.Tile; import com.android.settingslib.drawer.Tile;
import java.util.Collection;
/** /**
* The Home of all stupidly dynamic Settings Suggestions checks. * The Home of all stupidly dynamic Settings Suggestions checks.
*/ */
@@ -58,9 +52,7 @@ public class SuggestionsChecks {
public boolean isSuggestionComplete(Tile suggestion) { public boolean isSuggestionComplete(Tile suggestion) {
ComponentName component = suggestion.intent.getComponent(); ComponentName component = suggestion.intent.getComponent();
String className = component.getClassName(); String className = component.getClassName();
if (className.equals(ZenModeAutomationSuggestionActivity.class.getName())) { if (className.equals(WallpaperSuggestionActivity.class.getName())) {
return hasEnabledZenAutoRules();
} else if (className.equals(WallpaperSuggestionActivity.class.getName())) {
return hasWallpaperSet(); return hasWallpaperSet();
} else if (className.equals(WifiCallingSuggestionActivity.class.getName())) { } else if (className.equals(WifiCallingSuggestionActivity.class.getName())) {
return isWifiCallingUnavailableOrEnabled(); return isWifiCallingUnavailableOrEnabled();
@@ -103,17 +95,6 @@ public class SuggestionsChecks {
&& ImsManager.isNonTtyOrTtyOnVolteEnabled(mContext); && ImsManager.isNonTtyOrTtyOnVolteEnabled(mContext);
} }
private boolean hasEnabledZenAutoRules() {
Collection<AutomaticZenRule> zenRules =
NotificationManager.from(mContext).getAutomaticZenRules().values();
for (AutomaticZenRule rule : zenRules) {
if (rule.isEnabled()) {
return true;
}
}
return false;
}
@VisibleForTesting @VisibleForTesting
boolean hasWallpaperSet() { boolean hasWallpaperSet() {
return mWallpaperManager.getWallpaperId(WallpaperManager.FLAG_SYSTEM) > 0; return mWallpaperManager.getWallpaperId(WallpaperManager.FLAG_SYSTEM) > 0;

View File

@@ -31,8 +31,8 @@ import android.content.pm.PackageManager;
import android.hardware.fingerprint.FingerprintManager; import android.hardware.fingerprint.FingerprintManager;
import com.android.settings.Settings; import com.android.settings.Settings;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import com.android.settings.TestConfig; import com.android.settings.TestConfig;
import com.android.settings.testutils.SettingsRobolectricTestRunner;
import com.android.settingslib.drawer.Tile; import com.android.settingslib.drawer.Tile;
import org.junit.Before; import org.junit.Before;

View File

@@ -16,13 +16,16 @@
package com.android.settings.suggestions; package com.android.settings.suggestions;
import static com.android.settings.TestConfig.MANIFEST_PATH;
import static com.google.common.truth.Truth.assertThat;
import android.annotation.StringRes; import android.annotation.StringRes;
import android.content.Context; import android.content.Context;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.TestConfig;
import com.android.settings.fingerprint.FingerprintSuggestionActivity; import com.android.settings.fingerprint.FingerprintSuggestionActivity;
import com.android.settings.testutils.SettingsRobolectricTestRunner; import com.android.settings.testutils.SettingsRobolectricTestRunner;
import com.android.settings.TestConfig;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
@@ -37,9 +40,6 @@ import java.util.ArrayList;
import java.util.List; import java.util.List;
import java.util.Map; import java.util.Map;
import static com.android.settings.TestConfig.MANIFEST_PATH;
import static com.google.common.truth.Truth.assertThat;
@RunWith(SettingsRobolectricTestRunner.class) @RunWith(SettingsRobolectricTestRunner.class)
@Config(manifest = MANIFEST_PATH, sdk = TestConfig.SDK_VERSION) @Config(manifest = MANIFEST_PATH, sdk = TestConfig.SDK_VERSION)
public class SettingsSuggestionsTest { public class SettingsSuggestionsTest {
@@ -50,14 +50,6 @@ public class SettingsSuggestionsTest {
private static final String CATEGORY_SETTINGS_ONLY = private static final String CATEGORY_SETTINGS_ONLY =
"com.android.settings.suggested.category.SETTINGS_ONLY"; "com.android.settings.suggested.category.SETTINGS_ONLY";
@Test
public void zenModeAutomationSuggestion_isValid() {
assertSuggestionEquals("Settings$ZenModeAutomationSuggestionActivity",
CATEGORY_FIRST_IMPRESSION,
R.string.zen_mode_automation_suggestion_title,
R.string.zen_mode_automation_suggestion_summary);
}
@Test @Test
public void wallpaperSuggestion_isValid() { public void wallpaperSuggestion_isValid() {
assertSuggestionEquals("com.android.settings.wallpaper.WallpaperSuggestionActivity", assertSuggestionEquals("com.android.settings.wallpaper.WallpaperSuggestionActivity",