From 6eabafeacd6b97067982ed464f0db2698928dd9e Mon Sep 17 00:00:00 2001
From: Julia Reynolds <juliacr@google.com>
Date: Tue, 1 May 2018 16:57:50 -0400
Subject: [PATCH] Update onboarding

Change text and add 'repeat callers'

Test: robotests
Change-Id: I4b7125a2a9d70cf5ec7e4d184528a444d7d2bcf1
Fixes: 78448988
---
 res/values/strings.xml                                       |  4 ++--
 .../android/settings/notification/ZenOnboardingActivity.java | 12 ++++++------
 .../settings/notification/ZenOnboardingActivityTest.java     |  4 +++-
 3 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/res/values/strings.xml b/res/values/strings.xml
index 79c1f0e0d3..e5794feeed 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -7357,9 +7357,9 @@
     <!-- Do not disturb onboarding dialog, do not accept new settings [CHAR LIMIT=30]-->
     <string name="zen_onboarding_no_update">Don\'t update</string>
     <!-- Do not disturb onboarding dialog, description of new settings [CHAR LIMIT=NONE]-->
-    <string name="zen_onboarding_dnd_visual_disturbances_description">Your phone mutes sounds &amp; vibrations\n\nUpdate settings to also:\n\n- Hide notifications\n\n- Only show calls from starred contacts</string>
+    <string name="zen_onboarding_dnd_visual_disturbances_description">Your phone can do more to help you focus.\n\nUpdate settings to:\n\n- Hide notifications completely\n\n- Allow calls from starred contacts and repeat callers</string>
     <!-- Do not disturb onboarding dialog, header prompt for settings [CHAR LIMIT=80]-->
-    <string name="zen_onboarding_dnd_visual_disturbances_header">Update your Do Not Disturb settings?</string>
+    <string name="zen_onboarding_dnd_visual_disturbances_header">Update Do Not Disturb?</string>
 
     <!-- obsolete -->
     <string name="zen_onboarding_settings">Settings</string>
diff --git a/src/com/android/settings/notification/ZenOnboardingActivity.java b/src/com/android/settings/notification/ZenOnboardingActivity.java
index 922299e9ae..2215a1d9d7 100644
--- a/src/com/android/settings/notification/ZenOnboardingActivity.java
+++ b/src/com/android/settings/notification/ZenOnboardingActivity.java
@@ -18,12 +18,11 @@ package com.android.settings.notification;
 
 import android.app.Activity;
 import android.app.NotificationManager;
-import android.content.Intent;
+import android.app.NotificationManager.Policy;
 import android.os.Bundle;
 import android.provider.Settings;
 import androidx.annotation.VisibleForTesting;
 import android.view.View;
-import android.widget.CheckBox;
 
 import com.android.internal.logging.MetricsLogger;
 import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
@@ -67,12 +66,13 @@ public class ZenOnboardingActivity extends Activity {
 
     public void save(View button) {
         mMetrics.action(MetricsEvent.ACTION_ZEN_ONBOARDING_OK);
-        NotificationManager.Policy policy = mNm.getNotificationPolicy();
+        Policy policy = mNm.getNotificationPolicy();
 
-        NotificationManager.Policy newPolicy = new NotificationManager.Policy(
-                policy.priorityCategories, NotificationManager.Policy.PRIORITY_SENDERS_STARRED,
+        Policy newPolicy = new NotificationManager.Policy(
+                Policy.PRIORITY_CATEGORY_REPEAT_CALLERS | policy.priorityCategories,
+                Policy.PRIORITY_SENDERS_STARRED,
                 policy.priorityMessageSenders,
-                NotificationManager.Policy.getAllSuppressedVisualEffects());
+                Policy.getAllSuppressedVisualEffects());
         mNm.setNotificationPolicy(newPolicy);
 
         finishAndRemoveTask();
diff --git a/tests/robotests/src/com/android/settings/notification/ZenOnboardingActivityTest.java b/tests/robotests/src/com/android/settings/notification/ZenOnboardingActivityTest.java
index 57dc855a78..7bc9371397 100644
--- a/tests/robotests/src/com/android/settings/notification/ZenOnboardingActivityTest.java
+++ b/tests/robotests/src/com/android/settings/notification/ZenOnboardingActivityTest.java
@@ -17,6 +17,7 @@
 package com.android.settings.notification;
 
 import static android.app.NotificationManager.Policy.PRIORITY_CATEGORY_ALARMS;
+import static android.app.NotificationManager.Policy.PRIORITY_CATEGORY_REPEAT_CALLERS;
 import static android.app.NotificationManager.Policy.PRIORITY_SENDERS_ANY;
 import static android.app.NotificationManager.Policy.SUPPRESSED_EFFECT_AMBIENT;
 import static android.app.NotificationManager.Policy.SUPPRESSED_EFFECT_BADGE;
@@ -94,7 +95,8 @@ public class ZenOnboardingActivityTest {
         verify(mNm).setNotificationPolicy(captor.capture());
 
         Policy actual = captor.getValue();
-        assertThat(actual.priorityCategories).isEqualTo(PRIORITY_CATEGORY_ALARMS);
+        assertThat(actual.priorityCategories).isEqualTo(PRIORITY_CATEGORY_ALARMS
+                | PRIORITY_CATEGORY_REPEAT_CALLERS);
         assertThat(actual.priorityCallSenders).isEqualTo(Policy.PRIORITY_SENDERS_STARRED);
         assertThat(actual.priorityMessageSenders).isEqualTo(Policy.PRIORITY_SENDERS_ANY);
         assertThat(actual.suppressedVisualEffects).isEqualTo(
-- 
2.11.0