commit 75b927b4f2d4193d77c1721d8415aadac33f5b4e
author Vladislav Kaznacheev <kaznacheev@google.com> Mon Feb 04 18:54:40 2019
committer Commit Bot <commit-bot@chromium.org> Mon Feb 04 18:54:40 2019
tree e23e458d0a5f533321087676428e57c7bb73a140
parent 4351bedbce9b43cba24255a278dbb5314043c9a6

Fix volume button state

The spec requires the volume button to be grayed out when muted and
blue when not muted. The current behavior is the reverse.

Bug: 863488
Test: manual
Change-Id: Id2c794484761bbb46eef4ab805e77eed38b45b9c
Reviewed-on: https://chromium-review.googlesource.com/c/1448996
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Auto-Submit: Vladislav Kaznacheev <kaznacheev@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#628811}

ash/system/audio/unified_volume_view.cc

diff --git a/ash/system/audio/unified_volume_view.cc b/ash/system/audio/unified_volume_view.cc
index c280c64..6fe3630 100644
--- a/ash/system/audio/unified_volume_view.cc
+++ b/ash/system/audio/unified_volume_view.cc
@@ -143,7 +143,8 @@
   // Indicate that the slider is inactive when it's muted.
   slider()->UpdateState(!is_muted);
 
-  button()->SetToggled(is_muted);
+  // The button should be gray whay muted and colored otherwise.
+  button()->SetToggled(!is_muted);
   button()->SetVectorIcon(is_muted ? kUnifiedMenuVolumeMuteIcon
                                    : GetVolumeIconForLevel(level));