X11: Send IME update notification deferred

[HolonProduction]

Fixes #82338
Fixes #82608

I suggest only merging this once the ibus issue is resolved. The error message is not directly linked to the problem with accented characters not inserting, but it will help users who encounter the problem to find the related issue on Github.

[adamscott]

It should work! I'm approving this PR, but @bruvzg should have the last word on it.

[akien-mga]

I suggest only merging this once the ibus issue is resolved. The error message is not directly linked to the problem with accented characters not inserting, but it will help users who encounter the problem to find the related issue on Github.

Isn't this PR meant to workaround the ibus issue? If so it seems good to me to merge this now and cherry-pick for 4.2.1, as we don't know how long it will take for ibus to fix its regression and for Linux distros to pick up the fix.

[adamscott]

Isn't this PR meant to workaround the ibus issue?

No, it doesn't fix the IBus issue. It fixes the fact that the event is coming from a state where it's not thread-safe for nodes.

[akien-mga]

I'm still confused, it claims to fix the issues which were pinpointed as caused by the ibus regression.

[adamscott]

Well, it fixes the ERROR: Caller thread can't call this function in this node (/root). Use call_deferred() or call_thread_group() instead. part.

[bruvzg]

If I understand correctly, this will fix crash with IBus, but not missing dead keys (which is IBus bug).

[akien-mga]

Thanks!

[YuriSizov]

Cherry-picked for 4.2.1.