Use Blink accessibility enums in Chromium

content/browser/accessibility/browser_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 CONTENT_BROWSER_ACCESSIBILITY_BROWSER_ACCESSIBILITY_H_
 #define CONTENT_BROWSER_ACCESSIBILITY_BROWSER_ACCESSIBILITY_H_
 
 #include <map>
 #include <utility>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/strings/string16.h"
 #include "build/build_config.h"
 #include "content/common/accessibility_node_data.h"

[aboxhall, 2013/08/29 00:59:06]

Should this point to the webkit enums instead?

[dmazzoni, 2013/08/29 04:47:15]

In this case it needs both.

[aboxhall, 2013/08/29 15:34:09]

Where are the WebKit enums coming from in this file?

[dmazzoni, 2013/08/29 17:44:59]

Well, accessibility_node_data.h has to include WebAXEnums.h already, so from
there - but you're right, this file should include WebAXEnums.h explicitly
(include what you use)

 #include "content/common/content_export.h"
 
 #if defined(OS_MACOSX) && __OBJC__
 @class BrowserAccessibilityCocoa;
 #endif
 
 namespace content {
 class BrowserAccessibilityManager;
 #if defined(OS_WIN)
 class BrowserAccessibilityWin;
@@ skipping 198 matching lines @@
   // aria-selected  (selectable)
   // aria-grabbed   (grabbable)
   // aria-expanded  (expandable)
   // aria-pressed   (toggleable/pressable) -- supports 4th "mixed" state
   // aria-checked   (checkable) -- supports 4th "mixed state"
   bool GetAriaTristate(const char* attr_name,
                        bool* is_defined,
                        bool* is_mixed) const;
 
   // Returns true if the bit corresponding to the given state enum is 1.
-  bool HasState(AccessibilityNodeData::State state_enum) const;
+  bool HasState(WebKit::WebAXState state_enum) const;
 
   // Returns true if this node is an editable text field of any kind.
   bool IsEditableText() const;
 
   // Append the text from this node and its children.
   std::string GetTextRecursive() const;
 
  protected:
   // Perform platform specific initialization. This can be called multiple times
   // during the lifetime of this instance after the members of this base object
@@ skipping 45 matching lines @@
   // immediately return failure.
   bool instance_active_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BrowserAccessibility);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_ACCESSIBILITY_BROWSER_ACCESSIBILITY_H_