835591 - Disable cross-process IME when it's not used

Status: RESOLVED FIXED

(Core :: Widget, defect)

Description

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

We added cross-process IME support for xul-fennec, but no one uses it anymore.  The implementation itself isn't expensive, but it requires several sync IPC's that are happening in bad places during startup and causing O(10ms) hangs.

Since nobody is using this code atm, let's disable it and worry about optimizing this again when we need it for cross-process desktop.

Comment 1

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Attachment: Disable cross-process IME when it's not used

Comment 2

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Attachment: Rebased on inbound

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6cefdfbf3abd

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6cefdfbf3abd

Comment 5

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 707430 [details] [diff] [review]
Rebased on inbound

Safe patch that has content processes not do unnecessary work during early startup.

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 707430 [details] [diff] [review]
Rebased on inbound

Please go ahead with uplift to the tip of mozilla-b2g18 branch for v1.0.1

Comment 7

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/6d49867cede3

Comment 8

[Lukas Blakk [:lsblakk] use ?needinfo]

Batch edit: Bugs fixed on b2g18 after 1/25 merge to v1.0 branch are fixed on v1.0.1 branch.