1018035 - [Flame][v1.4][SMS]No cursor display when tap the message edit box.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[greatwall2686]

* Description:
  No cursor display when tap the message edit box.

* Reproduction steps:
  1. Tap message icon to launch
  2. Tap "+" to compose a message
  3. Tap edit box
 
  

* Expected result:
  The cursor should be display in edit box.

* Actual result:
  There is no cursor display.

* Reproduction build:(Buri - Firefox OS V1.4 inside)
 - Gaia      17b102ee8d9a724b62285547cc5f1c5d30bfb01c
 - Gecko     https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/95be84248033
 - BuildID   20140520000201
 - Version   30.0

* Reproduction Frequency
  - 100%

Comment 1

[greatwall2686]

Attachment: 2014-05-30-17-43-31.rar

update steps:
tap an call hostory to send SMS, tap the edit box, the cursor cannot display.

Comment 2

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

In bug 1014420 comment 1, Noemi could not reproduce in v1.4. Can we try with a newer build?

Comment 3

[Eric Kramer [:Kramer]]

This issue DOES repro on a newer Flame 1.4 build.

Environmental Variables:
Device: Flame 1.4
Build ID: 20140609000201
Gaia: 8b239e41bbd85aa7b6a2c5d388e775ba7de6fb2b
Gecko: e3f85877db29
Version: 30.0 (1.4) 
Firmware Version: v10G-2

Comment 4

[Jason Smith [:jsmith]]

Does this happen on 1.3?

Comment 5

[Eric Kramer [:Kramer]]

This issue does NOT repro on Flame 1.3

Environmental Variables:
Device: Flame 1.3
Build ID: 20140520094859
Gaia: a73235d23685e9898f40647cebd83b3fcbfd0117
Gecko: b637b0677e15318dcce703f0358b397e09b018af
Version: 28.0 (1.3) 
Firmware Version: v10G-2

Comment 6

[Jason Smith [:jsmith]]

Seems like something basic to an input field that's not working here.

Comment 7

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Actually, the composer's input is made of contenteditable div. And contenteditable stuff have lots of subtle issues that need work arounds :)

Waiting on the regression window.

Comment 8

[Preeti Raghunath(:Preeti)]

Seems like a basic use case isn't working. Bad UX too hence blocking for 1.4

Comment 9

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

So, I don't reproduce on any device. I tried Flame master (latest), Flame v1.4 (latest), Peak v1.4 (3 weeks old), Hamachi master (recent), Dolphin master (recent), Dolphin v1.4 (latest).

Here is the version I used to flash Flame v1.4:
Gaia      d1cf95dc5e8b2f52148487291318542f1396608e
Gecko     https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/a8bb6b76696b
BuildID   20140611000202
Version   30.0
ro.build.version.incremental=eng.vagrant.20140528.073244
ro.build.date=Wed May 28 07:33:48 UTC 2014

I don't know the Firmware version though.

Can QA double check on the newest firmware for Flame too? Can I have a video of the bug happening?

Thanks

Comment 10

[Chris Kreinbring [:CKreinbring]]

Able to repro on today's Flame 2.0.  Youtube clip has been added.

Build ID: 20140611091244
Gecko: https://hg.mozilla.org/releases/mozilla-aurora/rev/0c0effd600c4
Gaia: a0f9f1f41a436daad8a249ce85df80a81a5ba2d5
Platform Version: 32.0a2
Firmware Version: V10g-2
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Comment 11

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

QA: thanks, but the video doesn't do what the description in comment 0 is doing: simply launching the Messages app. It shows the issue reproduce with STR in comment 1 though which is a good information (and I admit I didn't try this).

Can you try again with the description in comment 0 please?

Comment 12

[Chris Kreinbring [:CKreinbring]]

Attempting with just the steps in comment 0, I was unable to repro with today's Flame 2.0 and 1.4.  The cursor appeared when the Messages textbox was tapped.

BuildID: 20140611000202
Gaia: d1cf95dc5e8b2f52148487291318542f1396608e
Gecko: a8bb6b76696b
Version: 30.0
Firmware Version: V10G-2
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0

