Disable cross-process IME when it's not used

RESOLVED FIXED in mozilla21

Status

()

defect
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: cjones, Assigned: cjones)

Tracking

unspecified
mozilla21
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(b2g1819+ fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 fixed)

Details

Attachments

(2 attachments)

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: 7 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.