Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 2581483002: Add support for faster, more limited accessibility modes. (Closed)
Patch Set: Android compile fix Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 11ae958f7a0f39ca8f319a4afb6316f184005c50..3e0fa1692f18b2160f03a801f618b8abbabc43fe 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -265,13 +265,23 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Accessibility.ModeFlag" enum="AccessibilityModeFlagEnum">
+ <owner>dmazzoni@chromium.org</owner>
+ <owner>aboxhall@chromium.org</owner>
+ <summary>
+ Whether individual accessibility mode flags are set. Tracked when each mode
+ flag is flipped from false to true.
Ilya Sherman 2016/12/17 00:57:12 Hmm, this seems like it might be a bit confusing t
dmazzoni 2016/12/19 22:51:51 I clarified in the comment that the flags are flip
+ </summary>
+</histogram>
+
<histogram name="Accessibility.State" enum="BooleanEnabled">
<owner>dmazzoni@chromium.org</owner>
<owner>kenjibaheux@google.com</owner>
<summary>
Whether Chrome has enabled accessibility support because it detects
supported assistive technology running, or due to being manually enabled via
- a command-line flag (checked once 45 secs after startup).
+ a command-line flag (checked once 45 secs after startup). NOTE:
+ Accessibility.ModeFlag is potentially more useful.
Ilya Sherman 2016/12/17 00:57:12 Is the new metrics more useful enough to where we
dmazzoni 2016/12/19 22:51:51 Sure, done. The main reason to keep it a bit longe
</summary>
</histogram>
@@ -75228,6 +75238,17 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<int value="2" label="ScrolledMainFrameWithUnacceleratedFixedRootBackground"/>
</enum>
+<enum name="AccessibilityModeFlagEnum" type="int">
+ <summary>Track individual accessibility mdoe flags that are enabled.</summary>
Ilya Sherman 2016/12/17 00:57:12 nit: s/mdoe/mode
dmazzoni 2016/12/19 22:51:51 Done.
+ <int value="0" label="UMA_AX_MODE_FLAG_NATIVE_APIS">Native APIs</int>
Ilya Sherman 2016/12/17 00:57:12 nit: The "Native APIs" text is only shown on hover
dmazzoni 2016/12/19 22:51:51 Sounds good, thanks.
+ <int value="1" label="UMA_AX_MODE_FLAG_WEB_CONTENTS">Web Contents</int>
+ <int value="2" label="UMA_AX_MODE_FLAG_INLINE_TEXT_BOXES">
+ Inline Text Boxes
+ </int>
+ <int value="3" label="UMA_AX_MODE_FLAG_SCREEN_READER">Screen Reader</int>
+ <int value="4" label="UMA_AX_MODE_FLAG_HTML">HTML</int>
+</enum>
+
<enum name="AccessibilityWinAPIEnum" type="int">
<summary>
Track which Windows accessibility APIs are being called by clients.

Powered by Google App Engine
This is Rietveld 408576698