Re-land: Implement NativeViewAccessibilityWin using AXPlatformNodeWin.

ui/accessibility/platform/ax_platform_node.h

Index: ui/accessibility/platform/ax_platform_node.h
diff --git a/ui/accessibility/platform/ax_platform_node.h b/ui/accessibility/platform/ax_platform_node.h
index 10ecba2064d0d76b9243d7a590ea98dbc969411b..b450121a52a3a55328a81a96585ff8b2ea5a5227 100644
--- a/ui/accessibility/platform/ax_platform_node.h
+++ b/ui/accessibility/platform/ax_platform_node.h
@@ -5,6 +5,7 @@
 #ifndef UI_ACCESSIBILITY_PLATFORM_AX_PLATFORM_NODE_H_
 #define UI_ACCESSIBILITY_PLATFORM_AX_PLATFORM_NODE_H_
 
+#include "ui/accessibility/ax_enums.h"
 #include "ui/accessibility/ax_export.h"
 #include "ui/gfx/native_widget_types.h"
 
@@ -12,17 +13,35 @@ namespace ui {
 
 class AXPlatformNodeDelegate;
 
+// AXPlatformNode is the abstract base class for an implementation of
+// native accessibility APIs on supported platforms (e.g. Windows, Mac OS X).
+// An object that wants to be accessible can derive from AXPlatformNodeDelegate
+// and then call AXPlatformNode::Create. The delegate implementation should
+// own the AXPlatformNode instance (or otherwise manage its lifecycle).
 class AX_EXPORT AXPlatformNode {
  public:
-  // Create a platform appropriate instance.
+  // Create an appropriate platform-specific instance. The delegate owns the
+  // AXPlatformNode instance (or manages its lifecycle in some other way).
   static AXPlatformNode* Create(AXPlatformNodeDelegate* delegate);
 
+  // Cast a gfx::NativeViewAccessible to an AXPlatformNode if it is one,

[David Tseng, 2015/02/13 20:10:10]

Is the relationship a cast or a lookup by the delegate?

[dmazzoni, 2015/02/17 22:48:04]

It's a cast, like the comment says. The actual implementation of the check will
vary by platform - on Windows we use QueryInterface to determine if it's an
instance of this particular class, on Mac we may use a custom selector, etc.

+  // or return NULL if it's not an instance of this class.
+  static AXPlatformNode* FromNativeViewAccessible(
+      gfx::NativeViewAccessible accessible);
+
   // Call Destroy rather than deleting this, because the subclass may
   // use reference counting.
   virtual void Destroy() = 0;
 
+  // Get the platform-specific accessible object type for this instance.
+  // On some platforms this is just a type cast, on others it may be a
+  // wrapper object or handle.
   virtual gfx::NativeViewAccessible GetNativeViewAccessible() = 0;
 
+  // Fire a platform-specific notification that an event has occurred on
+  // this object.
+  virtual void NotifyAccessibilityEvent(ui::AXEvent event_type) = 0;

[David Tseng, 2015/02/13 20:10:10]

Any thoughts to how actions are handled? (e.g. setting value)?

[dmazzoni, 2015/02/17 22:48:04]

Yes, each subclass of AXPlatformNode implelments platform-specific actions and
calls the appropriate AXPlatformNodeDelegate function to handle the action. This
changelist implements support for setting the value.

+
  protected:
   AXPlatformNode();
   virtual ~AXPlatformNode();