Closed Bug 1147722 Opened 9 years ago Closed 9 years ago

Disable TSF on e10s mode until bug 960836 is fixed

Categories

(Core :: Widget: Win32, defect)

x86
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
e10s m6+ ---
firefox39 --- fixed

People

(Reporter: m_kato, Assigned: m_kato)

References

Details

Attachments

(2 files)

All e10s M6 blocker bugs depend on TSF.   We should turn off TSF on e10s mode now, then all dependency bugs of bug 960836 are fixed, turn on TSF again.
Blocks: e10s-ime-tsf
Comment on attachment 8583675 [details] [diff] [review]
Turn off TSF when e10s auto start is enabled

I don't think that is is necessary to add new preferences for e10s TSF.

When e10s auto start is turned on, TSF will be disabled as default.
But if intl.tsf.force_enable = true, you can turn on TSF even if e10s auto start.
Attachment #8583675 - Flags: review?(wmccloskey)
Attachment #8583675 - Flags: review?(masayuki)
also, should I remove KeyboardMayHaveIME implementation from nsXULAppInfo?
Attachment #8583675 - Flags: review?(masayuki) → review+
Comment on attachment 8583675 [details] [diff] [review]
Turn off TSF when e10s auto start is enabled

Review of attachment 8583675 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks. And please do remove KeyboardMayHaveIME.
Attachment #8583675 - Flags: review?(wmccloskey) → review+
follow up this issue.  no longer needed keyboardMayHaveIME per comment #4.
Attachment #8584303 - Flags: review?(wmccloskey)
Attachment #8584303 - Flags: review?(wmccloskey) → review+
https://hg.mozilla.org/mozilla-central/rev/e519d5609c33
https://hg.mozilla.org/mozilla-central/rev/30bf6cb70f61
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Depends on: 1212080
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: