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

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

Issue 2969603002: Move two BrowserAccessibility role predicates to ax_role_properties. (Closed)
Patch Set: macos Created 3 years, 6 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.cc
diff --git a/content/browser/accessibility/browser_accessibility.cc b/content/browser/accessibility/browser_accessibility.cc
index 06f810608ca84573a7973ccc9c5d4038b91a0dbd..30a9ab4debd5250d8a859cdf28c4bf29eece8c2b 100644
--- a/content/browser/accessibility/browser_accessibility.cc
+++ b/content/browser/accessibility/browser_accessibility.cc
@@ -895,55 +895,6 @@ bool BrowserAccessibility::IsClickable() const {
return ui::IsRoleClickable(GetRole());
}
-bool BrowserAccessibility::IsControl() const {
- switch (GetRole()) {
- case ui::AX_ROLE_BUTTON:
- case ui::AX_ROLE_CHECK_BOX:
- case ui::AX_ROLE_COLOR_WELL:
- case ui::AX_ROLE_COMBO_BOX:
- case ui::AX_ROLE_DISCLOSURE_TRIANGLE:
- case ui::AX_ROLE_LIST_BOX:
- case ui::AX_ROLE_MENU:
- case ui::AX_ROLE_MENU_BAR:
- case ui::AX_ROLE_MENU_BUTTON:
- case ui::AX_ROLE_MENU_ITEM:
- case ui::AX_ROLE_MENU_ITEM_CHECK_BOX:
- case ui::AX_ROLE_MENU_ITEM_RADIO:
- case ui::AX_ROLE_MENU_LIST_OPTION:
- case ui::AX_ROLE_MENU_LIST_POPUP:
- case ui::AX_ROLE_POP_UP_BUTTON:
- case ui::AX_ROLE_RADIO_BUTTON:
- case ui::AX_ROLE_SCROLL_BAR:
- case ui::AX_ROLE_SEARCH_BOX:
- case ui::AX_ROLE_SLIDER:
- case ui::AX_ROLE_SPIN_BUTTON:
- case ui::AX_ROLE_SWITCH:
- case ui::AX_ROLE_TAB:
- case ui::AX_ROLE_TEXT_FIELD:
- case ui::AX_ROLE_TOGGLE_BUTTON:
- case ui::AX_ROLE_TREE:
- return true;
- default:
- return false;
- }
-}
-
-bool BrowserAccessibility::IsMenuRelated() const {
- switch (GetRole()) {
- case ui::AX_ROLE_MENU:
- case ui::AX_ROLE_MENU_BAR:
- case ui::AX_ROLE_MENU_BUTTON:
- case ui::AX_ROLE_MENU_ITEM:
- case ui::AX_ROLE_MENU_ITEM_CHECK_BOX:
- case ui::AX_ROLE_MENU_ITEM_RADIO:
- case ui::AX_ROLE_MENU_LIST_OPTION:
- case ui::AX_ROLE_MENU_LIST_POPUP:
- return true;
- default:
- return false;
- }
-}
-
bool BrowserAccessibility::IsNativeTextControl() const {
const std::string& html_tag = GetStringAttribute(ui::AX_ATTR_HTML_TAG);
if (html_tag == "input") {
« no previous file with comments | « content/browser/accessibility/browser_accessibility.h ('k') | content/browser/accessibility/browser_accessibility_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698