[Webkit-unassigned] [Bug 144489] AX: setting focus via accessibility object needs to set isSynchronizing in resulting selection intent

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=144489

--- Comment #5 from Darin Adler <darin at apple.com> ---
Comment on attachment 252192
  --> https://bugs.webkit.org/attachment.cgi?id=252192
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=252192&action=review

> Source/WebCore/accessibility/AXObjectCache.h:202
> -    void setTextSelectionIntent(AXTextStateChangeIntent);
> +    void setTextSelectionIntent(AXTextStateChangeIntent, bool isSynchronizing = false);

When callers are all passing constants to a boolean in a case like this, the WebKit project tradition is to either use two functions (which seems like the right solution in this case) so the names of the functions can make it clear what the difference is, or to use an enum so the names of the constants can make that clear.

Since every single caller uses a boolean, I suggest two separate named functions rather than a function with an optional boolean argument.

I also suggest changing this function to take const AXStateChangeIntent& since that would be slightly more efficient; not sure why this passes by value instead.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150501/c5e92b71/attachment-0001.html>