1051228 - The keyboard bn-Avro in SPRD version cannot work well with patches of bug977026

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: General, defect, P1)

Description

[helloarvin]

Attachment: bn-Avro-IME-comparision.zip

The keyboard bn-Avro in SPRD version cannot work well as the attached video shown. After testing, it seems the issue is introduced by the patches of bug977026 and we can see it works fine shown in '-without-patches-.3gp'.

It looks like that there's always an excrescent copy of current buffer(typed contents) inserted before the new input. Just like the Space typed, the actual content shown is 'previous contents'+' ' rather than ' '.

Confusingly, the weird problem only happened on the keyboard bn-Avro. There's too much modifications involed so that I cannot follow it.

Could you please help to check the issue?

Thanks a lot.

[1] https://bug977026.bugzilla.mozilla.org/attachment.cgi?id=8420000
[2] https://bug977026.bugzilla.mozilla.org/attachment.cgi?id=8420001
[3] https://bug977026.bugzilla.mozilla.org/attachment.cgi?id=8420003

Comment 1

[helloarvin]

Hi Thinker,

Could you please help to check the issue?
BTW, I'd like to ask you help invite tim because it's very hard for me to know which tim is the right person we'r lokking for :)

Thank you all.

Comment 2

[Thinker Li [:sinker]]

Tim, could you find some one to look into this bug from gaia?

Comment 3

[Tim Guan-tin Chien [:timdream] (please needinfo)]

I will put my money on IPC etc. We could confirm that by annotating forms.js and MozKeyboard.js.

Thinker, which team this work falls under?

Comment 4

[Thinker Li [:sinker]]

Patrick, could you take a look at this bug?

Comment 5

[Patrick Wang (Chih-Kai Wang) (:kk1fff)]

It looks chrome sends correct data to content, but child put the text directly into text box, instead of doing composition. The video clips show the same thing: in the video clip which demos expected behavior, there's a line under the compositing text, while in the the other video clip, there's not.

Comment 6

[Patrick Wang (Chih-Kai Wang) (:kk1fff)]

The input field doesn't get composition* events as well. The problem might be related to event dispatching. Will look into it.

Comment 7

[Patrick Wang (Chih-Kai Wang) (:kk1fff)]

The composition events are ignored by PuppetWidget::DispatchEvent() because of the their sequence numbers are less than blur sequence number which is held by PuppetWidget. However, the sequence numbers of composition events are not initialized, their values are acutally junk. This bug actually is not related to b2g loader and may occur randomly. There is a patch made for central in bug 946068, we should uplift it.

Comment 8

[Andreas Pehrson [:pehrsons]]

Patrick, see comments on bug 946068. It is already in v1.4.

Comment 9

[Patrick Wang (Chih-Kai Wang) (:kk1fff)]

Patches mentioned in comment 0 are in 1.3t, where bug 946068 is not applied.

Comment 10

[Andreas Pehrson [:pehrsons]]

Oh. Thanks. I must have been confused. Should perhaps be more clear with what versions a bug applies to in the future.

Sorry for taking your time!

Comment 11

[BMO Automation]

Firefox OS is not being worked on