Closed Bug 1444572 Opened 2 years ago Closed 2 years ago

[GTK][uim] Dispatch fake "keydown" and "keyup" events if active IM uses key snooper and send some preedit signals to us

Categories

(Core :: Widget: Gtk, enhancement)

All
Linux
enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox60 --- wontfix
firefox61 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

(Keywords: inputmethod)

Attachments

(1 file)

At least UIM which is really old IM but still is default IM of Debian 9.x, IM may use key snooper to handle key events *before* applications. In this case, we won't receive native key events if IME consumes key events for composition.  Therefore, if we detect key snooper mode of each IM, we should dispatch fake keydown and keyup events whose keyCode value is DOM_VK_PROCESSKEY, key value is KEY_NAME_INDEX_Process and code value is "unidentified".
FYI: As far as I tested, scim doesn't use key snooper on Debian. So, without any hack, I see keydown and keyup events as expected.
Comment on attachment 8958066 [details]
Bug 1444572 - IMContextWrapper should dispatch fake eKeyDown event during composition if active IM is uim

https://reviewboard.mozilla.org/r/227014/#review233368
Attachment #8958066 - Flags: review?(m_kato) → review+
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/e226de7caa88
IMContextWrapper should dispatch fake eKeyDown event during composition if active IM is uim r=m_kato
Backed out changeset e226de7caa88 (bug 1444572) for conflicts while backing out 1443421

https://hg.mozilla.org/integration/autoland/rev/3c176f583fe86a705f54053f9a6b6e161fb91734
Flags: needinfo?(masayuki)
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/b1826c93ea23
IMContextWrapper should dispatch fake eKeyDown event during composition if active IM is uim r=m_kato
Flags: needinfo?(masayuki)
https://hg.mozilla.org/mozilla-central/rev/b1826c93ea23
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.