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

Issue 117758 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Email to this user bounced
Closed: Mar 2012
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

IME failure on specific flows with a windowless Flash

Project Member Reported by kenjibaheux@chromium.org, Mar 12 2012

Issue description

Chrome Version       : 17, 18, 19
OS Version: 6.1 (Windows 7, Windows Server 2008 R2) probably other Windows version too
URLs (if applicable) : http://baseonmars.co.uk/bugs/wmode/

What steps will reproduce the problem?
1. Make sure you have the default NPAPI Flash plugin enabled
2. Search for baseonmars wmode bugs (Confirmed on Google and Yahoo)
3. Click on the link to http://baseonmars.co.uk/bugs/wmode/ 

What is the expected result?
4. You can use your IME on wmode=opaque and wmode=transparent embeds


What happens instead?
4. The IME is disabled on wmode=opaque and wmode=transparent embeds





 
Cc: takuya@chromium.org
Owner: hbono@chromium.org
Follow up investigation indicates that these flows fail because they exercise previously believed to be unused IME functions.

The fix is believed to be easy and low risk.

Comment 2 by hbono@chromium.org, Mar 12 2012

Status: Started
Greetings,

Thanks for your bug report.
It seems Flash calls the ImmSetAssociateContextEx function to re-activate an IME after WebKit deactivates it. Unfortunately, Flash cannot activate the IME with this function call just because my r103869 <http://crrev.com/103869> does not emulate this function. I have started implementing this function to emulate this function, i.e. fix this issue.

Regards,

Hironori Bono
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 13 2012

Summary: IME failure on specific flows with a windowless Flash
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=126347

------------------------------------------------------------------------
r126347 | hbono@chromium.org | Tue Mar 13 00:03:47 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/plugins/npapi/webplugin_ime_win.cc?r1=126347&r2=126346&pathrev=126347
 M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/plugins/npapi/webplugin_delegate_impl_win.cc?r1=126347&r2=126346&pathrev=126347
 M http://src.chromium.org/viewvc/chrome/trunk/src/content/plugin/webplugin_delegate_stub.cc?r1=126347&r2=126346&pathrev=126347

Emulate ImmSetAssociateContextEx() for Flash.
This change emulates ImmSetAssociateContextEx() calls from Flash so Flash can activate or deactivate IMEs. This change also restores this activation status when a plug-in element get the event focus.

BUG= 117758 
TEST=manual

Review URL: http://codereview.chromium.org/9666050
------------------------------------------------------------------------
Labels: ReleaseBlock-Stable Merge-Requested
Status: Verified
Verified on Canary 19.0.1068.0

Requesting merge to M18: fairly low risk CL and the issue affects all CJK users quite frequently in the normal course of browsing the web (the flows mentioned are extremely common and probably non-exhaustive).

Comment 5 by kareng@google.com, Mar 23 2012

Labels: -Merge-Requested Merge-Merged
landed this on m18
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 23 2012

Labels: merge-merged-1025
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=128577

------------------------------------------------------------------------
r128577 | karen@chromium.org | Fri Mar 23 14:59:00 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1025/src/webkit/plugins/npapi/webplugin_ime_win.cc?r1=128577&r2=128576&pathrev=128577
 M http://src.chromium.org/viewvc/chrome/branches/1025/src/webkit/plugins/npapi/webplugin_delegate_impl_win.cc?r1=128577&r2=128576&pathrev=128577
 M http://src.chromium.org/viewvc/chrome/branches/1025/src/content/plugin/webplugin_delegate_stub.cc?r1=128577&r2=128576&pathrev=128577

Merge 126347 - Emulate ImmSetAssociateContextEx() for Flash.
This change emulates ImmSetAssociateContextEx() calls from Flash so Flash can activate or deactivate IMEs. This change also restores this activation status when a plug-in element get the event focus.

BUG= 117758 
TEST=manual

Review URL: http://codereview.chromium.org/9666050

TBR=hbono@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9838080
------------------------------------------------------------------------
Verified as Fixed with 18.0.1025.140 (Official Build 128625) on Windows 7.
Working fine on Chrome OS also.
Project Member

Comment 9 by bugdroid1@chromium.org, Oct 13 2012

Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 10 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Area-UI -Mstone-18 -Feature-Flash -Feature-I18N Cr-Content-Plugins-Flash Cr-UI M-18 Cr-UI-I18N
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 13 2013

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Project Member

Comment 12 by bugdroid1@chromium.org, Mar 20 2013

Labels: -Cr-UI-I18N Cr-UI-Internationalization
Project Member

Comment 13 by bugdroid1@chromium.org, Apr 6 2013

Labels: Cr-Blink
Project Member

Comment 14 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content-Plugins-Flash Cr-Internals-Plugins-Flash

Sign in to add a comment