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

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

Issue 2341633006: MacViews/a11y: Allow accessibility clients to update the selected text. (Closed)
Patch Set: Make set value callbacks not have a const bool |replace| arg. Created 4 years, 2 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.cc
diff --git a/ui/views/accessibility/native_view_accessibility.cc b/ui/views/accessibility/native_view_accessibility.cc
index a984c3c6c08c901fbe65b13d87166e5c075d3a78..f23ce8c4162eda71f54a72d1b01f4769c8ca1bc9 100644
--- a/ui/views/accessibility/native_view_accessibility.cc
+++ b/ui/views/accessibility/native_view_accessibility.cc
@@ -213,14 +213,15 @@ void NativeViewAccessibility::DoDefaultAction() {
ui::EF_LEFT_MOUSE_BUTTON));
}
-bool NativeViewAccessibility::SetStringValue(const base::string16& new_value) {
+bool NativeViewAccessibility::SetStringValue(const base::string16& new_value,
+ bool replace) {
// Return an error if the view can't set the value.
if (!CanSetStringValue())
return false;
ui::AXViewState state;
view_->GetAccessibleState(&state);
- state.set_value_callback.Run(new_value);
+ state.set_value_callback.Run(new_value, replace);
return true;
}

Powered by Google App Engine
This is Rietveld 408576698