11845 - [dogfood] [key]Cannot switch keyboard layout. cannot type international characters

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P1)

Description

[Frank Tang]

Reproduce procedure-
1. install new keyboard layout by using "Keyboard" control panel
2. a keyboard menu will appeared in the right bottom corner of the screen in
Win95/98/NT
3. Launch AppRunner.
4. Switch keyboard to another one under browser or editro
5. AppRunner does not allow you to switch.

We need this to support multilingual input.

Comment 1

[tague]

Frank :- i can't reproduce this on NT using german, english or Japanese.  Can
you provide me more information about your system and what keyboards you were
trying to use.  I'll take a look at it once I get that information.

Comment 2

[Teruko Kobayashi]

Changed QA Contact to [e-mail] since he tested this before.

Comment 3

[Frank Tang]

It happened in my downloaded rel build. I can switch in my debug build but then
the input is wrong. Please try it again and mark it WORKFORME if both debug and
rel build are ok.

Comment 4

[tague]

setting milestone to m11, since it is "only" appearing in release builds.

Comment 5

[Frank Tang]

reassign this to ftang. Tague is doomed.

Comment 6

[Frank Tang]

maybe dup of 14058. Mark it as M11 assign for now. Add rods/tague/buster to the
cc list.

Comment 7

[Frank Tang]

*** Bug 14058 has been marked as a duplicate of this bug. ***

Comment 8

[Frank Tang]

more info cut&paste from mail send to tague 9/11
Now  you are using ACP as the initial value for the keyboard input CP. As I
mention eariler, you should use another value
for the init value. Here is more info

Call    GetKeyboardLayout  ( 0 ). It will return a HKL to you. The lower word of
HKL is a  Language Identifiers

You know the rest, you alrady hae the Language identifier to CP conversion in
the WM_INPUTLANGUAGE sectoin of
nsWindow::ProcessMessage in mozilla/widget/src/window/nsWindow.cpp

Comment 9

[Frank Tang]

The cut&paste lost HREF
HREF for GetKeyboardLayout -
http://msdn.microsoft.com/library/sdkdoc/winui/keybinpt_5sxg.htm
HREF for Language identifier -
http://msdn.microsoft.com/library/sdkdoc/winbase/nls_8xo3.htm

Comment 10

[Frank Tang]

I put in some tracing code and find out
1. all the nsWindow are init with mCurrentKeyboardCP == 0 (which is wrong and
should be 932 on my Japanese NT). The reason it is wrong is because we simply
set the mCurrentKeyboardCP to CP_ACP in the nsWindow::nsWindow. We should use
the GetKeyboardLayout code there find out the correct keyboard CP
2. the window which got the WM_INPUTLANGREQUEST/WM_INPUTLANGCHANGE are not the
same one which got the nsWindow::onChar. Could be a foucs or event propogation
issue.

Comment 11

[leger]

Putting on [PDT+] radar.

Comment 12

[Frank Tang]

fix in keyEvent_19991004_BRANCH landing

Comment 13

[Teruko Kobayashi]

I tested this in 10-15-08 Win32 build. It works fine.
I tested Dutch, German, French, Iceland, Swedish, Spanish, and Portugues.
I mark this as verified.

Comment 14

[Teruko Kobayashi]

I tested this in 10-20-08 Win32 build.  In the URL bar and forms, I cannot type
the accended characters. I tested French keyboard. Same as 14387 and 14388.

Comment 15

[Teruko Kobayashi]

The problem I found was not Keyboard switching problem.  It was rendering
problem.  I logged the different bug about it to 16986.  I will mark this
as fixed.

Comment 16

[Teruko Kobayashi]

I verified this in 10-20-08 build.