Switch AccessibilityMode to be a bitmap

content/common/view_message_enums.h

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..ebef6a4f3885e35cd2e611256627c26e894f1b0e 100644
--- a/content/common/view_message_enums.h
+++ b/content/common/view_message_enums.h
@@ -53,19 +53,39 @@ 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,
+
+  // Accessibility updates are processed to create platform trees and events are
+  // passed to platform APIs in the browser.
+  AccessibilityModeFlagPlatform = 1 << 1,
 
-  // WebKit accessibility is on, but only limited information about
+  // 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 AccessibilityMode {
+  // All accessibility is off.
+  AccessibilityModeOff = 0,
+
+  // Renderer accessibility is on, and platform APIs are called. Note that this
+  // is different to AccessibilityModeAll, which is defined to be all bits on.
+  AccessibilityModeComplete =
+      AccessibilityModeFlagRenderer | AccessibilityModeFlagPlatform,
+
+  // Renderer accessibility is on, platform APIs are called, but only limited
+  // information is available (see AccessibilityModeFlagEditableTextOnly).
+  AccessibilityModeEditableTextOnly = AccessibilityModeFlagEditableTextOnly,

[David Tseng, 2014/02/04 20:25:51]

I think you're expecting to also have the renderer bit set.

[aboxhall, 2014/02/05 16:02:47]

I still can't make up my mind on this one. 

 
-  // WebKit accessibility is on, and the full accessibility tree is synced
-  // to the browser process. Useful for screen readers and magnifiers.
-  AccessibilityModeComplete,
+  // All bits on.
+  AccessibilityModeAll = AccessibilityModeFlagRenderer |
+                         AccessibilityModeFlagPlatform |
+                         AccessibilityModeFlagEditableTextOnly
 };
 
 #endif  // CONTENT_COMMON_VIEW_MESSAGES_ENUMS_H_