Add style information to the snapshot node

content/browser/web_contents/web_contents_android.cc

 // Copyright 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/browser/web_contents/web_contents_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
@@ skipping 42 matching lines @@
 }
 
 ScopedJavaLocalRef<jobject> WalkAXTreeDepthFirst(JNIEnv* env,
       BrowserAccessibilityAndroid* node) {
 
   ScopedJavaLocalRef<jstring> j_text =
       ConvertUTF16ToJavaString(env, node->GetText());
   ScopedJavaLocalRef<jstring> j_class =
       ConvertUTF8ToJavaString(env, node->GetClassName());
   const gfx::Rect& location = node->GetLocation();
+  int color = node->GetIntAttribute(ui::AX_ATTR_COLOR_VALUE);

[dmazzoni, 2015/05/19 05:32:19]

This is the wrong attribute for text color - COLOR_VALUE is the value of <input
type=color>, you just want AX_ATTR_COLOR.

+  int bkgcolor = node->GetIntAttribute(ui::AX_ATTR_BACKGROUND_COLOR);
+  float size =  node->GetFloatAttribute(ui::AX_ATTR_FONT_SIZE);
+  int text_style = node->GetIntAttribute(ui::AX_ATTR_TEXT_STYLE);

[sgurun-gerrit only, 2015/05/18 23:51:09]

is this the right way of getting color, bkgcolor, size, etc? I wrote a unit test
and don't get the right data (always 0 for color for example)

+
   ScopedJavaLocalRef<jobject> j_node =
       Java_WebContentsImpl_createAccessibilitySnapshotNode(env,
           location.x(), location.y(), node->GetScrollX(),
           node->GetScrollY(), location.width(), location.height(),
-          j_text.obj(), j_class.obj());
+          j_text.obj(), color, bkgcolor, size, text_style, j_class.obj());
 
   for(uint32 i = 0; i < node->PlatformChildCount(); i++) {
     BrowserAccessibilityAndroid* child =
         static_cast<BrowserAccessibilityAndroid*>(
             node->PlatformGetChild(i));
     Java_WebContentsImpl_addAccessibilityNodeAsChild(env,
         j_node.obj(), WalkAXTreeDepthFirst(env, child).obj());
   }
   return j_node;
 }
@@ skipping 488 matching lines @@
   ScopedJavaGlobalRef<jobject> j_callback;
   j_callback.Reset(env, callback);
   WebContentsImpl::AXTreeSnapshotCallback snapshot_callback =
       base::Bind(&AXTreeSnapshotCallback, j_callback);
 
   static_cast<WebContentsImpl*>(web_contents_)->RequestAXTreeSnapshot(
       snapshot_callback);
 }
 
 }  // namespace content