RESOLVED FIXED 66499
Rename VisiblePosition::honorEditableBoundary* to VisiblePosition::honorEditingBoundary* 
https://bugs.webkit.org/show_bug.cgi?id=66499
Summary Rename VisiblePosition::honorEditableBoundary* to VisiblePosition::honorEditi...
Ryosuke Niwa
Reported 2011-08-18 14:40:27 PDT
The following member functions of VisiblePosition use the non-standard term edit*able* boundary. We should rename these functions to use edit*ing* boundary for consistency. VisiblePosition honorEditableBoundaryAtOrBefore(const VisiblePosition&) const; VisiblePosition honorEditableBoundaryAtOrAfter(const VisiblePosition&) const;
Attachments
Patch (8.52 KB, patch)
2011-08-19 10:39 PDT, Van Lam 		no flags
DetailsFormatted DiffDiff
Revised patch (8.49 KB, patch)
2011-08-19 13:50 PDT, Van Lam 		no flags
DetailsFormatted DiffDiff
Revised patch (8.49 KB, patch)
2011-08-19 14:03 PDT, Van Lam 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Van Lam
Comment 1 2011-08-19 10:39:44 PDT
Created attachment 104524 [details] Patch
Ryosuke Niwa
Comment 2 2011-08-19 10:47:11 PDT
Comment on attachment 104524 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104524&action=review > Source/WebCore/ChangeLog:8 > + No new tests. (OOPS!) Nit: you should remove this line.
Van Lam
Comment 3 2011-08-19 13:50:59 PDT
Created attachment 104558 [details] Revised patch Removed the "no new tests" line from changelog.
Van Lam
Comment 4 2011-08-19 14:03:40 PDT
Created attachment 104561 [details] Revised patch Fixed changelog.
WebKit Review Bot
Comment 5 2011-08-19 15:20:32 PDT
Comment on attachment 104561 [details] Revised patch Clearing flags on attachment: 104561 Committed r93443: <http://trac.webkit.org/changeset/93443>
WebKit Review Bot
Comment 6 2011-08-19 15:20:37 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.