Convert set{End,Start}RespectingGranularity() in VisibleSlection as pure functions

Convert set{End,Start}RespectingGranularity() in VisibleSlection as pure functions

This patch converts |set{End,Start}RespectingGranularity()| in |VisibleSlection|
to pure functions for improving readability.

This patch is intended to help debugging http://crbug.com/654710, e.g. checking
positions before/after expanding selection by granularity in selection.

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

Committed: https://crrev.com/97263d0bef05d9dfe570d49516d916bba9ca95cc
Cr-Commit-Position: refs/heads/master@{#424947}

[yosin_UTC9, 2016-10-12 09:57:27]

The CQ bit was checked by yosin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-12 09:57:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yosin_UTC9, 2016-10-12 10:52:27]

The CQ bit was checked by yosin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-12 10:52:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-12 14:31:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-12 14:31:21]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-10-13 01:05:59]

Description was changed from

==========
2016-10-12T18:55:02

BUG=
==========

to

==========
Convert set{End,Start}RespectingGranularity() in VisibleSlection as pure
functions

This patch converts |set{End,Start}RespectingGranularity()| in |VisibleSlection|
to pure functions for improving readability.

This patch is intended to help debugging http://crbug.com/654710, e.g. checking
positions before/after expanding selection by granularity in selection.

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

[yosin_UTC9, 2016-10-13 01:06:13]

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

[yosin_UTC9, 2016-10-13 01:06:13]

PTAL

[Xiaocheng, 2016-10-13 02:09:20]

lgtm

[yosin_UTC9, 2016-10-13 03:30:23]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-10-13 03:30:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-13 03:37:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-13 03:40:35]

Description was changed from

==========
Convert set{End,Start}RespectingGranularity() in VisibleSlection as pure
functions

This patch converts |set{End,Start}RespectingGranularity()| in |VisibleSlection|
to pure functions for improving readability.

This patch is intended to help debugging http://crbug.com/654710, e.g. checking
positions before/after expanding selection by granularity in selection.

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

to

==========
Convert set{End,Start}RespectingGranularity() in VisibleSlection as pure
functions

This patch converts |set{End,Start}RespectingGranularity()| in |VisibleSlection|
to pure functions for improving readability.

This patch is intended to help debugging http://crbug.com/654710, e.g. checking
positions before/after expanding selection by granularity in selection.

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

Committed: https://crrev.com/97263d0bef05d9dfe570d49516d916bba9ca95cc
Cr-Commit-Position: refs/heads/master@{#424947}
==========

[commit-bot: I haz the power, 2016-10-13 03:40:36]

Patchset 2 (id:??) landed as
https://crrev.com/97263d0bef05d9dfe570d49516d916bba9ca95cc
Cr-Commit-Position: refs/heads/master@{#424947}