List all a11y featuers in ash system menu

ash/common/system/tray_accessibility.h

 // 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.
 
 #ifndef ASH_COMMON_SYSTEM_TRAY_ACCESSIBILITY_H_
 #define ASH_COMMON_SYSTEM_TRAY_ACCESSIBILITY_H_
 
 #include <stdint.h>
 
 #include "ash/common/accessibility_delegate.h"
@@ skipping 62 matching lines @@
   void ShowHelp();
 
   // Add the accessibility feature list.
   void AppendAccessibilityList();
 
   // Helper function to create entries in the detailed accessibility view.
   HoverHighlightView* AddScrollListItem(const base::string16& text,
                                         bool highlight,
                                         bool checked,
                                         const gfx::VectorIcon& icon);
+  HoverHighlightView* AddScrollListItemWithoutIcon(const base::string16& text,
+                                                   bool checked);
+
+  void CheckOrUncheckMark(HoverHighlightView** container, bool checked);

[xiyuan, 2017/03/28 20:11:25]

This function seems not touching any state in AccessibilityDetailedView. Can we
move it into cc file in the anonymous namespace.

[xiyuan, 2017/03/28 20:11:25]

Can we pass HoverHighlightView* instead of **?

[xiyuan, 2017/03/28 20:11:25]

nit: I'd call this UpdateCheckMark.

[minch1, 2017/03/29 01:15:11]

Done.

[minch1, 2017/03/29 01:15:11]

Done.

[minch1, 2017/03/29 01:15:11]

Done.

+
+  void AddSubHeader(int message_id);

[xiyuan, 2017/03/28 20:11:25]

nit: Can we pass in the string instead?
i.e.
   void AddSubHeader(const base::string16& header);

[minch1, 2017/03/29 01:15:11]

Hi,xiyuan. I guess here is to make the function readable? I changed the function
as following, 
 void AddSubHeader(const base::string16& header_text)

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

SG.

 
   views::View* spoken_feedback_view_;
   views::View* high_contrast_view_;
   views::View* screen_magnifier_view_;
   views::View* large_cursor_view_;
   views::CustomButton* help_view_;
   views::CustomButton* settings_view_;
   views::View* autoclick_view_;
   views::View* virtual_keyboard_view_;
+  views::View* mono_audio_view_;
+  views::View* caret_highlight_view_;
+  views::View* highlight_mouse_cursor_view_;
+  views::View* highlight_keyboard_focus_view_;
 
   bool spoken_feedback_enabled_;
   bool high_contrast_enabled_;
   bool screen_magnifier_enabled_;
   bool large_cursor_enabled_;
   bool autoclick_enabled_;
   bool virtual_keyboard_enabled_;
+  bool mono_audio_enabled_;
+  bool caret_highlight_enabled_;
+  bool highlight_mouse_cursor_enabled_;
+  bool highlight_keyboard_focus_enabled_;
   LoginStatus login_;
 
   friend class chromeos::TrayAccessibilityTest;
   DISALLOW_COPY_AND_ASSIGN(AccessibilityDetailedView);
 };
 
 }  // namespace tray
 
 class TrayAccessibility : public TrayImageItem, public AccessibilityObserver {
  public:
@@ skipping 33 matching lines @@
   // A11y feature status on just entering the lock screen.
   bool show_a11y_menu_on_lock_screen_;
 
   friend class chromeos::TrayAccessibilityTest;
   DISALLOW_COPY_AND_ASSIGN(TrayAccessibility);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_SYSTEM_TRAY_ACCESSIBILITY_H_