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

Unified Diff: ui/views/widget/native_widget_mac_accessibility_unittest.mm

Issue 2230093002: MacViews a11y: Allow accessibility clients to set the AXValue on some controls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Don't show attributes are writable if there is not callback provided to change its value. Created 4 years, 4 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/widget/native_widget_mac_accessibility_unittest.mm
diff --git a/ui/views/widget/native_widget_mac_accessibility_unittest.mm b/ui/views/widget/native_widget_mac_accessibility_unittest.mm
index a95cf0ad3f595956c94771b595f3fb05656a0e0f..92d883872643d96cda8b5df53c96682dca1a6feb 100644
--- a/ui/views/widget/native_widget_mac_accessibility_unittest.mm
+++ b/ui/views/widget/native_widget_mac_accessibility_unittest.mm
@@ -304,4 +304,27 @@ TEST_F(NativeWidgetMacAccessibilityTest, TextfieldEditableAttributes) {
NSAccessibilityVisibleCharacterRangeAttribute) rangeValue]));
}
+// Test writing accessibility attributes via an accessibility client.
+TEST_F(NativeWidgetMacAccessibilityTest, TextfieldWritableAttributes) {
+ AddChildTextfield(GetWidgetBounds().size());
+
+ // Get the textfield accessibility object.
+ NSPoint midpoint = gfx::ScreenPointToNSPoint(GetWidgetBounds().CenterPoint());
+ id textfield = [widget()->GetNativeWindow() accessibilityHitTest:midpoint];
+ EXPECT_TRUE(textfield);
+
+ // Make sure it's the correct accessibility object.
+ id value =
+ [textfield accessibilityAttributeValue:NSAccessibilityValueAttribute];
+ EXPECT_NSEQ(kTestStringValue, value);
+
+ // Write a new NSAccessibilityValueAttribute.
+ EXPECT_TRUE([textfield
+ accessibilityIsAttributeSettable:NSAccessibilityValueAttribute]);
+ [textfield accessibilitySetValue:kTestPlaceholderText
+ forAttribute:NSAccessibilityValueAttribute];
+ EXPECT_NSEQ(kTestPlaceholderText,
+ AttributeValueAtMidpoint(NSAccessibilityValueAttribute));
+}
tapted 2016/08/11 04:23:26 We should also check that the value of the views::
Patti Lor 2016/08/12 01:57:42 Done. Also added tests for a read-only textfield.
+
} // namespace views

Powered by Google App Engine
This is Rietveld 408576698