Closed Bug 835591 Opened 9 years ago Closed 9 years ago

Disable cross-process IME when it's not used

Categories

(Core :: Widget, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla21
Tracking Status
b2g18 19+ fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- fixed

People

(Reporter: cjones, Assigned: cjones)

References

Details

Attachments

(2 files)

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.
Assignee: nobody → jones.chris.g
Attachment #707344 - Flags: review?(roc)
https://hg.mozilla.org/mozilla-central/rev/6cefdfbf3abd
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment on attachment 707430 [details] [diff] [review]
Rebased on inbound

Safe patch that has content processes not do unnecessary work during early startup.
Attachment #707430 - Flags: approval-mozilla-b2g18?
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
Attachment #707430 - Flags: approval-mozilla-b2g18? → approval-mozilla-b2g18+
Batch edit: Bugs fixed on b2g18 after 1/25 merge to v1.0 branch are fixed on v1.0.1 branch.
You need to log in before you can comment on or make changes to this bug.