Change SelectionStartEnd() returning values instead of assigning into ref args.

Change SelectionStartEnd() returning values instead of assigning into ref args.

LayoutSelection::SelectionStartEnd(int& start_pos, int& end_pos);
=> std::pair<int, int> LayoutSelection::SelectionStartEnd();

TEST=No change in behavior.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2833343002
Cr-Commit-Position: refs/heads/master@{#466933}
Committed: https://chromium.googlesource.com/chromium/src/+/9e8f21af25c5425136583d7658850e4a7db2146e

[yoichio, 2017-04-24 07:43:07]

Description was changed from

==========
Change SelectionStartEnd() returning std::pair(start,end)

BUG=708453
==========

to

==========
Change SelectionStartEnd() returning std::pair(start,end)

BUG=708453
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[yoichio, 2017-04-24 07:47:37]

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

[commit-bot: I haz the power, 2017-04-24 07:47:55]

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, 2017-04-24 07:50:56]

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

[commit-bot: I haz the power, 2017-04-24 07:50:57]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yoichio, 2017-04-25 02:23:40]

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

[commit-bot: I haz the power, 2017-04-25 02:24:06]

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, 2017-04-25 03:56:34]

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

[commit-bot: I haz the power, 2017-04-25 03:56:34]

Dry run: This issue passed the CQ dry run.

[yoichio, 2017-04-25 04:04:44]

Patchset #1 (id:1) has been deleted

[yoichio, 2017-04-25 04:07:21]

Description was changed from

==========
Change SelectionStartEnd() returning std::pair(start,end)

BUG=708453
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Change SelectionStartEnd() returning values instead of assigning into ref args.

LayoutSelection::SelectionStartEnd(int& start_pos, int& end_pos);
=> std::pair<int, int> LayoutSelection::SelectionStartEnd();

TEST=No change in behavior.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[yoichio, 2017-04-25 04:07:21]

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

[yosin_UTC9, 2017-04-25 07:05:37]

lgtm

Nice!

[kojii, 2017-04-25 08:51:08]

lgtm

[yoichio, 2017-04-25 09:11:22]

The CQ bit was checked by yoichio@chromium.org

[commit-bot: I haz the power, 2017-04-25 09:11:40]

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-04-25 09:15:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493111482469610,
"parent_rev": "20a75f52c468e41ecc06a80b3cd7486b203c00a7", "commit_rev":
"9e8f21af25c5425136583d7658850e4a7db2146e"}

[commit-bot: I haz the power, 2017-04-25 09:15:35]

Description was changed from

==========
Change SelectionStartEnd() returning values instead of assigning into ref args.

LayoutSelection::SelectionStartEnd(int& start_pos, int& end_pos);
=> std::pair<int, int> LayoutSelection::SelectionStartEnd();

TEST=No change in behavior.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Change SelectionStartEnd() returning values instead of assigning into ref args.

LayoutSelection::SelectionStartEnd(int& start_pos, int& end_pos);
=> std::pair<int, int> LayoutSelection::SelectionStartEnd();

TEST=No change in behavior.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2833343002
Cr-Commit-Position: refs/heads/master@{#466933}
Committed:
https://chromium.googlesource.com/chromium/src/+/9e8f21af25c5425136583d765885...
==========

[commit-bot: I haz the power, 2017-04-25 09:15:36]

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