[DMC #1.2] Add DocumentMarkerListEditor::MoveMarkers()

Add DocumentMarkerListEditor::MoveMarkers()

Follow-up to https://codereview.chromium.org/2812423002/ (add
DocumentMarkerListEditor class) and https://codereview.chromium.org/2819173002
(refactor DocumentMarkerController::MoveMarkers()): this CL moves some of the
DocumentMarkerController::MoveMarkers() code into a new
DocumentMarkerListEditor::MoveMarkers() method

BUG=707867
Review-Url: https://codereview.chromium.org/2820983002
Cr-Commit-Position: refs/heads/master@{#465894}
Committed: https://chromium.googlesource.com/chromium/src/+/954977d31f907b814ca4aef51bbd76fb9a55f1f5

[rlanday, 2017-04-17 06:47:06]

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

[rlanday, 2017-04-17 07:21:35]

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

[commit-bot: I haz the power, 2017-04-17 07:21:50]

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-17 07:30:38]

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

[commit-bot: I haz the power, 2017-04-17 07:30:39]

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...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rlanday, 2017-04-17 07:51:56]

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

[commit-bot: I haz the power, 2017-04-17 07:52:09]

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-17 07:54:52]

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

[commit-bot: I haz the power, 2017-04-17 07:54:53]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[rlanday, 2017-04-17 08:11:19]

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

[commit-bot: I haz the power, 2017-04-17 08:11:38]

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-17 09:59:13]

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

[commit-bot: I haz the power, 2017-04-17 09:59:14]

Dry run: This issue passed the CQ dry run.

[rlanday, 2017-04-18 01:52:32]

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

[Xiaocheng, 2017-04-18 03:45:53]

lgtm

[rlanday, 2017-04-18 05:08:45]

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

[commit-bot: I haz the power, 2017-04-18 05:09:05]

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-18 07:12:17]

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

[commit-bot: I haz the power, 2017-04-18 07:12:18]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2017-04-20 01:26:56]

lgtm w/ small nit

https://codereview.chromium.org/2820983002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
(right):

https://codereview.chromium.org/2820983002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp:342:
unsigned end_offset = length - 1;
nit: s/unsigned/const unsigned/

[rlanday, 2017-04-20 02:15:34]

The CQ bit was checked by rlanday@chromium.org

[rlanday, 2017-04-20 02:15:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
xiaochengh@chromium.org, yosin@chromium.org
Link to the patchset: https://codereview.chromium.org/2820983002/#ps100001
(title: "Fix nit")

[commit-bot: I haz the power, 2017-04-20 02:16:00]

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-20 03:56:16]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1492654534162860,
"parent_rev": "5a560fe4e50c4532a181a24f26bdb6f572344a63", "commit_rev":
"954977d31f907b814ca4aef51bbd76fb9a55f1f5"}

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

Description was changed from

==========
Add DocumentMarkerListEditor::MoveMarkers()

Follow-up to https://codereview.chromium.org/2812423002/ (add
DocumentMarkerListEditor class) and https://codereview.chromium.org/2819173002
(refactor DocumentMarkerController::MoveMarkers()): this CL moves some of the
DocumentMarkerController::MoveMarkers() code into a new
DocumentMarkerListEditor::MoveMarkers() method

BUG=707867
==========

to

==========
Add DocumentMarkerListEditor::MoveMarkers()

Follow-up to https://codereview.chromium.org/2812423002/ (add
DocumentMarkerListEditor class) and https://codereview.chromium.org/2819173002
(refactor DocumentMarkerController::MoveMarkers()): this CL moves some of the
DocumentMarkerController::MoveMarkers() code into a new
DocumentMarkerListEditor::MoveMarkers() method

BUG=707867

Review-Url: https://codereview.chromium.org/2820983002
Cr-Commit-Position: refs/heads/master@{#465894}
Committed:
https://chromium.googlesource.com/chromium/src/+/954977d31f907b814ca4aef51bbd...
==========

[commit-bot: I haz the power, 2017-04-20 03:56:28]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/954977d31f907b814ca4aef51bbd...