Re-factor two parameter version of endOfLine()

Re-factor two parameter version of endOfLine()

This patch expands two version of |endOfLine()| which changes behavior with
|mode| parameter into callers, one parameter version of |endOfLine()| and
|logicalEndOfLine()| to simplify implementation as a preparation for
templatizing |endOfLine()| and |logicalEndOfLine()| to introduce composed tree
version.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; no behavior changes

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201970

[yosin_UTC9, 2015-09-09 09:22:25]

yosin@chromium.org changed reviewers:
+ hajimehoshi@chromium.org, yoichio@chromium.org

[yosin_UTC9, 2015-09-09 09:22:33]

PTAL

[hajimehoshi, 2015-09-09 10:59:09]

https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
File Source/core/editing/VisibleUnits.cpp (right):

https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
Source/core/editing/VisibleUnits.cpp:1073: // a xyz xyz xyz xyz xyz xyz xyz xyz
xyz xyz </div>
Why did you change this comment?

[yosin_UTC9, 2015-09-09 11:07:35]

On 2015/09/09 10:59:09, hajimehoshi wrote:
>
https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
> File Source/core/editing/VisibleUnits.cpp (right):
> 
>
https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
> Source/core/editing/VisibleUnits.cpp:1073: // a xyz xyz xyz xyz xyz xyz xyz
xyz
> xyz xyz </div>
> Why did you change this comment?

to fit into 80 chars

[yosin_UTC9, 2015-09-09 11:08:49]

https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
File Source/core/editing/VisibleUnits.cpp (right):

https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
Source/core/editing/VisibleUnits.cpp:1073: // a xyz xyz xyz xyz xyz xyz xyz xyz
xyz xyz </div>
On 2015/09/09 10:59:09, hajimehoshi wrote:
> Why did you change this comment?

to fit into 80 chars 
in previous line.

[hajimehoshi, 2015-09-09 11:11:26]

lgtm

https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
File Source/core/editing/VisibleUnits.cpp (right):

https://codereview.chromium.org/1326403003/diff/1/Source/core/editing/Visible...
Source/core/editing/VisibleUnits.cpp:1073: // a xyz xyz xyz xyz xyz xyz xyz xyz
xyz xyz </div>
On 2015/09/09 11:08:49, Yosi_UTC9 wrote:
> On 2015/09/09 10:59:09, hajimehoshi wrote:
> > Why did you change this comment?
> 
> to fit into 80 chars 
> in previous line.

Hm, you could keep abcdefg, but that's OK to me

[hajimehoshi, 2015-09-09 11:11:32]

The CQ bit was checked by hajimehoshi@chromium.org

[commit-bot: I haz the power, 2015-09-09 11:11:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1326403003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1326403003/1

[commit-bot: I haz the power, 2015-09-09 11:16:38]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201970