Make Position::AfterNode() to take const Node& instead of Node*

Make Position::AfterNode() to take const Node& instead of Node*

This patch makes |Position::AfterNode()| to take |const Node&| instead of
|Node*| since |Node| parameter can not be nullptr for improving code health.


This patch is a preparation of make |Position| to hold |const Node&|[1] and
similar to the patch[2] which change |BeforeNode()|.

[1] http://crbug.com/735327 Position should hold const Node instead of Node
[2] http://crbug.com/2950053002: Make Position::BeforeNode() to take const Node& 
instead of Node*


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

Review-Url: https://codereview.chromium.org/2953683004
Cr-Commit-Position: refs/heads/master@{#481450}
Committed: https://chromium.googlesource.com/chromium/src/+/1eeccaf7268c04ca0f50c076c9ac827167b6735a

[yosin_UTC9, 2017-06-22 03:40:13]

Description was changed from

==========
2017-06-22T12:37:16

BUG=

2017-06-22T12:36:39
==========

to

==========
Make Position::AfterNode() to take const Node& instead of Node*

This patch makes |Position::AfterNode()| to take |const Node&| instead of
|Node*| since |Node| parameter can not be nullptr for improving code health.


This patch is a preparation of make |Position| to hold |const Node&|[1] and
similar to the patch[2] which change |BeforeNode()|.

[1] http://crbug.com/735327 Position should hold const Node instead of Node
[2] http://crbug.com/2950053002: Make Position::BeforeNode() to take const Node&

instead of Node*


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

[yosin_UTC9, 2017-06-22 03:40:19]

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

[commit-bot: I haz the power, 2017-06-22 03:40:43]

Dry run: CQ is trying da patch.

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

[yosin_UTC9, 2017-06-22 04:53:56]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org

[yosin_UTC9, 2017-06-22 04:53:56]

PTAL

Including core/dom/RangeTest.cpp and core/editing/

[tkent, 2017-06-22 04:58:38]

lgtm

[yosin_UTC9, 2017-06-22 05:00:45]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2017-06-22 05:00:46]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-06-22 05:01:05]

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, 2017-06-22 05:34:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498107646713220, "parent_rev":
"aa449b76a4ea265126b7c3bdd5084589e5b66adf", "commit_rev":
"1eeccaf7268c04ca0f50c076c9ac827167b6735a"}

[commit-bot: I haz the power, 2017-06-22 05:34:38]

Description was changed from

==========
Make Position::AfterNode() to take const Node& instead of Node*

This patch makes |Position::AfterNode()| to take |const Node&| instead of
|Node*| since |Node| parameter can not be nullptr for improving code health.


This patch is a preparation of make |Position| to hold |const Node&|[1] and
similar to the patch[2] which change |BeforeNode()|.

[1] http://crbug.com/735327 Position should hold const Node instead of Node
[2] http://crbug.com/2950053002: Make Position::BeforeNode() to take const Node&

instead of Node*


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

to

==========
Make Position::AfterNode() to take const Node& instead of Node*

This patch makes |Position::AfterNode()| to take |const Node&| instead of
|Node*| since |Node| parameter can not be nullptr for improving code health.


This patch is a preparation of make |Position| to hold |const Node&|[1] and
similar to the patch[2] which change |BeforeNode()|.

[1] http://crbug.com/735327 Position should hold const Node instead of Node
[2] http://crbug.com/2950053002: Make Position::BeforeNode() to take const Node&

instead of Node*


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

Review-Url: https://codereview.chromium.org/2953683004
Cr-Commit-Position: refs/heads/master@{#481450}
Committed:
https://chromium.googlesource.com/chromium/src/+/1eeccaf7268c04ca0f50c076c9ac...
==========

[commit-bot: I haz the power, 2017-06-22 05:34:39]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1eeccaf7268c04ca0f50c076c9ac...