New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 889350 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 27
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
M70
M71

Blocking:
issue 888402



Sign in to add a comment

Sets the correct context to decoder.

Project Member Reported by wuyingbing@chromium.org, Sep 26

Issue description

Chrome Version: (copy from chrome://version)
OS: M70, M71

What steps will reproduce the problem?
(1) Open VK, copy "i am the very long context, 0123456789, 9876543210." text into input box.
(2) Click any key on VK. Watch context from log.


What is the expected result?
The context should be "ntext, 0123456789, 9876543210."

What happens instead?
It's "i am the very long context, 01".


The root cause it calculate the context from the beginning.
Should calculate the context backward 30 characters from the focus.




 
Blocking: 888402
Status: Fixed (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 28

Labels: Merge-Request-70
Project Member

Comment 5 by sheriffbot@chromium.org, Sep 29

Labels: -Merge-Request-70 Merge-Review-70 Hotlist-Merge-Review
This bug requires manual review: M70 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Please mark which OS's this is impacting.
Labels: OS-Chrome
Labels: -Merge-Review-70 Merge-Approved-70
Cc: shend@chromium.org wuyingbing@chromium.org
 Issue 893018  has been merged into this issue.
Project Member

Comment 10 by sheriffbot@chromium.org, Oct 9

Cc: geo...@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-70

Sign in to add a comment