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

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

Issue 925923003: Update BrowserAccessibilityManager to support Lollipop SDK. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@hover_debug
Patch Set: Created 5 years, 10 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 | « no previous file | content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 0950c6769b517d5dc54bb24fa5b31b5480cf8917..0772b1e71ff79260e77ec4b018ec029630af59a8 100644
--- a/content/browser/accessibility/browser_accessibility_manager_android.cc
+++ b/content/browser/accessibility/browser_accessibility_manager_android.cc
@@ -253,12 +253,9 @@ jboolean BrowserAccessibilityManagerAndroid::PopulateAccessibilityNodeInfo(
Java_BrowserAccessibilityManager_setAccessibilityNodeInfoBooleanAttributes(
env, obj, info,
id,
- node->CanScrollForward(),
- node->CanScrollBackward(),
node->IsCheckable(),
node->IsChecked(),
node->IsClickable(),
- node->IsEditableText(),
node->IsEnabled(),
node->IsFocusable(),
node->IsFocused(),
@@ -266,6 +263,16 @@ jboolean BrowserAccessibilityManagerAndroid::PopulateAccessibilityNodeInfo(
node->IsScrollable(),
node->IsSelected(),
node->IsVisibleToUser());
+ Java_BrowserAccessibilityManager_addAccessibilityNodeInfoActions(
+ env, obj, info,
+ id,
+ node->CanScrollForward(),
+ node->CanScrollBackward(),
+ node->IsClickable(),
+ node->IsEditableText(),
+ node->IsEnabled(),
+ node->IsFocusable(),
+ node->IsFocused());
Java_BrowserAccessibilityManager_setAccessibilityNodeInfoClassName(
env, obj, info,
base::android::ConvertUTF8ToJavaString(env, node->GetClassName()).obj());
@@ -292,8 +299,7 @@ jboolean BrowserAccessibilityManagerAndroid::PopulateAccessibilityNodeInfo(
absolute_rect.width(), absolute_rect.height(),
is_root);
- // New KitKat APIs
- Java_BrowserAccessibilityManager_setAccessibilityNodeInfoKitKatAttributes(
+ Java_BrowserAccessibilityManager_setAccessibilityNodeInfoLollipopAttributes(
env, obj, info,
node->CanOpenPopup(),
node->IsContentInvalid(),
@@ -392,8 +398,7 @@ jboolean BrowserAccessibilityManagerAndroid::PopulateAccessibilityEvent(
break;
}
- // Backwards-compatible fallback for new KitKat APIs.
- Java_BrowserAccessibilityManager_setAccessibilityEventKitKatAttributes(
+ Java_BrowserAccessibilityManager_setAccessibilityEventLollipopAttributes(
env, obj, event,
node->CanOpenPopup(),
node->IsContentInvalid(),
« no previous file with comments | « no previous file | content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698