Closed Bug 835591 Opened 12 years ago Closed 12 years ago

Disable cross-process IME when it's not used

Categories

(Core :: Widget, defect)

Product:
Core
Component:
Widget
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla21
Tracking Flags:
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)

Disable cross-process IME when it's not used
4.09 KB, patch
roc
: review+
Details | Diff | Splinter Review
Rebased on inbound
4.21 KB, patch
cjones
: review+
lsblakk
: approval-mozilla-b2g18+
Details | Diff | Splinter Review
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: 12 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.
status-b2g18-v1.0.1: --- → fixed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: