Support accessible inline text boxes.

content/browser/accessibility/accessibility_tree_formatter.cc

Index: content/browser/accessibility/accessibility_tree_formatter.cc
diff --git a/content/browser/accessibility/accessibility_tree_formatter.cc b/content/browser/accessibility/accessibility_tree_formatter.cc
index 4368cf132179b383932f018e4f667f2153fece95..cde18862c4a8fdf75415cf31caa3ec23b837ed12 100644
--- a/content/browser/accessibility/accessibility_tree_formatter.cc
+++ b/content/browser/accessibility/accessibility_tree_formatter.cc
@@ -66,7 +66,7 @@ void AccessibilityTreeFormatter::RecursiveBuildAccessibilityTree(
 
   base::ListValue* children = new base::ListValue;
   dict->Set(kChildrenDictAttr, children);
-  if (!IncludeChildren(node))
+  if (node.IsLeaf())

[David Tseng, 2013/10/17 22:56:29]

Btw, isLeaf worries me because it's often possible to get focus set on a
descendant of a leaf (depending on how we define leaves). We should be careful
when stating what is a leaf and what isn't.

[dmazzoni, 2013/10/21 17:26:57]

My intent is that this is only used to really mean a leaf - we literally don't
expose any children of this node to the platform. On Android, we sometimes prune
children of a focusable node that has a title, but we're careful to only do this
when it doesn't itself have any focusable children - so there isn't any worry
about this.

Would some documentation help, or can you think of a better name?

     return;
 
   for (size_t i = 0; i < node.children().size(); ++i) {
@@ -93,13 +93,6 @@ void AccessibilityTreeFormatter::RecursiveFormatAccessibilityTree(
   }
 }
 
-#if !defined(OS_ANDROID)
-bool AccessibilityTreeFormatter::IncludeChildren(
-    const BrowserAccessibility& node) {
-  return true;
-}
-#endif
-
 #if (!defined(OS_WIN) && !defined(OS_MACOSX) && !defined(OS_ANDROID) && \
      !defined(TOOLKIT_GTK))
 void AccessibilityTreeFormatter::AddProperties(const BrowserAccessibility& node,