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

Unified Diff: third_party/WebKit/Source/core/dom/AccessibleNode.cpp

Issue 2973973002: Implement AccessibleNode.hasPopUp for Accessibility Object Model phase 1 (Closed)
Patch Set: Fix handling of aria-haspopup=false Created 3 years, 5 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: third_party/WebKit/Source/core/dom/AccessibleNode.cpp
diff --git a/third_party/WebKit/Source/core/dom/AccessibleNode.cpp b/third_party/WebKit/Source/core/dom/AccessibleNode.cpp
index 18a467368d886561072f64aab44652f6c855dd18..9823ee43d484ccf42b31f95202d2688d7366970b 100644
--- a/third_party/WebKit/Source/core/dom/AccessibleNode.cpp
+++ b/third_party/WebKit/Source/core/dom/AccessibleNode.cpp
@@ -24,6 +24,8 @@ QualifiedName GetCorrespondingARIAAttribute(AOMStringProperty property) {
return aria_checkedAttr;
case AOMStringProperty::kCurrent:
return aria_currentAttr;
+ case AOMStringProperty::kHasPopUp:
+ return aria_haspopupAttr;
case AOMStringProperty::kInvalid:
return aria_invalidAttr;
case AOMStringProperty::kKeyShortcuts:
@@ -685,6 +687,15 @@ void AccessibleNode::setFlowTo(AccessibleNodeList* flow_to) {
NotifyAttributeChanged(aria_flowtoAttr);
}
+AtomicString AccessibleNode::hasPopUp() const {
+ return GetProperty(element_, AOMStringProperty::kHasPopUp);
+}
+
+void AccessibleNode::setHasPopUp(const AtomicString& has_popup) {
+ SetStringProperty(AOMStringProperty::kHasPopUp, has_popup);
+ NotifyAttributeChanged(aria_haspopupAttr);
+}
+
bool AccessibleNode::hidden(bool& is_null) const {
return GetProperty(element_, AOMBooleanProperty::kHidden, is_null);
}
« no previous file with comments | « third_party/WebKit/Source/core/dom/AccessibleNode.h ('k') | third_party/WebKit/Source/core/dom/AccessibleNode.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698