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

Unified Diff: ui/views/accessibility/native_view_accessibility_base.cc

Issue 2860883003: A11y: Add/refactor methods for manipulating bitfields on AXNodeData. (Closed)
Patch Set: Revert comment. Created 3 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: ui/views/accessibility/native_view_accessibility_base.cc
diff --git a/ui/views/accessibility/native_view_accessibility_base.cc b/ui/views/accessibility/native_view_accessibility_base.cc
index 2ed7e26fafaed75d104c732a70d5a6ee1dc336f5..4bc12bf8f6f912f738a3e82ea25a0b2743045a93 100644
--- a/ui/views/accessibility/native_view_accessibility_base.cc
+++ b/ui/views/accessibility/native_view_accessibility_base.cc
@@ -40,13 +40,13 @@ void NativeViewAccessibilityBase::NotifyAccessibilityEvent(
const ui::AXNodeData& NativeViewAccessibilityBase::GetData() const {
data_ = ui::AXNodeData();
- data_.state = 0;
+ data_.ClearBitfields();
// Views may misbehave if their widget is closed; return an unknown role
// rather than possibly crashing.
if (!view_->GetWidget() || view_->GetWidget()->IsClosed()) {
data_.role = ui::AX_ROLE_UNKNOWN;
- data_.state = 1 << ui::AX_STATE_DISABLED;
+ data_.AddState(ui::AX_STATE_DISABLED);
return data_;
}
@@ -58,13 +58,13 @@ const ui::AXNodeData& NativeViewAccessibilityBase::GetData() const {
base::UTF16ToUTF8(description));
if (view_->IsAccessibilityFocusable())
- data_.state |= (1 << ui::AX_STATE_FOCUSABLE);
+ data_.AddState(ui::AX_STATE_FOCUSABLE);
if (!view_->enabled())
- data_.state |= (1 << ui::AX_STATE_DISABLED);
+ data_.AddState(ui::AX_STATE_DISABLED);
if (!view_->IsDrawn())
- data_.state |= (1 << ui::AX_STATE_INVISIBLE);
+ data_.AddState(ui::AX_STATE_INVISIBLE);
return data_;
}

Powered by Google App Engine
This is Rietveld 408576698