|
|
Chromium Code Reviews| Index: content/common/view_message_enums.h |
| diff --git a/content/common/view_message_enums.h b/content/common/view_message_enums.h |
| index 59a6da11b0aed6f04f35803b5ecde6422763356d..a54d7e9caaf834f9280f512627496beb274a3207 100644 |
| --- a/content/common/view_message_enums.h |
| +++ b/content/common/view_message_enums.h |
| @@ -53,19 +53,34 @@ struct ViewMsg_Navigate_Type { |
| }; |
| }; |
| -enum AccessibilityMode { |
| - // WebKit accessibility is off and no accessibility information is |
| - // sent from the renderer to the browser process. |
| - AccessibilityModeOff, |
| +enum AccessibilityModeFlag { |
| + // Blink accessibility is on and accessibility information is sent from the |
| + // renderer to the browser process. Note that none of the other flags do |
| + // anything if this flag is off. |
| + AccessibilityModeFlagRenderer = 1 << 0, |
| - // WebKit accessibility is on, but only limited information about |
| + // Accessibility updates are processed to create platform trees and events are |
| + // passed to platform APIs in the browser. |
| + AccessibilityModeFlagPlatform = 1 << 1, |
| + |
| + // Blink accessibility is on, but only limited information about |
| // editable text nodes is sent to the browser process. Useful for |
| // implementing limited UIA on tablets. |
| - AccessibilityModeEditableTextOnly, |
| + AccessibilityModeFlagEditableTextOnly = 1 << 2, |
| +}; |
| + |
| +enum AccessibilityModeConstants { |
| + // All accessibility is off. |
| + AccessibilityModeOff = 0, |
| + |
| + // Renderer accessibility is on, and platform APIs are called. |
| + AccessibilityModeComplete = |
| + AccessibilityModeFlagRenderer | AccessibilityModeFlagPlatform, |
| - // WebKit accessibility is on, and the full accessibility tree is synced |
| - // to the browser process. Useful for screen readers and magnifiers. |
| - AccessibilityModeComplete, |
| + // Renderer accessibility is on, platform APIs are called, but only limited |
| + // information is available (see AccessibilityModeFlagEditableTextOnly). |
| + AccessibilityModeEditableTextOnly = |
| + AccessibilityModeComplete | AccessibilityModeFlagEditableTextOnly |
|
David Tseng
2014/01/31 18:38:23
This is a little confusing because text only isn't
This is a little confusing because text only isn't really a superset of complete
(rather a subset).
In the code, they're mutually exclusive; you can only be in text only mode or
complete (not both).
I also think we shouldn't mix constants and flags when defining constants
aboxhall
2014/02/03 23:58:38
Done.
On 2014/01/31 18:38:23, David Tseng wrote:
> This is a little confusing because text only isn't really a superset of
complete
> (rather a subset).
>
> In the code, they're mutually exclusive; you can only be in text only mode or
> complete (not both).
>
> I also think we shouldn't mix constants and flags when defining constants
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
Done.
|
| }; |
| #endif // CONTENT_COMMON_VIEW_MESSAGES_ENUMS_H_ |
