1179122 - [e10s] TextComposition should manage target in parent process even when its child process has focus

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(away: 6/25-6/28)]

Currently, EventStateManager sends composition events to TabParent. Therefore, TextComposition is now created only in the process which has composition actually.

But this causes a lot of bugs which are detected by |Assertion failure: aCompositionEvent->message == (2200)| since if TabParent becomes deactive during a composition, WidgetCompositionEvent is dispatched to different content in the parent.

For preventing this issue, TextComposition should guarantee that all composition events are sent to same TabParent.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(away: 6/25-6/28)]

Attachment: Patch

See comment 0 for the detail. Although, even with this patch, I sometimes see hitting the assertion. Sigh... (Of course, this actually fixes some cases of that, though.)

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6d96d03b8b20

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6d96d03b8b20