616404 - send notification and hint states atomically with focus

Status: NEW

(Core :: IPC, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

see bug 615208 for some discussion of what could go wrong

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to comment #0)
> see bug 615208 for some discussion of what could go wrong
from cjones:

Also, I think Matt was onto a second bug lurking here.  I think that in
PuppetWidget::OnIMEFocusChange(), we need to send the result of
SendNotifyIMEFocus() atomically with the result of the immediately-after
OnIMETextChange(), which can end up sending TextHint and TextChange.  Otherwise
an IME event sent from chrome could race with those results and result in those
being discarded in the parent.  If that happened, we would see the same
symptoms as the above bug.  I'd like to fix that too, but I guess it doesn't
need to block b3 since it seems fairly improbable that it would happen in the
wild.  Followup suits me.

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Chris should this block?

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

I wouldn't at this point, since we're apparently not seeing this in the field.