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

Issue 602746 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

CJK and Complex script languages don't work after using them on Incognito window

Project Member Reported by songsuk@chromium.org, Apr 12 2016

Issue description

Chrome Version       : 	51.0.2704.0 
Platform             :  8172.1.0 - Daisy, Candy


What steps will reproduce the problem?
(1)  set IME to Korean
(2)  enter "gksrmf", 한글, on Omnibox
(3)  open Incognito window, and enter "gksrmf", 한글, on omnibox
(4)  close the incognito window
(5)  enter "gksrmf", 한글, on Omnibox


What is the expected result?  
What happens instead?
On step#5, nothing happens. unable to enter Korean. IME doesn't work.


Please provide any additional information below. Attach a screenshot if
possible.
Able to use the IME after resigning to the User.
 

Comment 1 Deleted

Chrome Canary  52.0.2705.0 canary (64-bit) which is the latest ver also cannot support Japanese on Gmail, Calendar and Drive. It might have some problem on multi-byte input.
We expect that this issue won't be appeared on Stable Version and will be fixed on Canary ASAP. 

Cc: xiangye@chromium.org shuchen@chromium.org
Owner: wuyingbing@chromium.org
Status: Assigned (was: Untriaged)
@shunka.kyo@g.softbank.co.jp, 

I'm trying to reproduce the issue. Could you provide your device OS info.
Owner: shuchen@chromium.org
Hi shu,
It's very severe bug. It's kind of IMF issue.
After 1~4 step, then extension can't receive key event and commit text.
And all system shortcut disabled.

Not only Korean but all Input tools
Owner: azurewei@chromium.org
Azure, this seems related to the ime event router refactoring.

Notice that when an Incognito is opened, a new IME extension instance is created/loaded.
However, they may share the "same" profile: the Incognito profile is the off-the-record profile of the main profile.

So the router map in the event router factory may need to consider that case.

Project Member

Comment 8 by bugdroid1@chromium.org, Apr 20 2016

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

commit 13814091e327ebc1792dfb329b88d2289568a78a
Author: azurewei <azurewei@chromium.org>
Date: Wed Apr 20 05:39:06 2016

Instead of using ProfileCompare in InputImeEventRouterFactory::router_map_ when trying to remove profile, just compares the profile directly.

BUG= 602746 
TEST=Verfied not repo on local build.

Review URL: https://codereview.chromium.org/1896993003

Cr-Commit-Position: refs/heads/master@{#388431}

[modify] https://crrev.com/13814091e327ebc1792dfb329b88d2289568a78a/chrome/browser/extensions/api/input_ime/input_ime_api.cc

Status: Fixed (was: Assigned)
Status: Started (was: Fixed)
Labels: Merge-Request-51

Comment 12 by tin...@google.com, Apr 21 2016

Labels: -Merge-Request-51 Merge-Approved-51 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M51 (branch: 2704)
Project Member

Comment 13 by bugdroid1@chromium.org, Apr 21 2016

Labels: -merge-approved-51 merge-merged-2704
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5387cfbe0137772ae3b263144e4cd8f735baf9f1

commit 5387cfbe0137772ae3b263144e4cd8f735baf9f1
Author: Shu Chen <shuchen@google.com>
Date: Thu Apr 21 05:05:42 2016

Instead of using ProfileCompare in InputImeEventRouterFactory::router_map_ when trying to remove profile, just compares the profile directly.

BUG= 602746 
TEST=Verfied not repo on local build.

Review URL: https://codereview.chromium.org/1896993003

Cr-Commit-Position: refs/heads/master@{#388431}
(cherry picked from commit 13814091e327ebc1792dfb329b88d2289568a78a)

Review URL: https://codereview.chromium.org/1912503003 .

Cr-Commit-Position: refs/branch-heads/2704@{#160}
Cr-Branched-From: 6e53600def8f60d8c632fadc70d7c1939ccea347-refs/heads/master@{#386251}

[modify] https://crrev.com/5387cfbe0137772ae3b263144e4cd8f735baf9f1/chrome/browser/extensions/api/input_ime/input_ime_api.cc

Status: Fixed (was: Started)
Labels: VerifyIn-54

Comment 16 by ka...@chromium.org, Aug 31 2016

Labels: Bulk-Verified
Status: Verified (was: Fixed)

Sign in to add a comment