Make SpellChecker::removeMarkers() to take EphemeralRange instead of VisibleSelection

Make SpellChecker::removeMarkers() to take EphemeralRange instead of VisibleSelection

This patch makes |SpellChecker::removeMarkers()| to take |EphemeralRange|
instead of |VisibleSelection| as preparation of reduce usage of
|VisibleSelection| for improving code health.

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

Review-Url: https://codereview.chromium.org/2740873003
Cr-Commit-Position: refs/heads/master@{#455705}
Committed: https://chromium.googlesource.com/chromium/src/+/72f3c9b47d08185135faeab172c1130f34f1ca66

[yosin_UTC9, 2017-03-09 05:48:24]

Description was changed from

==========
2017-03-09T14:47:18

2017-03-09T14:47:12

BUG=
==========

to

==========
2017-03-09T14:47:18

2017-03-09T14:47:12

BUG=657237
==========

[yosin_UTC9, 2017-03-09 05:48:53]

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

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

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-03-09 06:55:10]

Description was changed from

==========
2017-03-09T14:47:18

2017-03-09T14:47:12

BUG=657237
==========

to

==========
Make SpellChecker::removeMarkers() to take EphemeralRange instead of
VisibleSelection

This patch makes |SpellChecker::removeMarkers()| to take |EphemeralRange|
instead of |VisibleSelection| to reduce usage of |VisibleSelection| for
improving code health.

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

[yosin_UTC9, 2017-03-09 06:55:46]

Description was changed from

==========
Make SpellChecker::removeMarkers() to take EphemeralRange instead of
VisibleSelection

This patch makes |SpellChecker::removeMarkers()| to take |EphemeralRange|
instead of |VisibleSelection| to reduce usage of |VisibleSelection| for
improving code health.

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

to

==========
Make SpellChecker::removeMarkers() to take EphemeralRange instead of
VisibleSelection

This patch makes |SpellChecker::removeMarkers()| to take |EphemeralRange|
instead of |VisibleSelection| as preparation of reduce usage of
|VisibleSelection| for improving code health.

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

[yosin_UTC9, 2017-03-09 06:56:02]

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

[yosin_UTC9, 2017-03-09 06:56:03]

PTAL

[commit-bot: I haz the power, 2017-03-09 07:14:09]

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

[commit-bot: I haz the power, 2017-03-09 07:14:10]

Dry run: This issue passed the CQ dry run.

[yoichio, 2017-03-09 08:54:02]

lgtm

[yosin_UTC9, 2017-03-09 09:03:33]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-03-09 09:03:57]

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-03-09 09:09:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489050213398710, "parent_rev":
"a9a8f4e062bd1c72c3b511de3c41611e24b7949e", "commit_rev":
"72f3c9b47d08185135faeab172c1130f34f1ca66"}

[commit-bot: I haz the power, 2017-03-09 09:10:04]

Description was changed from

==========
Make SpellChecker::removeMarkers() to take EphemeralRange instead of
VisibleSelection

This patch makes |SpellChecker::removeMarkers()| to take |EphemeralRange|
instead of |VisibleSelection| as preparation of reduce usage of
|VisibleSelection| for improving code health.

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

to

==========
Make SpellChecker::removeMarkers() to take EphemeralRange instead of
VisibleSelection

This patch makes |SpellChecker::removeMarkers()| to take |EphemeralRange|
instead of |VisibleSelection| as preparation of reduce usage of
|VisibleSelection| for improving code health.

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

Review-Url: https://codereview.chromium.org/2740873003
Cr-Commit-Position: refs/heads/master@{#455705}
Committed:
https://chromium.googlesource.com/chromium/src/+/72f3c9b47d08185135faeab172c1...
==========

[commit-bot: I haz the power, 2017-03-09 09:10:06]

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