Comment 13

[Jason Smith [:jsmith]]

Please make sure you flag Joshua down when QA requests are finished.

Comment 14

[Chris Kreinbring [:CKreinbring]]

Regression window:
Last Working
Build ID: 20140510183010
Gecko: https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/ec24f847e7c0
Gaia: 17fb44880e95bc7ae363a609d811bf5a9a067b5b
Platform Version: 30.0
Firmware Version: V10g-2

First Broken
Build ID: 20140512123005
Gecko: https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/cc1987fe9ee4
Gaia: 69bd6293c3e138258257af37a33765997e49bd6e
Platform Version: 30.0
Firmware Version: V10g-2

Last Working Gaia / First Broken Gecko - OK
Gaia: 17fb44880e95bc7ae363a609d811bf5a9a067b5b
Gecko: https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/cc1987fe9ee4

First Broken Gaia / Last Working Gecko - Fail
Gaia: 69bd6293c3e138258257af37a33765997e49bd6e
Gecko: https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/ec24f847e7c0

Gaia push log: https://github.com/mozilla-b2g/gaia/compare/17fb44880e95bc7ae363a609d811bf5a9a067b5b...69bd6293c3e138258257af37a33765997e49bd6e

Comment 15

[Joshua Mitchell (Inactive)]

Regression window provided -  The initial window had a VERY small push-log so I did not have the tester proceed to a secondary window.

Comment 16

[Jason Smith [:jsmith]]

Broken by bug 1003384.

Comment 17

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

I highly doubt that bug 1003384 broke this.
Gonna investigate on my side, with the STR in comment 1.

Comment 18

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

I did a bisect from the pushlog, and the failing patch is bug 999286.

And actually, I reproduce on a current v1.3 (on a Inari). I don't know why we don't see it in comment 5, my guess is that the tester used the STR from comment 0.

Comment 19

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Attachment: github PR for master

Still need to update unit tests.

Comment 20

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Attachment: github PR for v1.4

Comment 21

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Comment on attachment 8439400 [details] [review]
github PR for master

I couldn't reproduce on a simpler testcase to report the bug to gecko, so here is only the workaround because sometimes we just need to move forward.

There still is an issue on master: when it's an activity, the iframe itself has not the focus and as a result we don't have the focus in the activity. I don't know yet if it works in v1.4. I'll file a separate bug for this.

Comment 22

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Ok, the bug is not in v1.4 :)

Comment 23

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Filed bug 1025169.

Comment 24

[Oleg Zasypkin [:azasypkin]]

Comment on attachment 8439400 [details] [review]
github PR for master

Looks good to me, just left tiny nit at github, thanks! But travis is red for some reason.

Comment 25

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Updated both pull requests, waiting for Travis.

Comment 26

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

v1.4: 4b3428e85b428e577a0f96cba6121c4ca1c91f8a
master: 23245b524afd918c6110d675d8b175556368179e

Comment 27

[Ryan VanderMeulen [:RyanVM]]

(In reply to Julien Wajsberg [:julienw] from comment #26)
> v1.4: 4b3428e85b428e577a0f96cba6121c4ca1c91f8a
> master: 23245b524afd918c6110d675d8b175556368179e

Missing a v2.0 uplift there?

Comment 28

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

Yep Ryan, I was expecting you ;)

Comment 29

[Ryan VanderMeulen [:RyanVM]]

If you're already pushing to v1.4 at the same time, might as well take care of v2.0 while you're at it, IMO. BTW, we generally recommend that you *not* push to release branches at the same time as master, fwiw. Makes backouts a bigger PITA should one become necessary.

Comment 30

[Julien Wajsberg [:julienw] (PTO -> Jul 24)]

I pushed to v1.4 because a v1.4-specific patch was needed.

NI me to push to v2.0.

Comment 31

[Ryan VanderMeulen [:RyanVM]]

v2.0: https://github.com/mozilla-b2g/gaia/commit/46a17fd576d29272ffdab110701cef8a3ef644c3