Spellcheck : Fix caching in cases where text is deleted.

Spellcheck : Fix caching in cases where text is deleted.

Merge to M56 (branch 2924)

This patch fixes spellcheck caching in the following case:
1. paragraph is edited by removing text from the back and
2. paragraph does not contain misspelled words.

Currently this case results in a cache miss and consequently
spellcheck requests being sent to the spellchecking service
while completely unnecessary. This fix applies to all
platforms.

This patch also adds a few unit tests for the relevant
caching specific behavior.

BUG=664247, 629609
TBR=tobiasjs@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2494123002
Cr-Commit-Position: refs/heads/master@{#434710}
(cherry picked from commit cb1e0169211135db1dd2e0220434bcec2efd7f16)

[timvolodine, 2016-12-02 16:02:48]

Description was changed from

==========
Spellcheck : Fix caching in cases where text is deleted.

Merge to M56 (branch 2924)

This patch fixes spellcheck caching in the following case:
1. paragraph is edited by removing text from the back and
2. paragraph does not contain misspelled words.

Currently this case results in a cache miss and consequently
spellcheck requests being sent to the spellchecking service
while completely unnecessary. This fix applies to all
platforms.

This patch also adds a few unit tests for the relevant
caching specific behavior.

BUG=664247,629609

Review-Url: https://codereview.chromium.org/2494123002
Cr-Commit-Position: refs/heads/master@{#434710}
(cherry picked from commit cb1e0169211135db1dd2e0220434bcec2efd7f16)
==========

to

==========
Spellcheck : Fix caching in cases where text is deleted.

Merge to M56 (branch 2924)

This patch fixes spellcheck caching in the following case:
1. paragraph is edited by removing text from the back and
2. paragraph does not contain misspelled words.

Currently this case results in a cache miss and consequently
spellcheck requests being sent to the spellchecking service
while completely unnecessary. This fix applies to all
platforms.

This patch also adds a few unit tests for the relevant
caching specific behavior.

BUG=664247,629609
TBR=tobiasjs@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2494123002
Cr-Commit-Position: refs/heads/master@{#434710}
(cherry picked from commit cb1e0169211135db1dd2e0220434bcec2efd7f16)
==========

[timvolodine, 2016-12-02 16:03:20]

timvolodine@chromium.org changed reviewers:
+ tobiasjs@chromium.org

[timvolodine, 2016-12-02 16:04:11]

merge to M56 of https://codereview.chromium.org/2494123002/

[timvolodine, 2016-12-02 16:04:20]

The CQ bit was checked by timvolodine@chromium.org

[commit-bot: I haz the power, 2016-12-02 16:04:55]

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, 2016-12-02 16:09:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480694660947050, "parent_rev":
"eb97d8902df9ae7c4de18700ce905f351588becc", "commit_rev":
"fb06655310dd648d9e3cbd62824d01064191a7c2"}

[commit-bot: I haz the power, 2016-12-02 16:09:31]

Description was changed from

==========
Spellcheck : Fix caching in cases where text is deleted.

Merge to M56 (branch 2924)

This patch fixes spellcheck caching in the following case:
1. paragraph is edited by removing text from the back and
2. paragraph does not contain misspelled words.

Currently this case results in a cache miss and consequently
spellcheck requests being sent to the spellchecking service
while completely unnecessary. This fix applies to all
platforms.

This patch also adds a few unit tests for the relevant
caching specific behavior.

BUG=664247,629609
TBR=tobiasjs@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2494123002
Cr-Commit-Position: refs/heads/master@{#434710}
(cherry picked from commit cb1e0169211135db1dd2e0220434bcec2efd7f16)
==========

to

==========
Spellcheck : Fix caching in cases where text is deleted.

Merge to M56 (branch 2924)

This patch fixes spellcheck caching in the following case:
1. paragraph is edited by removing text from the back and
2. paragraph does not contain misspelled words.

Currently this case results in a cache miss and consequently
spellcheck requests being sent to the spellchecking service
while completely unnecessary. This fix applies to all
platforms.

This patch also adds a few unit tests for the relevant
caching specific behavior.

BUG=664247,629609
TBR=tobiasjs@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2494123002
Cr-Commit-Position: refs/heads/master@{#434710}
(cherry picked from commit cb1e0169211135db1dd2e0220434bcec2efd7f16)
==========

[commit-bot: I haz the power, 2016-12-02 16:09:33]

Committed patchset #1 (id:1)