[Webkit-unassigned] [Bug 32605] Regression: Selection anchor + focus swap when arrow keys after setBaseAndExtent

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 8 08:17:19 PDT 2010


--- Comment #14 from Darin Adler <darin at apple.com>  2010-07-08 08:17:18 PST ---
(From update of attachment 60770)
> -        m_frame->selection()->setSelection(newSelection, m_frame->selectionGranularity());
> +        m_frame->selection()->setSelection(newSelection, m_frame->selectionGranularity(), false);

This is a classic example of why we don't use boolean arguments when constants are passed. This patch would be considerably better if the argument used an enum with good names instead.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list