|
|
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_ |