Hope crrev.com/2221903002 is a correct fix.

It turns out that crrev.com/2221903002 broke unit test SpellCheckProviderTest.MultiLineText, failing the 2nd and 3rd assertions. It seems that the behavior reported in this issue is intended, as the unit test states "SpellCheckProvider class does not spellcheck text while we are typing a word".

However, it also mistakenly determines some other cases as "while typing", including the case in the issue description.

I'm wondering whether SpellCheckProvider should judge if the user is typing. It seems that Blink is a better place to do this job.

Changing issue summary. SpellCheckProvider has no reliable way to check the typing progress, and hence, should not check it. This job should be done in Blink.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0fcdf64bf2b57cfc3c21965bd037d198f7fabd4e

commit 0fcdf64bf2b57cfc3c21965bd037d198f7fabd4e
Author: xiaochengh <xiaochengh@chromium.org>
Date: Fri Feb 24 19:18:23 2017

Stop SpellCheckProvider from deciding typing progress

In order not to check spelling in the middle of typing a word,
SpellCheckProvider used to decide the typing process, which is
a job that it can't reliably accomplish due to lack of information.

Now that the Blink-side SpellChecker class is no longer sending
spell check requests in the middle of typing a word, there is no
need for SpellCheckProvider to perform the check. Hence, this patch
removes the related code from SpellCheckProvider, and modifies
tests accordingly.

BUG= 635504 
TEST=SpellCheckProviderTest.MultiLineText

Review-Url: https://codereview.chromium.org/2712833004
Cr-Commit-Position: refs/heads/master@{#452899}

[modify] https://crrev.com/0fcdf64bf2b57cfc3c21965bd037d198f7fabd4e/components/spellcheck/renderer/spellcheck_provider.cc
[modify] https://crrev.com/0fcdf64bf2b57cfc3c21965bd037d198f7fabd4e/components/spellcheck/renderer/spellcheck_provider.h
[modify] https://crrev.com/0fcdf64bf2b57cfc3c21965bd037d198f7fabd4e/components/spellcheck/renderer/spellcheck_provider_hunspell_unittest.cc

 Issue 512271  has been merged into this issue.

 Issue 610557  has been merged into this issue.

 Issue 667760  has been merged into this issue.