Change DocumentMarkerController::AddCompositionMarker() to take EphemeralRange

Change DMC::AddCompositionMarker() to take EphemeralRange

Change DocumentMarkerController::AddCompositionMarker() from taking two
Positions (start and end) to taking an EphemeralRange. This matches
AddTextMatchMarker() and is how we eventually want all the Add*Marker() methods
to work.

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

[rlanday, 2017-05-22 23:39:42]

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

[rlanday, 2017-05-22 23:40:06]

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

[commit-bot: I haz the power, 2017-05-22 23:40:11]

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-05-23 01:38:09]

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

[commit-bot: I haz the power, 2017-05-23 01:38:10]

Dry run: This issue passed the CQ dry run.

[Xiaocheng, 2017-05-24 05:23:20]

lgtm

[yosin_UTC9, 2017-05-24 09:00:04]

lgtm

Please avoid to use abbrev, DMC, in summary line. People other than us don't
know what DMC
stands for.

I updated a summary for committing this patch.

[yosin_UTC9, 2017-05-24 09:00:16]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2017-05-24 09:00:17]

lgtm

[commit-bot: I haz the power, 2017-05-24 09:00:47]

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-05-24 09:08:48]

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

[commit-bot: I haz the power, 2017-05-24 09:08:49]

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...)

[rlanday, 2017-05-24 21:20:55]

On 2017/05/24 at 09:00:04, yosin wrote:
> lgtm
> 
> Please avoid to use abbrev, DMC, in summary line. People other than us don't
know what DMC
> stands for.
> 
> I updated a summary for committing this patch.

Oh hmm, I didn't realize that fit in 80 characters :)

[rlanday, 2017-05-24 21:23:28]

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

[rlanday, 2017-05-24 21:23:36]

Adding tkent for third_party/WebKit/Source/core/testing/Internals.cpp

[tkent, 2017-05-24 23:43:33]

lgtm

[rlanday, 2017-05-24 23:43:52]

The CQ bit was checked by rlanday@chromium.org

[commit-bot: I haz the power, 2017-05-24 23:44:29]

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-05-24 23:52:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495669432326200, "parent_rev":
"97827ee523ac3f3f1d3b8b7c84b0c1fc13fc651b", "commit_rev":
"f0115f469e06ebf49f0265ec89eee5e9220f187c"}

[commit-bot: I haz the power, 2017-05-24 23:52:50]

Description was changed from

==========
Change DMC::AddCompositionMarker() to take EphemeralRange

Change DocumentMarkerController::AddCompositionMarker() from taking two
Positions (start and end) to taking an EphemeralRange. This matches
AddTextMatchMarker() and is how we eventually want all the Add*Marker() methods
to work.

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

to

==========
Change DMC::AddCompositionMarker() to take EphemeralRange

Change DocumentMarkerController::AddCompositionMarker() from taking two
Positions (start and end) to taking an EphemeralRange. This matches
AddTextMatchMarker() and is how we eventually want all the Add*Marker() methods
to work.

BUG=707867

Review-Url: https://codereview.chromium.org/2899923002
Cr-Commit-Position: refs/heads/master@{#474480}
Committed:
https://chromium.googlesource.com/chromium/src/+/f0115f469e06ebf49f0265ec89ee...
==========

[commit-bot: I haz the power, 2017-05-24 23:52:51]

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