Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(881)

Unified Diff: content/browser/web_contents/web_contents_android.cc

Issue 1137393003: Add style information to the snapshot node (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add content dependency to ui_accessibility_java for GN Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/gyp/java_cpp_enum.py ('k') | content/public/android/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 ce9bdf50eb2fdbf6a7defe844a7531061e44a2ab..663e0804e01b598bfc8fff8c55d098bf3a28c77d 100644
--- a/content/browser/web_contents/web_contents_android.cc
+++ b/content/browser/web_contents/web_contents_android.cc
@@ -60,11 +60,25 @@ ScopedJavaLocalRef<jobject> WalkAXTreeDepthFirst(JNIEnv* env,
ScopedJavaLocalRef<jstring> j_class =
ConvertUTF8ToJavaString(env, node->GetClassName());
const gfx::Rect& location = node->GetLocation();
+ // The style attributes exists and valid if size attribute exists. Otherwise,
+ // they are not. Use a negative size information to indicate the existence
+ // of style information.
+ float size = -1.0;
+ int color = 0;
+ int bgcolor = 0;
+ int text_style = 0;
+ if (node->HasFloatAttribute(ui::AX_ATTR_FONT_SIZE)) {
+ color = node->GetIntAttribute(ui::AX_ATTR_COLOR);
+ bgcolor = node->GetIntAttribute(ui::AX_ATTR_BACKGROUND_COLOR);
+ size = node->GetFloatAttribute(ui::AX_ATTR_FONT_SIZE);
+ 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 =
« no previous file with comments | « build/android/gyp/java_cpp_enum.py ('k') | content/public/android/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698