Adds color, font size, text direction and text styles to the accessibility tree.

content/renderer/accessibility/blink_ax_enum_conversion.cc

 // Copyright (c) 2013 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.
 
 #include "content/renderer/accessibility/blink_ax_enum_conversion.h"
 
 #include "base/logging.h"
 
 namespace content {
 
@@ skipping 374 matching lines @@
     default:
       // We can't add an assertion here, that prevents us
       // from adding new event enums in Blink.
       return ui::AX_EVENT_NONE;
   }
 }
 
 ui::AXTextDirection AXTextDirectionFromBlink(
     blink::WebAXTextDirection text_direction) {
   switch (text_direction) {
+    case blink::WebAXTextDirectionUnknown:
+      return ui::AX_TEXT_DIRECTION_NONE;
     case blink::WebAXTextDirectionLR:
-      return ui::AX_TEXT_DIRECTION_LR;
+      return ui::AX_TEXT_DIRECTION_LTR;
     case blink::WebAXTextDirectionRL:
-      return ui::AX_TEXT_DIRECTION_RL;
+      return ui::AX_TEXT_DIRECTION_RTL;
     case blink::WebAXTextDirectionTB:
-      return ui::AX_TEXT_DIRECTION_TB;
+      return ui::AX_TEXT_DIRECTION_TTB;
     case blink::WebAXTextDirectionBT:
-      return ui::AX_TEXT_DIRECTION_BT;
+      return ui::AX_TEXT_DIRECTION_BTT;
     default:
       NOTREACHED();
   }
 
   return ui::AX_TEXT_DIRECTION_NONE;
 }
 
+ui::AXTextStyle AXTextStyleFromBlink(blink::WebAXTextStyle text_style) {
+  switch (text_style) {
+    case blink::WebAXTextStyleNone:

[sgurun-gerrit only, 2015/05/12 20:11:58]

no member named 'WebAXTextDirectionUnknown' in namespace 'blink'

+      return ui::AX_TEXT_STYLE_NONE;
+    case blink::WebAXTextStyleBold:
+      return ui::AX_TEXT_STYLE_BOLD;
+    case blink::WebAXTextStyleItalic:
+      return ui::AX_TEXT_STYLE_ITALIC;
+    case blink::WebAXTextStyleUnderline:
+      return ui::AX_TEXT_STYLE_UNDERLINE;

[dmazzoni, 2015/05/12 18:34:48]

I just realized, these shouldn't be mutually exclusive, they should be additive.
It's possible to be bold, italic, underline, and linethrough simultaneously.

Can you patch this in Blink?

+    case blink::WebAXTextStyleLineThrough:
+      return ui::AX_TEXT_STYLE_LINE_THROUGH;
+    default:
+      NOTREACHED();
+  }
+
+  return ui::AX_TEXT_STYLE_NONE;
+}
+
 ui::AXInvalidState AXInvalidStateFromBlink(
     blink::WebAXInvalidState invalid_state) {
   switch (invalid_state) {
     case blink::WebAXInvalidStateUndefined:
       return ui::AX_INVALID_STATE_NONE;
     case blink::WebAXInvalidStateFalse:
       return ui::AX_INVALID_STATE_FALSE;
     case blink::WebAXInvalidStateTrue:
       return ui::AX_INVALID_STATE_TRUE;
     case blink::WebAXInvalidStateSpelling:
@@ skipping 23 matching lines @@
     case blink::WebAXSortDirectionOther:
       return ui::AX_SORT_DIRECTION_OTHER;
     default:
       NOTREACHED();
   }
 
   return ui::AX_SORT_DIRECTION_NONE;
 }
 
 }  // Namespace content.