List all a11y featuers in ash system menu

ash/common/system/tray_accessibility.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/common/system/tray_accessibility.h"
 
 #include "ash/common/accessibility_delegate.h"
 #include "ash/common/accessibility_types.h"
 #include "ash/common/session/session_state_delegate.h"
 #include "ash/common/system/tray/hover_highlight_view.h"
@@ skipping 14 matching lines @@
 #include "ash/strings/grit/ash_strings.h"
 #include "base/strings/utf_string_conversions.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/color_palette.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/views/controls/button/custom_button.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
+#include "ui/views/controls/separator.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/widget/widget.h"
 
 namespace ash {
 namespace {
 
 enum AccessibilityState {
   A11Y_NONE = 0,
   A11Y_SPOKEN_FEEDBACK = 1 << 0,
   A11Y_HIGH_CONTRAST = 1 << 1,
   A11Y_SCREEN_MAGNIFIER = 1 << 2,
   A11Y_LARGE_CURSOR = 1 << 3,
   A11Y_AUTOCLICK = 1 << 4,
   A11Y_VIRTUAL_KEYBOARD = 1 << 5,
   A11Y_BRAILLE_DISPLAY_CONNECTED = 1 << 6,
+  A11Y_MONO_AUDIO = 1 << 7,
+  A11Y_CARET_HIGHLIGHT = 1 << 8,
+  A11Y_HIGHLIGHT_MOUSE_CURSOR = 1 << 9,
+  A11Y_HIGHLIGHT_KEYBOARD_FOCUS = 1 << 10,
 };
 
 uint32_t GetAccessibilityState() {
   AccessibilityDelegate* delegate =
       Shell::GetInstance()->accessibility_delegate();
   uint32_t state = A11Y_NONE;
   if (delegate->IsSpokenFeedbackEnabled())
     state |= A11Y_SPOKEN_FEEDBACK;
   if (delegate->IsHighContrastEnabled())
     state |= A11Y_HIGH_CONTRAST;
   if (delegate->IsMagnifierEnabled())
     state |= A11Y_SCREEN_MAGNIFIER;
   if (delegate->IsLargeCursorEnabled())
     state |= A11Y_LARGE_CURSOR;
   if (delegate->IsAutoclickEnabled())
     state |= A11Y_AUTOCLICK;
   if (delegate->IsVirtualKeyboardEnabled())
     state |= A11Y_VIRTUAL_KEYBOARD;
   if (delegate->IsBrailleDisplayConnected())
     state |= A11Y_BRAILLE_DISPLAY_CONNECTED;
+  if (delegate->IsMonoAudioEnabled())
+    state |= A11Y_MONO_AUDIO;
+  if (delegate->IsCaretHighlightEnabled())
+    state |= A11Y_CARET_HIGHLIGHT;
+  if (delegate->IsCursorHighlightEnabled())
+    state |= A11Y_HIGHLIGHT_MOUSE_CURSOR;
+  if (delegate->IsFocusHighlightEnabled())
+    state |= A11Y_HIGHLIGHT_KEYBOARD_FOCUS;
   return state;
 }
 
 LoginStatus GetCurrentLoginStatus() {
   return Shell::Get()->system_tray_delegate()->GetUserLoginStatus();
 }
 
+void UpdateCheckMark(HoverHighlightView* container, bool checked) {

[tdanderson, 2017/03/29 18:57:03]

Thanks for moving this into a helper function.

It looks like this code is also duplicated in some other detailed views, (see
audio_detailed_view.cc as an example) and it is always good to share code
wherever possible. Don't worry about changing it in this CL, but I will make a
note of this and other refactoring opportunities in a follow-on bug.

+  if (checked) {
+    gfx::ImageSkia check_mark =
+        CreateVectorIcon(kCheckCircleIcon, gfx::kGoogleGreen700);
+    container->AddRightIcon(check_mark, check_mark.width());
+    container->SetRightViewVisible(true);
+    container->SetAccessiblityState(
+        HoverHighlightView::AccessibilityState::CHECKED_CHECKBOX);
+  } else {
+    container->SetAccessiblityState(
+        HoverHighlightView::AccessibilityState::UNCHECKED_CHECKBOX);
+  }
+}

[xiyuan, 2017/03/29 17:59:07]

nit: insert a blank line after this line

 }  // namespace
 
 namespace tray {
 
 class DefaultAccessibilityView : public TrayItemMore {
  public:
   explicit DefaultAccessibilityView(SystemTrayItem* owner)
       : TrayItemMore(owner) {
     base::string16 label =
         l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_ACCESSIBILITY);
@@ skipping 45 matching lines @@
   label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   return label;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ash::tray::AccessibilityDetailedView
 
 AccessibilityDetailedView::AccessibilityDetailedView(SystemTrayItem* owner,
                                                      LoginStatus login)
     : TrayDetailsView(owner),
-      spoken_feedback_view_(nullptr),
-      high_contrast_view_(nullptr),
-      screen_magnifier_view_(nullptr),
-      large_cursor_view_(nullptr),
-      help_view_(nullptr),
-      settings_view_(nullptr),
-      autoclick_view_(nullptr),
-      virtual_keyboard_view_(nullptr),
-      spoken_feedback_enabled_(false),
-      high_contrast_enabled_(false),
-      screen_magnifier_enabled_(false),
-      large_cursor_enabled_(false),
-      autoclick_enabled_(false),
-      virtual_keyboard_enabled_(false),
       login_(login) {
   Reset();
   AppendAccessibilityList();
   CreateTitleRow(IDS_ASH_STATUS_TRAY_ACCESSIBILITY_TITLE);
   Layout();
 }
 
 void AccessibilityDetailedView::AppendAccessibilityList() {
   CreateScrollableList();
   ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
 
   AccessibilityDelegate* delegate =
       Shell::GetInstance()->accessibility_delegate();
   spoken_feedback_enabled_ = delegate->IsSpokenFeedbackEnabled();
   spoken_feedback_view_ =
       AddScrollListItem(bundle.GetLocalizedString(
                             IDS_ASH_STATUS_TRAY_ACCESSIBILITY_SPOKEN_FEEDBACK),
                         spoken_feedback_enabled_, spoken_feedback_enabled_,
                         kSystemMenuAccessibilityChromevoxIcon);
 
-  // Large Cursor item is shown only in Login screen.
-  if (login_ == LoginStatus::NOT_LOGGED_IN) {
-    large_cursor_enabled_ = delegate->IsLargeCursorEnabled();
-    large_cursor_view_ =
-        AddScrollListItem(bundle.GetLocalizedString(
-                              IDS_ASH_STATUS_TRAY_ACCESSIBILITY_LARGE_CURSOR),
-                          large_cursor_enabled_, large_cursor_enabled_,
-                          kSystemMenuAccessibilityLargeCursorIcon);
-  }
-
   high_contrast_enabled_ = delegate->IsHighContrastEnabled();
   high_contrast_view_ = AddScrollListItem(
       bundle.GetLocalizedString(
           IDS_ASH_STATUS_TRAY_ACCESSIBILITY_HIGH_CONTRAST_MODE),
       high_contrast_enabled_, high_contrast_enabled_,
       kSystemMenuAccessibilityContrastIcon);
   screen_magnifier_enabled_ = delegate->IsMagnifierEnabled();
   screen_magnifier_view_ =
       AddScrollListItem(bundle.GetLocalizedString(
                             IDS_ASH_STATUS_TRAY_ACCESSIBILITY_SCREEN_MAGNIFIER),
                         screen_magnifier_enabled_, screen_magnifier_enabled_,
                         kSystemMenuAccessibilityScreenMagnifierIcon);
 
-  // Don't show autoclick option at login screen.
-  if (login_ != LoginStatus::NOT_LOGGED_IN) {
-    autoclick_enabled_ = delegate->IsAutoclickEnabled();
-    autoclick_view_ = AddScrollListItem(
-        bundle.GetLocalizedString(IDS_ASH_STATUS_TRAY_ACCESSIBILITY_AUTOCLICK),
-        autoclick_enabled_, autoclick_enabled_,
-        kSystemMenuAccessibilityAutoClickIcon);
-  }
+  autoclick_enabled_ = delegate->IsAutoclickEnabled();
+  autoclick_view_ = AddScrollListItem(
+      bundle.GetLocalizedString(IDS_ASH_STATUS_TRAY_ACCESSIBILITY_AUTOCLICK),
+      autoclick_enabled_, autoclick_enabled_,
+      kSystemMenuAccessibilityAutoClickIcon);
 
   virtual_keyboard_enabled_ = delegate->IsVirtualKeyboardEnabled();
   virtual_keyboard_view_ =
       AddScrollListItem(bundle.GetLocalizedString(
                             IDS_ASH_STATUS_TRAY_ACCESSIBILITY_VIRTUAL_KEYBOARD),
                         virtual_keyboard_enabled_, virtual_keyboard_enabled_,
                         kSystemMenuKeyboardIcon);
+
+  AddSubHeader(l10n_util::GetStringUTF16(
+      IDS_ASH_STATUS_TRAY_ACCESSIBILITY_ADDITIONAL_SETTINGS));
+
+  large_cursor_enabled_ = delegate->IsLargeCursorEnabled();
+  large_cursor_view_ = AddScrollListItemWithoutIcon(
+      bundle.GetLocalizedString(IDS_ASH_STATUS_TRAY_ACCESSIBILITY_LARGE_CURSOR),
+      large_cursor_enabled_);
+
+  mono_audio_enabled_ = delegate->IsMonoAudioEnabled();
+  mono_audio_view_ = AddScrollListItemWithoutIcon(
+      bundle.GetLocalizedString(IDS_ASH_STATUS_TRAY_ACCESSIBILITY_MONO_AUDIO),
+      mono_audio_enabled_);
+
+  caret_highlight_enabled_ = delegate->IsCaretHighlightEnabled();
+  caret_highlight_view_ = AddScrollListItemWithoutIcon(
+      bundle.GetLocalizedString(
+          IDS_ASH_STATUS_TRAY_ACCESSIBILITY_CARET_HIGHLIGHT),
+      caret_highlight_enabled_);
+
+  highlight_mouse_cursor_enabled_ = delegate->IsCursorHighlightEnabled();
+  highlight_mouse_cursor_view_ = AddScrollListItemWithoutIcon(
+      bundle.GetLocalizedString(
+          IDS_ASH_STATUS_TRAY_ACCESSIBILITY_HIGHLIGHT_MOUSE_CURSOR),
+      highlight_mouse_cursor_enabled_);
+
+  // Focus highlighting can't be on when spoken
+  // feedback is on because ChromeVox does its own focus highlighting.

[tdanderson, 2017/03/29 18:57:03]

Thanks for adding this comment, it's very helpful.

nit: Please have the first line wrap as close to 80 characters as possible.

+  if (!spoken_feedback_enabled_) {
+    highlight_keyboard_focus_enabled_ = delegate->IsFocusHighlightEnabled();
+    highlight_keyboard_focus_view_ = AddScrollListItemWithoutIcon(
+        bundle.GetLocalizedString(
+            IDS_ASH_STATUS_TRAY_ACCESSIBILITY_HIGHLIGHT_KEYBOARD_FOCUS),
+        highlight_keyboard_focus_enabled_);
+  }
 }
 
 HoverHighlightView* AccessibilityDetailedView::AddScrollListItem(
     const base::string16& text,
     bool highlight,
     bool checked,
     const gfx::VectorIcon& icon) {
   HoverHighlightView* container = new HoverHighlightView(this);
   gfx::ImageSkia image = CreateVectorIcon(icon, kMenuIconColor);
   const int padding = (kMenuButtonSize - image.width()) / 2;
   container->AddIconAndLabelCustomSize(
       image, text, highlight, image.width() + kMenuSeparatorVerticalPadding * 2,
       padding, padding);
 
-  if (checked) {
-    gfx::ImageSkia check_mark =
-        CreateVectorIcon(kCheckCircleIcon, gfx::kGoogleGreen700);
-    container->AddRightIcon(check_mark, check_mark.width());
-    container->SetRightViewVisible(true);
-    container->SetAccessiblityState(
-        HoverHighlightView::AccessibilityState::CHECKED_CHECKBOX);
-  } else {
-    container->SetAccessiblityState(
-        HoverHighlightView::AccessibilityState::UNCHECKED_CHECKBOX);
-  }
+  UpdateCheckMark(container, checked);
 
   scroll_content()->AddChildView(container);
   return container;
 }
 
+HoverHighlightView* AccessibilityDetailedView::AddScrollListItemWithoutIcon(
+    const base::string16& text,
+    bool checked) {
+  HoverHighlightView* container = new HoverHighlightView(this);
+  container->AddLabelRowMd(text);
+
+  UpdateCheckMark(container, checked);
+
+  scroll_content()->AddChildView(container);
+  return container;
+}
+
+void AccessibilityDetailedView::AddSubHeader(
+    const base::string16& header_text) {
+  TriView* header = TrayPopupUtils::CreateSubHeaderRowView();
+  TrayPopupUtils::ConfigureAsStickyHeader(header);
+
+  views::Label* label = TrayPopupUtils::CreateDefaultLabel();
+  label->SetText(header_text);
+  TrayPopupItemStyle style(TrayPopupItemStyle::FontStyle::SUB_HEADER);
+  style.SetupLabel(label);
+  header->AddView(TriView::Container::CENTER, label);
+
+  scroll_content()->AddChildView(new views::Separator());

[tdanderson, 2017/03/29 18:57:03]

Just instantiating a Separator() will not end up giving you the correct
separator color and padding values above/below. Have a look at
TrayPopupUtils::CreateListSubHeaderSeparator() used by some of the other
detailed views in the system menu; that utility function sets up the color and
padding for the separator in a way that will be consistent across all detailed
views.

nit: also, since the separator is not part of the sub-header, could you move
this line out of AddSubHeader() and place it above line 223?

+  scroll_content()->AddChildView(header);
+}
+
 void AccessibilityDetailedView::HandleViewClicked(views::View* view) {
   AccessibilityDelegate* delegate =
       Shell::GetInstance()->accessibility_delegate();
   UserMetricsAction user_action;
   if (view == spoken_feedback_view_) {
     user_action = delegate->IsSpokenFeedbackEnabled()
                       ? ash::UMA_STATUS_AREA_DISABLE_SPOKEN_FEEDBACK
                       : ash::UMA_STATUS_AREA_ENABLE_SPOKEN_FEEDBACK;
     delegate->ToggleSpokenFeedback(A11Y_NOTIFICATION_NONE);
   } else if (view == high_contrast_view_) {
@@ skipping 14 matching lines @@
   } else if (autoclick_view_ && view == autoclick_view_) {
     user_action = delegate->IsAutoclickEnabled()
                       ? ash::UMA_STATUS_AREA_DISABLE_AUTO_CLICK
                       : ash::UMA_STATUS_AREA_ENABLE_AUTO_CLICK;
     delegate->SetAutoclickEnabled(!delegate->IsAutoclickEnabled());
   } else if (virtual_keyboard_view_ && view == virtual_keyboard_view_) {
     user_action = delegate->IsVirtualKeyboardEnabled()
                       ? ash::UMA_STATUS_AREA_DISABLE_VIRTUAL_KEYBOARD
                       : ash::UMA_STATUS_AREA_ENABLE_VIRTUAL_KEYBOARD;
     delegate->SetVirtualKeyboardEnabled(!delegate->IsVirtualKeyboardEnabled());
+  } else if (caret_highlight_view_ && view == caret_highlight_view_) {
+    user_action = delegate->IsCaretHighlightEnabled()
+                      ? ash::UMA_STATUS_AREA_DISABLE_CARET_HIGHLIGHT
+                      : ash::UMA_STATUS_AREA_ENABLE_CARET_HIGHLIGHT;
+    delegate->SetCaretHighlightEnabled(!delegate->IsCaretHighlightEnabled());
+  } else if (mono_audio_view_ && view == mono_audio_view_) {
+    user_action = delegate->IsMonoAudioEnabled()
+                      ? ash::UMA_STATUS_AREA_DISABLE_MONO_AUDIO
+                      : ash::UMA_STATUS_AREA_ENABLE_MONO_AUDIO;
+    delegate->SetMonoAudioEnabled(!delegate->IsMonoAudioEnabled());
+  } else if (highlight_mouse_cursor_view_ &&
+             view == highlight_mouse_cursor_view_) {
+    user_action = delegate->IsCursorHighlightEnabled()
+                      ? ash::UMA_STATUS_AREA_DISABLE_HIGHLIGHT_MOUSE_CURSOR
+                      : ash::UMA_STATUS_AREA_ENABLE_HIGHLIGHT_MOUSE_CURSOR;
+    delegate->SetCursorHighlightEnabled(!delegate->IsCursorHighlightEnabled());
+  } else if (highlight_keyboard_focus_view_ &&
+             view == highlight_keyboard_focus_view_) {
+    user_action = delegate->IsFocusHighlightEnabled()
+                      ? ash::UMA_STATUS_AREA_DISABLE_HIGHLIGHT_KEYBOARD_FOCUS
+                      : ash::UMA_STATUS_AREA_ENABLE_HIGHLIGHT_KEYBOARD_FOCUS;
+    delegate->SetFocusHighlightEnabled(!delegate->IsFocusHighlightEnabled());
   } else {
     return;
   }
   WmShell::Get()->RecordUserMetricsAction(user_action);
 }
 
 void AccessibilityDetailedView::HandleButtonPressed(views::Button* sender,
                                                     const ui::Event& event) {
   if (sender == help_view_)
     ShowHelp();
@@ skipping 151 matching lines @@
     if (detailed_popup_)
       detailed_popup_->GetWidget()->Close();
     if (detailed_menu_)
       detailed_menu_->GetWidget()->Close();
   }
 
   previous_accessibility_state_ = accessibility_state;
 }
 
 }  // namespace ash