MacViews/a11y: Allow accessibility clients to update the selected text.

ui/accessibility/ax_view_state.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_ACCESSIBILITY_AX_VIEW_STATE_H_
 #define UI_ACCESSIBILITY_AX_VIEW_STATE_H_
 
 #include <stdint.h>
 
 #include "base/callback.h"
@@ skipping 57 matching lines @@
   // The selected item's index and the count of the number of items.
   // Only applies to views with multiple choices like a listbox; both
   // index and count should be -1 otherwise.
   int index;
   int count;
 
   // An optional callback that can be used by accessibility clients to
   // set the string value of this view. This only applies to roles where
   // setting the value makes sense, like a text box. Not often used by
   // screen readers, but often used by automation software to script
-  // things like logging into portals or filling forms.
+  // things like logging into portals or filling forms. Set the second argument
+  // to insert at the cursor or replace the currently selected text.
   //
-  // This callback is only valid for the lifetime of the view, and should
+  // These callbacks are only valid for the lifetime of the view, and should

[tapted, 2016/10/20 05:54:58]

These callbacks are -> This callback is

[Patti Lor, 2016/10/24 04:31:48]

Done.

   // be a safe no-op if the view is deleted. Typically, accessible views
   // should use a WeakPtr when binding the callback.
-  base::Callback<void(const base::string16&)> set_value_callback;
+  base::Callback<void(const base::string16&, const bool&)> set_value_callback;

[tapted, 2016/10/20 05:54:59]

`const bool&` -> just `bool` -- typically only classes/structs get passed by
const-reference

[Patti Lor, 2016/10/24 04:31:48]

Done.

 
  private:
   uint32_t state_;
 };
 
 }  // namespace ui
 
 #endif  // UI_ACCESSIBILITY_AX_VIEW_STATE_H_