Makoto Kato [:m_kato] Assignee
	Description • 11 years ago

When turning on e10s, IME on content doesn't works on Windows nightly build.  Of course, it works on chrome (ex. address bar)

GetDesiredIMEState() always returns IMEState::DISABLE on e10s, so nsIMEStateManager::GetNewIMEState turns off IME state.

nsIMEStateManaber should query content process for IME state.

	Makoto Kato [:m_kato] Assignee
	Comment 1 • 11 years ago

Also, before IME_START_COMPOSITION, NS_QUERY_SELECTED_TEXT is called.  So when calling it, GetIMETabParent is still null.

	Makoto Kato [:m_kato] Assignee
	Comment 3 • 11 years ago

IPDL generates default constructor for IPC.  So we should remove MOZ_CRASH for it.

	Makoto Kato [:m_kato] Assignee
	Comment 4 • 11 years ago

TODO:
IME requires NS_QUERY_TEXT_RECT and NS_QUERY_CARET_RECT for content process (TabParent)

	Makoto Kato [:m_kato] Assignee
	Comment 5 • 11 years ago

Comment on attachment 818171 [details] [diff] [review]
always use CROSS_IME

Always use MOZ_CROSS_PROCESS_IME.

	Makoto Kato [:m_kato] Assignee
	Comment 6 • 11 years ago

After this, user can input text via IME.  But candidate window may be wrong position.   Bug 935821 will fix this issue.

	Chris Peterson [:cpeterson]
	Comment 7 • 11 years ago

Comment on attachment 818171 [details] [diff] [review]
always use CROSS_IME

Bill, can you please Makoto's IME changes?

	Bill McCloskey [inactive unless it's an emergency] (:billm)
	Comment 8 • 11 years ago

Comment on attachment 818171 [details] [diff] [review]
always use CROSS_IME

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

Thanks very much for working on this! It looks like bug 835591 disabled this stuff because it makes b2g startup slower. So I think it would be better to put this at the top of the file instead:

#ifndef MOZ_B2G
#define MOZ_CROSS_PROCESS_IME
#endif

Also, I think :roc would be a good reviewer for this code in the future.

	Makoto Kato [:m_kato] Assignee
	Comment 9 • 11 years ago

MOZ_CROSS_PROCESS_IME is required for e10s IME

But, comment #8 by billm, defined MOZ_CROSS_PROCESS_IME is performance problem on B2G, so MOZ_CROSS_PROCESS_IME is only defined on non-B2G build

	Makoto Kato [:m_kato] Assignee
	Comment 10 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/5c1168eb4503

	Ryan VanderMeulen [:RyanVM]
	Comment 11 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/5c1168eb4503