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

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

Issue 1952863003: Implemented the "aria-current" state on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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_win.cc
diff --git a/content/browser/accessibility/browser_accessibility_win.cc b/content/browser/accessibility/browser_accessibility_win.cc
index 73f13ecfa8a42f248725a85d6674e9b935658080..f9299bfc5a1dac59149eb2121843b53818941892 100644
--- a/content/browser/accessibility/browser_accessibility_win.cc
+++ b/content/browser/accessibility/browser_accessibility_win.cc
@@ -3381,6 +3381,33 @@ void BrowserAccessibilityWin::UpdateStep1ComputeWinAttributes() {
win_attributes_->ia2_attributes.push_back(L"explicit-name:true");
}
+ // Expose the aria-current attribute.
+ int32_t aria_current;
+ if (GetIntAttribute(ui::AX_ATTR_ARIA_CURRENT, &aria_current)) {
+ switch (static_cast<ui::AXAriaCurrent>(aria_current)) {
+ case ui::AX_ARIA_CURRENT_NONE:
+ break;
+ case ui::AX_ARIA_CURRENT_FALSE:
+ win_attributes_->ia2_attributes.push_back(L"current:false");
+ break;
+ case ui::AX_ARIA_CURRENT_TRUE:
+ win_attributes_->ia2_attributes.push_back(L"current:true");
+ break;
+ case ui::AX_ARIA_CURRENT_PAGE:
+ win_attributes_->ia2_attributes.push_back(L"current:page");
+ break;
+ case ui::AX_ARIA_CURRENT_LOCATION:
+ win_attributes_->ia2_attributes.push_back(L"current:location");
+ break;
+ case ui::AX_ARIA_CURRENT_DATE:
+ win_attributes_->ia2_attributes.push_back(L"current:date");
+ break;
+ case ui::AX_ARIA_CURRENT_TIME:
+ win_attributes_->ia2_attributes.push_back(L"current:time");
+ break;
+ }
+ }
+
// Expose table cell index.
if (IsCellOrTableHeaderRole()) {
BrowserAccessibility* table = GetParent();
@@ -3403,7 +3430,7 @@ void BrowserAccessibilityWin::UpdateStep1ComputeWinAttributes() {
if ((ia_role() == ROLE_SYSTEM_COLUMNHEADER ||
ia_role() == ROLE_SYSTEM_ROWHEADER) &&
GetIntAttribute(ui::AX_ATTR_SORT_DIRECTION, &sort_direction)) {
- switch (sort_direction) {
+ switch (static_cast<ui::AXSortDirection>(sort_direction)) {
case ui::AX_SORT_DIRECTION_UNSORTED:
win_attributes_->ia2_attributes.push_back(L"sort:none");
break;
@@ -3416,8 +3443,6 @@ void BrowserAccessibilityWin::UpdateStep1ComputeWinAttributes() {
case ui::AX_SORT_DIRECTION_OTHER:
win_attributes_->ia2_attributes.push_back(L"sort:other");
break;
- default:
- NOTREACHED();
}
}
@@ -3740,8 +3765,6 @@ std::vector<base::string16> BrowserAccessibilityWin::ComputeTextAttributes()
}
break;
}
- default:
- NOTREACHED();
}
base::string16 language(GetInheritedString16Attribute(ui::AX_ATTR_LANGUAGE));
@@ -3786,8 +3809,6 @@ std::vector<base::string16> BrowserAccessibilityWin::ComputeTextAttributes()
// Not listed in the IA2 Spec.
attributes.push_back(L"writing-mode:bt");
break;
- default:
- NOTREACHED();
}
return attributes;
@@ -4231,9 +4252,8 @@ ui::TextBoundaryType BrowserAccessibilityWin::IA2TextBoundaryToTextBoundary(
return ui::PARAGRAPH_BOUNDARY;
case IA2_TEXT_BOUNDARY_ALL:
return ui::ALL_BOUNDARY;
- default:
- NOTREACHED();
}
+ NOTREACHED();
return ui::CHAR_BOUNDARY;
}
@@ -4278,8 +4298,6 @@ LONG BrowserAccessibilityWin::FindStartOfStyle(
return text_length;
return static_cast<LONG>(iterator->first);
}
- default:
- NOTREACHED();
}
return start_offset;

Powered by Google App Engine
This is Rietveld 408576698