From 0540dc569e5761378182485aadf4c91641f67d56 Mon Sep 17 00:00:00 2001 From: Jocelyn Turcotte Date: Thu, 21 Aug 2014 13:15:29 +0200 Subject: Revert "Improve roles" This reverts commit 7ea45766f6c49de9a81a387013fb1b98ab82766d. This blocks our integration into qt5.git for now as we would need to update qtbase as well and prevent us from pushing build fixes beyond this change. Let's get it in again after the alpha. Change-Id: Ie57c99dd91fc19ae2695f8d881a0a8318e988806 Reviewed-by: Frederik Gladhorn --- src/core/browser_accessibility_qt.cpp | 103 +++++++++++----------------------- 1 file changed, 33 insertions(+), 70 deletions(-) (limited to 'src/core/browser_accessibility_qt.cpp') diff --git a/src/core/browser_accessibility_qt.cpp b/src/core/browser_accessibility_qt.cpp index d2710b0ea..13f2adba5 100644 --- a/src/core/browser_accessibility_qt.cpp +++ b/src/core/browser_accessibility_qt.cpp @@ -170,51 +170,38 @@ QAccessible::Role BrowserAccessibilityQt::role() const // These roles all directly correspond to blink accessibility roles, // keep these alphabetical. case ui::AX_ROLE_ALERT: - case ui::AX_ROLE_ALERT_DIALOG: return QAccessible::AlertMessage; case ui::AX_ROLE_ANNOTATION: - return QAccessible::StaticText; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_APPLICATION: return QAccessible::Document; // returning Application here makes Qt return the top level app object case ui::AX_ROLE_ARTICLE: - return QAccessible::Section; + return QAccessible::Document; // FIXME case ui::AX_ROLE_BROWSER: return QAccessible::Document; // FIXME - case ui::AX_ROLE_BANNER: - return QAccessible::Section; case ui::AX_ROLE_BUSY_INDICATOR: return QAccessible::Animation; // FIXME case ui::AX_ROLE_BUTTON: return QAccessible::Button; - case ui::AX_ROLE_BUTTON_DROP_DOWN: - return QAccessible::Button; case ui::AX_ROLE_CANVAS: return QAccessible::Canvas; case ui::AX_ROLE_CELL: return QAccessible::Cell; case ui::AX_ROLE_CHECK_BOX: return QAccessible::CheckBox; - case ui::AX_ROLE_CLIENT: - return QAccessible::Client; case ui::AX_ROLE_COLOR_WELL: - return QAccessible::ColorChooser; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_COLUMN: return QAccessible::Column; case ui::AX_ROLE_COLUMN_HEADER: return QAccessible::ColumnHeader; case ui::AX_ROLE_COMBO_BOX: return QAccessible::ComboBox; - case ui::AX_ROLE_COMPLEMENTARY: - return QAccessible::ComplementaryContent; - case ui::AX_ROLE_CONTENT_INFO: - return QAccessible::Section; case ui::AX_ROLE_DEFINITION: - return QAccessible::Paragraph; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_DESCRIPTION_LIST_DETAIL: - return QAccessible::Paragraph; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_DESCRIPTION_LIST_TERM: - return QAccessible::ListItem; - case ui::AX_ROLE_DESKTOP: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_DIALOG: return QAccessible::Dialog; @@ -227,69 +214,57 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_DOCUMENT: return QAccessible::Document; case ui::AX_ROLE_DRAWER: - return QAccessible::Client; // FIXME + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_EDITABLE_TEXT: return QAccessible::EditableText; - case ui::AX_ROLE_EMBEDDED_OBJECT: - return QAccessible::Grouping; // FIXME case ui::AX_ROLE_FOOTER: - return QAccessible::Footer; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_FORM: - return QAccessible::Form; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_GRID: - return QAccessible::Table; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_GROUP: return QAccessible::Grouping; case ui::AX_ROLE_GROW_AREA: - return QAccessible::Grip; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_HEADING: - return QAccessible::Heading; + return QAccessible::StaticText; // FIXME case ui::AX_ROLE_HELP_TAG: - return QAccessible::HelpBalloon; // FIXME + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_HORIZONTAL_RULE: - return QAccessible::Separator; - case ui::AX_ROLE_IFRAME: - return QAccessible::Grouping; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_IGNORED: return QAccessible::NoRole; case ui::AX_ROLE_IMAGE: return QAccessible::Graphic; case ui::AX_ROLE_IMAGE_MAP: - return QAccessible::Graphic; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_IMAGE_MAP_LINK: - return QAccessible::Link; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_INCREMENTOR: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_INLINE_TEXT_BOX: return QAccessible::EditableText; case ui::AX_ROLE_LABEL_TEXT: return QAccessible::StaticText; - case ui::AX_ROLE_LEGEND: - return QAccessible::NoRole; // FIXME case ui::AX_ROLE_LINK: return QAccessible::Link; case ui::AX_ROLE_LIST: return QAccessible::List; case ui::AX_ROLE_LIST_BOX: - return QAccessible::List; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_LIST_BOX_OPTION: - return QAccessible::ListItem; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_LIST_ITEM: return QAccessible::ListItem; case ui::AX_ROLE_LIST_MARKER: - return QAccessible::StaticText; - case ui::AX_ROLE_LOCATION_BAR: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_LOG: return QAccessible::NoRole; // FIXME - case ui::AX_ROLE_MAIN: - return QAccessible::NoRole; // FIXME case ui::AX_ROLE_MARQUEE: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_MATH: - return QAccessible::Equation; - case ui::AX_ROLE_MATH_ELEMENT: - return QAccessible::Equation; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_MATTE: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_MENU: @@ -301,21 +276,15 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_MENU_BUTTON: return QAccessible::MenuItem; case ui::AX_ROLE_MENU_LIST_OPTION: - return QAccessible::MenuItem; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_MENU_LIST_POPUP: return QAccessible::PopupMenu; - case ui::AX_ROLE_NAVIGATION: - return QAccessible::Section; case ui::AX_ROLE_NOTE: - return QAccessible::Note; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_OUTLINE: - return QAccessible::Tree; - case ui::AX_ROLE_PANE: - return QAccessible::Pane; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_PARAGRAPH: return QAccessible::Paragraph; - case ui::AX_ROLE_POP_UP_BUTTON: - return QAccessible::ComboBox; case ui::AX_ROLE_PRESENTATIONAL: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_PROGRESS_INDICATOR: @@ -323,9 +292,9 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_RADIO_BUTTON: return QAccessible::RadioButton; case ui::AX_ROLE_RADIO_GROUP: - return QAccessible::Grouping; + return QAccessible::RadioButton; case ui::AX_ROLE_REGION: - return QAccessible::Section; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_ROW: return QAccessible::Row; case ui::AX_ROLE_ROW_HEADER: @@ -335,13 +304,9 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_RULER_MARKER: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_SCROLL_AREA: - return QAccessible::Client; // FIXME + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_SCROLL_BAR: return QAccessible::ScrollBar; - case ui::AX_ROLE_SEAMLESS_WEB_AREA: - return QAccessible::NoRole; // FIXME - case ui::AX_ROLE_SEARCH: - return QAccessible::Section; case ui::AX_ROLE_SHEET: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_SLIDER: @@ -355,21 +320,21 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_SPLITTER: return QAccessible::Splitter; case ui::AX_ROLE_SPLIT_GROUP: - return QAccessible::Splitter; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_STATIC_TEXT: return QAccessible::StaticText; case ui::AX_ROLE_STATUS: - return QAccessible::StatusBar; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_SVG_ROOT: - return QAccessible::Graphic; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_SYSTEM_WIDE: return QAccessible::NoRole; // FIXME + case ui::AX_ROLE_TAB: + return QAccessible::PageTab; case ui::AX_ROLE_TABLE: return QAccessible::Table; case ui::AX_ROLE_TABLE_HEADER_CONTAINER: - return QAccessible::Section; - case ui::AX_ROLE_TAB: - return QAccessible::PageTab; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_TAB_GROUP: // blink doesn't use (uses ROLE_TAB_LIST) return QAccessible::NoRole; // FIXME case ui::AX_ROLE_TAB_LIST: @@ -381,11 +346,9 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_TEXT_FIELD: return QAccessible::EditableText; case ui::AX_ROLE_TIMER: - return QAccessible::Clock; - case ui::AX_ROLE_TITLE_BAR: return QAccessible::NoRole; // FIXME case ui::AX_ROLE_TOGGLE_BUTTON: - return QAccessible::Button; + return QAccessible::Button; // FIXME case ui::AX_ROLE_TOOLBAR: return QAccessible::ToolBar; case ui::AX_ROLE_TOOLTIP: @@ -393,11 +356,11 @@ QAccessible::Role BrowserAccessibilityQt::role() const case ui::AX_ROLE_TREE: return QAccessible::Tree; case ui::AX_ROLE_TREE_GRID: - return QAccessible::Tree; + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_TREE_ITEM: return QAccessible::TreeItem; case ui::AX_ROLE_VALUE_INDICATOR: - return QAccessible::Client; // FIXME + return QAccessible::NoRole; // FIXME case ui::AX_ROLE_WINDOW: return QAccessible::Window; } -- cgit v1.2.3