Add style information to the snapshot node

content/browser/web_contents/web_contents_android.cc

Index: content/browser/web_contents/web_contents_android.cc
diff --git a/content/browser/web_contents/web_contents_android.cc b/content/browser/web_contents/web_contents_android.cc
index dc23623cf475b2867bf7d055483f239f88f0bd7c..46ade0b21ea8043dc05292d5ee1f442aed9a861a 100644
--- a/content/browser/web_contents/web_contents_android.cc
+++ b/content/browser/web_contents/web_contents_android.cc
@@ -60,11 +60,16 @@ ScopedJavaLocalRef<jobject> WalkAXTreeDepthFirst(JNIEnv* env,
   ScopedJavaLocalRef<jstring> j_class =
       ConvertUTF8ToJavaString(env, node->GetClassName());
   const gfx::Rect& location = node->GetLocation();
+  int color = node->GetIntAttribute(ui::AX_ATTR_COLOR);

[Ted C, 2015/05/27 18:45:39]

GetIntAttribute returns 0 when this isn't defined.  For color, that will default
to transparent on android, which probably isn't what you want.

Should this use the function that returns a boolean and only use the color if
defined otherwise default to 0xFF000000 ?

Same for bgcolor and size, should we default to some decent constant if
undefined?

[sgurun-gerrit only, 2015/05/27 19:27:24]

good point, I actually missed it. It is more attractive to pass a decent
constant (for not making JNI interface even more crowded) but then I don't know
what a decent constant would be.

[sgurun-gerrit only, 2015/05/27 19:42:26]

ok, talked to Dominic about it. We will check if size exists or not. if size
exists, we will read all the attributes and pass to Java, otherwise, we will not
pass anything.

[sgurun-gerrit only, 2015/05/29 00:35:24]

Done.

+  int bgcolor = 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);
+
   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, bgcolor, size, text_style, j_class.obj());
 
   for(uint32 i = 0; i < node->PlatformChildCount(); i++) {
     BrowserAccessibilityAndroid* child =