277795 - Dropdown lists are rolluped by MS-IME

Status: RESOLVED FIXED

(Core :: Internationalization, defect, P1)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

If text service is disabled and using MS-IME,
Mozilla's rollup window(e.g., menus, dropdown lists) are disapeare immediately.

I will make a patch.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch rv1.0

When the window got focus, MS-IME sent WM_GETMINMAXINFO message(only the text
service is turned off).
Currently, if we get WM_GETMINMAXINFO message, the popup window is rolled-up.
Therefore, we should ignore WM_GETMINMAXINFO on non top level window.
Because in normaly, the WM_GETMINMAXINFO is sent only top level window.
If we get WM_GETMINMAXINFO message on non top level window, the sender may be
MS-IME. So, the message does not need.

Comment 2

[Ere Maijala (slow)]

Comment on attachment 171118 [details] [diff] [review]
Patch rv1.0

Looks ok to me.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 171118 [details] [diff] [review]
Patch rv1.0

>+        // If non top level window, in normaly, we don't get WM_GETMINMAXINFO.
>+        // Therefore if non top level window gets this message, should ignore.
>+        // See https://bugzilla.mozilla.org/show_bug.cgi?id=277795

I'd say something more like:

  "Non-toplevel windows normally don't get WM_GETMINMAXINFO.
   Therefore if a non-toplevel window gets this message, we should
   ignore it."

sr=bzbarsky with the wording change.

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch for check-in

Ere:

Please check-in this patch.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch for check-in

Oops...

Sorry. Please check-in this patch.

Comment 6

[Ere Maijala (slow)]

Patch checked in.

Comment 7

[[e-mail]]

So before this was fixed, bug 165055 actually worked, because something was
killing the autocomplete popup?

Comment 8

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to comment #7)
> So before this was fixed, bug 165055 actually worked, because something was
> killing the autocomplete popup?

This bug does not relate bug 165055.
This bug is reproduced when the control got focus, immediately.
not when inputting.