Stop SpellChecker from adding marker when checking range is modified

Stop SpellChecker from adding marker when checking range is modified

This patch makes SpellChecker not to add spelling markers if the
checking range has been modified. It also modifies layout test
editing/spelling/spellcheck-async-mutation.html, which is supposed
to test behavior in such case but was incorrectly designed.

BUG=693795
TEST=editing/spelling/spellcheck-async-mutation.html

Review-Url: https://codereview.chromium.org/2711113004
Cr-Commit-Position: refs/heads/master@{#452719}
Committed: https://chromium.googlesource.com/chromium/src/+/e7f1cb881be7f52498d58b21ea855a5db487a515

[Xiaocheng, 2017-02-23 22:44:22]

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

[commit-bot: I haz the power, 2017-02-23 22:45:04]

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-02-24 00:10:40]

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

[commit-bot: I haz the power, 2017-02-24 00:10:40]

Dry run: This issue passed the CQ dry run.

[Xiaocheng, 2017-02-24 01:43:05]

Description was changed from

==========
Stop SpellChecker from adding marker when checking range is modified

BUG=693795
==========

to

==========
Stop SpellChecker from adding marker when checking range is modified

This patch makes SpellChecker not to add spelling markers if the
checking range has been modified. It also modifies layout test
editing/spelling/spellcheck-async-mutation.html, which is supposed
to test behavior in such case but was incorrectly designed.

BUG=693795
TEST=editing/spelling/spellcheck-async-mutation.html
==========

[Xiaocheng, 2017-02-24 01:43:13]

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

[Xiaocheng, 2017-02-24 01:43:14]

PTAL.

[yosin_UTC9, 2017-02-24 02:15:13]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2017-02-24 02:15:14]

lgtm

[commit-bot: I haz the power, 2017-02-24 02:16:16]

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-02-24 02:23:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487902513875210, "parent_rev":
"b1424a997d92b500ccd797d4ee690246e40182b4", "commit_rev":
"e7f1cb881be7f52498d58b21ea855a5db487a515"}

[commit-bot: I haz the power, 2017-02-24 02:24:31]

Description was changed from

==========
Stop SpellChecker from adding marker when checking range is modified

This patch makes SpellChecker not to add spelling markers if the
checking range has been modified. It also modifies layout test
editing/spelling/spellcheck-async-mutation.html, which is supposed
to test behavior in such case but was incorrectly designed.

BUG=693795
TEST=editing/spelling/spellcheck-async-mutation.html
==========

to

==========
Stop SpellChecker from adding marker when checking range is modified

This patch makes SpellChecker not to add spelling markers if the
checking range has been modified. It also modifies layout test
editing/spelling/spellcheck-async-mutation.html, which is supposed
to test behavior in such case but was incorrectly designed.

BUG=693795
TEST=editing/spelling/spellcheck-async-mutation.html

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

[commit-bot: I haz the power, 2017-02-24 02:24:32]

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