Misspelled words are not marked after consecutive InsertText commands |
||||||
Issue descriptionVersion: ToT (54.0.2823.0) OS: All What steps will reproduce the problem? (1) Create a content editable <div> (2) Insert "zz apple" into the <div> by using 8 consecutive InsertText commands, each of which inserts a single character The attached test.html does the above steps. What is the expected output? "zz" should be marked as misspelling What do you see instead? No misspelling marker appears Please use labels and text to provide additional information.
,
Aug 8 2016
,
Aug 9 2016
I'm not going forward on this issue. I tried to fix it with https://codereview.chromium.org/2221783004, but it fails WebFrameTest.SlowSpellcheckMarkerPosition. The unit test was introduced in rouslan@'s fix to issue 179639 . However, the issue still seems to be there :( The following page partially marks "hewellco", which repros in stable and ToT <!doctype html> <div id="div" contenteditable></div> <script> var div = document.getElementById('div'); div.focus(); document.execCommand('InsertText', false, 'wellcome '); window.getSelection().collapse(div, 0); document.execCommand('InsertText', false, 'he'); </script> Besides, r350944 removed one line "webInputElement.setSelectionRange(0, 0);" from the unit test. It seems to be changing the logic of the test.
,
Aug 9 2016
I've seen partial markings myself as well. Not sure how to fix and have not investigated.
,
Oct 5 2016
,
Feb 24 2017
This issue is due to the current way that spell checker gets triggered, and will be fixed by idle time spell checker.
,
Sep 6 2017
Still reproduces with idle time spell checker, as the second InsertText command prevents spellchecker from being invoked. I don't see an easy way of fixing this without regressing performance. Anyway, this issue seems rather low-priority and artificial, so I'm closing it. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by xiaoche...@chromium.org
, Aug 8 2016