New issue
Advanced search Search tips

Issue 906511 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Nov 22
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

[M17n] Submitted character not correct for Arabic keyboard

Project Member Reported by zhan...@chromium.org, Nov 19

Issue description

Version: 72.0.3612.011276.0.0 dev-channel bob test
Google input tools version:72.0.20181114.221567824

Steps;
1.Switch to Arabic keyboard
2.Tap some keys ,or tapping shift + some keys

Actual:
Submitted character is not correct ,
see attached video

Note:
If disable enable-ime-service flash ,this issue not repro

 
Nov 19, 2018 2_54 PM (1).webm
212 KB View Download
Cc: shuchen@chromium.org
 Issue 906504  has been merged into this issue.
Status: Started (was: Assigned)
2 issues:
 - Backquote key handling.
 - All keys contains " or \ will fail to commit.

 Issue 906546  has been merged into this issue.
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 20

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

commit 41e5d958be36e93412509e557881d5118f9b5544
Author: Shu Chen <shuchen@google.com>
Date: Tue Nov 20 02:58:31 2018

Fix typo: BackQuote -> Backquote.

Bug:  906511 
Change-Id: I7ea2770072347eb37a0b9800a2ba0d17cecfe496
Reviewed-on: https://chromium-review.googlesource.com/c/1343577
Reviewed-by: Leo Zhang <googleo@chromium.org>
Commit-Queue: Shu Chen <shuchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609576}
[modify] https://crrev.com/41e5d958be36e93412509e557881d5118f9b5544/chromeos/services/ime/public/cpp/rulebased/rules_data.cc

Project Member

Comment 5 by bugdroid1@chromium.org, Nov 20

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

commit 6d58017cfabb68b46fd1a26390bab82875dbb1bf
Author: Shu Chen <shuchen@google.com>
Date: Tue Nov 20 03:30:01 2018

The response JSON string should be escaped.

Bug:  906511 
Change-Id: Ia8a1cd0ef0fabb9dfc3777f0758cd4470f75ac3f
Reviewed-on: https://chromium-review.googlesource.com/c/1343461
Reviewed-by: Leo Zhang <googleo@chromium.org>
Commit-Queue: Shu Chen <shuchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609583}
[modify] https://crrev.com/6d58017cfabb68b46fd1a26390bab82875dbb1bf/chromeos/services/ime/input_engine.cc

Status: Fixed (was: Started)
Please verify this bug on version 3617.0.0+.

3617.0.0+ has not come out due to Thanksgiving holidays. Please wait for 3617.0.0+ coming out and verify.

Status: Verified (was: Fixed)
Verified in 72.3618.0 , 11296.0.0 dev-channel kevin test 
Input tools version: 72.0.20181122.222570995

Sign in to add a comment