From 617a7804bb2ffcf465f39e0e219951881ee44e57 Mon Sep 17 00:00:00 2001
From: Julia Reynolds <juliacr@google.com>
Date: Fri, 29 Apr 2016 14:32:46 -0400
Subject: [PATCH] Uncheck unselected radio buttons.

Fixes: 28518097
Change-Id: I22cce7837802a97f15af4170f8991db1685ee121
---
 .../android/systemui/volume/ZenModePanel.java | 28 ++++++++++++-------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/packages/SystemUI/src/com/android/systemui/volume/ZenModePanel.java b/packages/SystemUI/src/com/android/systemui/volume/ZenModePanel.java
index e3ed92cb5e882..fda57eb188ff3 100644
--- a/packages/SystemUI/src/com/android/systemui/volume/ZenModePanel.java
+++ b/packages/SystemUI/src/com/android/systemui/volume/ZenModePanel.java
@@ -576,14 +576,27 @@ public class ZenModePanel extends LinearLayout {
         if (DEBUG) Log.d(mTag, "Selecting a default");
         final int favoriteIndex = mPrefs.getMinuteIndex();
         if (favoriteIndex == -1 || !mCountdownConditionSupported) {
-            foreverTag.rb.setChecked(true);
+            setChecked(foreverTag.rb, true);
         } else {
             mTimeCondition = ZenModeConfig.toTimeCondition(mContext,
                     MINUTE_BUCKETS[favoriteIndex], ActivityManager.getCurrentUser());
             mBucketIndex = favoriteIndex;
             bind(mTimeCondition, mZenConditions.getChildAt(COUNTDOWN_CONDITION_INDEX),
                     COUNTDOWN_CONDITION_INDEX);
-            getConditionTagAt(COUNTDOWN_CONDITION_INDEX).rb.setChecked(true);
+            setChecked(getConditionTagAt(COUNTDOWN_CONDITION_INDEX).rb, true);
+        }
+    }
+
+    private void setChecked(RadioButton rb, boolean checked) {
+        final int N = getVisibleConditions();
+        for (int i = 0; i < N; i++) {
+            final ConditionTag tag = getConditionTagAt(i);
+            if (tag != null && tag.rb.isChecked() && !Objects.equals(tag.rb, rb)) {
+                tag.rb.setChecked(false);
+            }
+        }
+        if (rb.isChecked() != checked) {
+            rb.setChecked(checked);
         }
     }
 
@@ -614,13 +627,8 @@ public class ZenModePanel extends LinearLayout {
             @Override
             public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                 if (mExpanded && isChecked) {
+                    setChecked(tag.rb, isChecked);
                     if (DEBUG) Log.d(mTag, "onCheckedChanged " + conditionId);
-                    final int N = getVisibleConditions();
-                    for (int i = 0; i < N; i++) {
-                        final ConditionTag childTag = getConditionTagAt(i);
-                        if (childTag == null || childTag == tag) continue;
-                        childTag.rb.setChecked(false);
-                    }
                     MetricsLogger.action(mContext, MetricsEvent.QS_DND_CONDITION_SELECT);
                     select(tag.condition);
                     announceConditionSelection(tag);
@@ -670,7 +678,7 @@ public class ZenModePanel extends LinearLayout {
         tag.lines.setOnClickListener(new OnClickListener() {
             @Override
             public void onClick(View v) {
-                tag.rb.setChecked(true);
+                setChecked(tag.rb, true);
             }
         });
 
@@ -759,7 +767,7 @@ public class ZenModePanel extends LinearLayout {
         }
         mTimeCondition = newCondition;
         bind(mTimeCondition, row, rowId);
-        tag.rb.setChecked(true);
+        setChecked(tag.rb, true);
         select(mTimeCondition);
         announceConditionSelection(tag);
     }