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

Unified Diff: content/browser/accessibility/browser_accessibility_manager_android.cc

Issue 25943003: Support accessible inline text boxes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typo Created 7 years, 2 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
Index: content/browser/accessibility/browser_accessibility_manager_android.cc
diff --git a/content/browser/accessibility/browser_accessibility_manager_android.cc b/content/browser/accessibility/browser_accessibility_manager_android.cc
index 2943c5cf4be6d6805555cfc494258e30038422e4..3d19d07d6f766f496ad4e5fbbba6d2449c965af0 100644
--- a/content/browser/accessibility/browser_accessibility_manager_android.cc
+++ b/content/browser/accessibility/browser_accessibility_manager_android.cc
@@ -182,11 +182,9 @@ jboolean BrowserAccessibilityManagerAndroid::PopulateAccessibilityNodeInfo(
Java_BrowserAccessibilityManager_setAccessibilityNodeInfoParent(
env, obj, info, node->parent()->renderer_id());
}
- if (!node->IsLeaf()) {
- for (unsigned i = 0; i < node->child_count(); ++i) {
- Java_BrowserAccessibilityManager_addAccessibilityNodeInfoChild(
- env, obj, info, node->children()[i]->renderer_id());
- }
+ for (unsigned i = 0; i < node->PlatformChildCount(); ++i) {
+ Java_BrowserAccessibilityManager_addAccessibilityNodeInfoChild(
+ env, obj, info, node->children()[i]->renderer_id());
}
Java_BrowserAccessibilityManager_setAccessibilityNodeInfoBooleanAttributes(
env, obj, info,
@@ -328,11 +326,9 @@ void BrowserAccessibilityManagerAndroid::FuzzyHitTestImpl(
return;
}
- if (!node->IsLeaf()) {
- for (uint32 i = 0; i < node->child_count(); i++) {
- BrowserAccessibility* child = node->GetChild(i);
- FuzzyHitTestImpl(x, y, child, nearest_candidate, nearest_distance);
- }
+ for (uint32 i = 0; i < node->PlatformChildCount(); i++) {
+ BrowserAccessibility* child = node->PlatformGetChild(i);
+ FuzzyHitTestImpl(x, y, child, nearest_candidate, nearest_distance);
}
}

Powered by Google App Engine
This is Rietveld 408576698