1147722 - Disable TSF on e10s mode until bug 960836 is fixed

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

[Makoto Kato [:m_kato]]

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.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: Turn off TSF when e10s auto start is enabled

Comment 2

[Makoto Kato [:m_kato]]

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.

Comment 3

[Makoto Kato [:m_kato]]

also, should I remove KeyboardMayHaveIME implementation from nsXULAppInfo?

Comment 4

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

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.

Comment 5

[Makoto Kato [:m_kato]]

Attachment: Part 2. Remove keyboardMayHaveIME method

follow up this issue.  no longer needed keyboardMayHaveIME per comment #4.

Comment 6

[Makoto Kato [:m_kato]]

 https://hg.mozilla.org/integration/mozilla-inbound/rev/e519d5609c33
 https://hg.mozilla.org/integration/mozilla-inbound/rev/30bf6cb70f61

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e519d5609c33
https://hg.mozilla.org/mozilla-central/rev/30bf6cb70f61