Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node*
This patch makes |VisiblePosition::FirstPositionInNode()| to take |const Node&|
as |Position::FirstPositionInNode()| for improving code health.
This is mechanical change:
s/VisiblePositon::FirstPositionInNode(/VisiblePositon::FirstPositionInNode(*/
BUG=n/a
TEST=n/a; no behavior changes
Review-Url: https://codereview.chromium.org/2955823002
Cr-Commit-Position: refs/heads/master@{#482237}
Committed: https://chromium.googlesource.com/chromium/src/+/cbf7050ee81b7ca0390e634cfd08f764f33f1307
Description was changed from ========== 2017-06-26T13:57:23 BUG= 2017-06-26T13:57:19 ========== to ========== Make VisiblePosition::FirstPositionInNode() to take ...
3 years, 5 months ago
(2017-06-26 04:58:36 UTC)
#1
Description was changed from
==========
2017-06-26T13:57:23
BUG=
2017-06-26T13:57:19
==========
to
==========
Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node*
This patch makes |VisiblePosition::FirstPositionInNode()| to take |const Node&|
as |Position::FirstPositionInNode()| for improving code health.
This is mechanical change:
s/VisiblePositon::FirstPositionInNode(/VisiblePositon::FirstPositionInNode(*/
BUG=n/a
TEST=n/a; no behavior changes
==========
yosin_UTC9
The CQ bit was checked by yosin@chromium.org to run a CQ dry run
3 years, 5 months ago
(2017-06-26 04:58:47 UTC)
#2
CQ is committing da patch. Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498469249154360, "parent_rev": "b4ef7c0fece5fdbf8d51f4a6865056a1b03298c8", "commit_rev": "0c442e1219add2c46221b33377410a84506d94d6"}
3 years, 5 months ago
(2017-06-26 10:57:52 UTC)
#16
CQ is committing da patch.
Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498469249154360,
"parent_rev": "b4ef7c0fece5fdbf8d51f4a6865056a1b03298c8", "commit_rev":
"0c442e1219add2c46221b33377410a84506d94d6"}
commit-bot: I haz the power
CQ is committing da patch. Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498469249154360, "parent_rev": "09c04f87d103c9f0cfdddfaf5b024168e2276df6", "commit_rev": "cbf7050ee81b7ca0390e634cfd08f764f33f1307"}
3 years, 5 months ago
(2017-06-26 10:58:16 UTC)
#17
CQ is committing da patch.
Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498469249154360,
"parent_rev": "09c04f87d103c9f0cfdddfaf5b024168e2276df6", "commit_rev":
"cbf7050ee81b7ca0390e634cfd08f764f33f1307"}
commit-bot: I haz the power
Description was changed from ========== Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node* This ...
3 years, 5 months ago
(2017-06-26 10:58:26 UTC)
#18
Message was sent while issue was closed.
Description was changed from
==========
Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node*
This patch makes |VisiblePosition::FirstPositionInNode()| to take |const Node&|
as |Position::FirstPositionInNode()| for improving code health.
This is mechanical change:
s/VisiblePositon::FirstPositionInNode(/VisiblePositon::FirstPositionInNode(*/
BUG=n/a
TEST=n/a; no behavior changes
==========
to
==========
Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node*
This patch makes |VisiblePosition::FirstPositionInNode()| to take |const Node&|
as |Position::FirstPositionInNode()| for improving code health.
This is mechanical change:
s/VisiblePositon::FirstPositionInNode(/VisiblePositon::FirstPositionInNode(*/
BUG=n/a
TEST=n/a; no behavior changes
Review-Url: https://codereview.chromium.org/2955823002
Cr-Commit-Position: refs/heads/master@{#482237}
Committed:
https://chromium.googlesource.com/chromium/src/+/cbf7050ee81b7ca0390e634cfd08...
==========
commit-bot: I haz the power
Committed patchset #2 (id:20001) as https://chromium.googlesource.com/chromium/src/+/cbf7050ee81b7ca0390e634cfd08f764f33f1307
3 years, 5 months ago
(2017-06-26 10:58:28 UTC)
#19
Issue 2955823002: Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node*
(Closed)
Created 3 years, 5 months ago by yosin_UTC9
Modified 3 years, 5 months ago
Reviewers: tkent
Base URL:
Comments: 4