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

Unified Diff: content/shell/renderer/test_runner/web_ax_object_proxy.cc

Issue 665843002: Change condition for adding AX_STATE_EXPANDED (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove AXExpanded Created 6 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/shell/renderer/test_runner/web_ax_object_proxy.cc
diff --git a/content/shell/renderer/test_runner/web_ax_object_proxy.cc b/content/shell/renderer/test_runner/web_ax_object_proxy.cc
index e52343cd087a975126e1dda06765d08a9d4294b3..ca7264fd90dce9ffd3081572a59c15cfb59c11f2 100644
--- a/content/shell/renderer/test_runner/web_ax_object_proxy.cc
+++ b/content/shell/renderer/test_runner/web_ax_object_proxy.cc
@@ -756,7 +756,7 @@ bool WebAXObjectProxy::IsSelectedOptionActive() {
bool WebAXObjectProxy::IsExpanded() {
accessibility_object_.updateLayoutAndCheckValidity();
- return !accessibility_object_.isCollapsed();
+ return accessibility_object_.isExpanded() == blink::WebAXExpandedExpanded;
}
bool WebAXObjectProxy::IsChecked() {
@@ -764,6 +764,11 @@ bool WebAXObjectProxy::IsChecked() {
return accessibility_object_.isChecked();
}
+bool WebAXObjectProxy::IsCollapsed() {
+ accessibility_object_.updateLayoutAndCheckValidity();
+ return accessibility_object_.isExpanded() == blink::WebAXExpandedCollapsed;
+}
+
bool WebAXObjectProxy::IsVisible() {
accessibility_object_.updateLayoutAndCheckValidity();
return accessibility_object_.isVisible();
@@ -774,11 +779,6 @@ bool WebAXObjectProxy::IsOffScreen() {
return accessibility_object_.isOffScreen();
}
-bool WebAXObjectProxy::IsCollapsed() {
- accessibility_object_.updateLayoutAndCheckValidity();
- return accessibility_object_.isCollapsed();
-}
-
bool WebAXObjectProxy::HasPopup() {
accessibility_object_.updateLayoutAndCheckValidity();
return accessibility_object_.ariaHasPopup();
« no previous file with comments | « content/renderer/accessibility/blink_ax_tree_source.cc ('k') | content/test/data/accessibility/aria-expanded.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698