1107446 - [Flame][Music]Device will jump to other page.

Status: RESOLVED WONTFIX

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

Description

[Lance（leave from Mozilla）]

Attachment: 1603.MP4

[1.Description]:
[Flame][v2.1][Music]Device will jump to other page when you slide down on songs page.
Found time:16:03
See attachment:1603.mp4 and logcat_1603.txt

[2.Testing Steps]: 
Premise:Device have more than 40 songs.
1.Launch Music.
2.Switch to songs page.
3.Slide down page quickly.

[3.Expected Result]: 
3.The page should not jump to other, it should keep up songs page.

[4.Actual Result]: 
3.The page will jump to other.

[5.Reproduction build]: 
Flame 2.1 build:
Gaia-Rev        dbaf3e31c9ba9c3436e074381744f2971e15c7bf
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-2g34_v2_1/rev/ebce587d2194
Build-ID        20141203001205
Version         34.0

[6.Reproduction Frequency]: 
occasionally Recurrence,3/5
TCID:Free Test

Comment 1

[Lance（leave from Mozilla）]

Attachment: logcat_1603.txt

Comment 2

[Shine(Leave from Mozilla)]

Attachment: verify logcat

Hi Mike,
This problem can be repro on latest Flame 2.1, could you please help with it, thanks!
See attachment: Verify_logcat_0347.txt
Occurrence rate: 3/5

Flame 2.1 build:
Gaia-Rev        ed2e278753e8c9301ba322dcf2c3591f5928408d
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/127a0ead5f83
Build-ID        20150108001214
Version         34.0

Comment 3

[Mike Lien[:mlien]]

Hi Dominic, could you help with this, thanks.
v2.2 doesn't have this problem

Comment 4

[Dominic Kuo [:dkuo]]

This should be a gecko issue because looks like for some reason, though the user didn't actually touch the tabs area, but scrolling will trigger the tabs.

Can we request for regression window? kinda of sure it's a gecko regression, thanks.

Comment 5

[Mike Lien[:mlien]]

verify with v2.0 and it doesn't have this issue
Gaia-Rev        2989f2b2bd12fcc0e9c017d2db766e76a55873b8
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/6150867fc6a8
Build-ID        20150129160233
Version         32.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150129.200525
FW-Date         Thu Jan 29 20:05:36 EST 2015
Bootloader      L1TC100118D0

Comment 6

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

I tried for about an hour on a build around the same time as Shine's in comment 2 as well as today's 2.1 but was unable to reproduce.  It's possible I'm doing something wrong though.  I cannot view the attached mp4 because I get a corrupted file error.  Leaving the tags so someone else can work on this.

Environmental Variables:
Device: Flame 2.1
BuildID: 20150108063358
Gaia: ed2e278753e8c9301ba322dcf2c3591f5928408d
Gecko: 67911f617573
Version: 34.0 (2.1) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Comment 7

[Pi Wei Cheng [:piwei] (inactive)]

I was able to repro. Getting a reverse window now.

Comment 8

[Pi Wei Cheng [:piwei] (inactive)]

I was unable to find a reverse window because I couldn't reproduce the bug anywhere in Central. The following is the actual regression window within 2.1 branch.

2.1 Aurora regression window:

Last Working Environmental Variables:
Device: Flame
BuildID: 20141007143218
Gaia: 46fc0163f38d2fa04f7d41d7e8ca697b6849899c
Gecko: 99a3e25bc9d0
Version: 34.0a2 (2.1)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

First Broken Environmental Variables:
Device: Flame
BuildID: 20141007145817
Gaia: e9b9ee15eded2671656efbc40f8437ae8a61d55c
Gecko: 32f99eac065c
Version: 34.0a2 (2.1)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Last Working Gaia & First Broken Gecko - issue does NOT repro
Gaia: 46fc0163f38d2fa04f7d41d7e8ca697b6849899c
Gecko: 32f99eac065c

Last Working Gecko & First Broken Gaia - issue DOES repro
Gaia: e9b9ee15eded2671656efbc40f8437ae8a61d55c
Gecko: 99a3e25bc9d0

Gaia pushlog:
https://github.com/mozilla-b2g/gaia/compare/46fc0163f38d2fa04f7d41d7e8ca697b6849899c...e9b9ee15eded2671656efbc40f8437ae8a61d55c

Caused by the patch to Bug 1043712.

Comment 9

[KTucker [:KTucker][Inactive 3/4/2016]]

Dominic, can you take a look at this please? This might have been caused by the work done on Bug 1043712.

Comment 10

[Lance（leave from Mozilla）]

The bug still exist in latest Flame 2.1 build.

Fail rate:3/5

Flame 2.1:
Build ID               20150330161200
Gaia Revision          6f39e4e876152de1dcdcc0e7656197f22f105e4b
Gaia Date              2015-03-25 11:16:16
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/275ad18f587b
Gecko Version          34.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150330.193934
Firmware Date          Mon Mar 30 19:39:44 EDT 2015
Bootloader             L1TC000118D0

Comment 11

[Mike Lien[:mlien]]

[Blocking Requested - why for this release]: nom for regression

Comment 12

[Hema Koka [:hema]]

(In reply to KTucker [:KTucker] from comment #9)
> Dominic, can you take a look at this please? This might have been caused by
> the work done on Bug 1043712.

Dominic: Could you provide your input here please?

Thanks
Hema

Comment 13

[Hema Koka [:hema]]

NI TAM, Styang to triage for blocking decision on 2.1 release

Comment 14

[Dominic Kuo [:dkuo]]

(In reply to Hema Koka [:hema] from comment #12)
> (In reply to KTucker [:KTucker] from comment #9)
> > Dominic, can you take a look at this please? This might have been caused by
> > the work done on Bug 1043712.
> 
> Dominic: Could you provide your input here please?
> 
> Thanks
> Hema

Bug 1043712 is also fixed on 2.2, but status-b2g-v2.2 of this bug is unaffected, so probably not related. I need to investigate it but still I guess this maybe more related to gecko issue.

Comment 15

[GitHub Autolander Bot]

Attachment: [gaia] dominickuo:v2.1-bug-1107446 > mozilla-b2g:v2.1

Comment 16

[Dominic Kuo [:dkuo]]

Comment on attachment 8590123 [details] [review]
[gaia] dominickuo:v2.1-bug-1107446 > mozilla-b2g:v2.1

Okay, I kinda sure this is a gecko issue, because after I used the webide to inspect the styles of the bottom tabs, I found it didn't overlap the list area. And the strange thing is while the user scrolling the list, it looks like if the touchend event fires very close to the tabs, it will trigger the tabs touchend handler then switches to another view accidentally, this might means gecko reports wrong position to the document because it only happens in 2.1.

We do have some way to prevent this and a trivial one is to disable the tabs while scrolling, and my patch does it.

Comment 17

[Jim Porter (:squib)]

Comment on attachment 8590123 [details] [review]
[gaia] dominickuo:v2.1-bug-1107446 > mozilla-b2g:v2.1

I'm worried that this will just hide a more serious issue that could cause issues in other parts of the phone. However, if we can't figure out a real fix for this, this patch is good.

Comment 18

[Dominic Kuo [:dkuo]]

(In reply to Jim Porter (:squib) from comment #17)
> Comment on attachment 8590123 [details] [review]
> [gaia] dominickuo:v2.1-bug-1107446 > mozilla-b2g:v2.1
> 
> I'm worried that this will just hide a more serious issue that could cause
> issues in other parts of the phone. However, if we can't figure out a real
> fix for this, this patch is good.

Indeed, if this confirmed to be a gecko issue, then other apps will encounter the same problem. Probably we should let gecko devs know about it, though I don't know who to ask. I will leave the r+ patch here without landing it, in case this became a blocker or partner needs it.

Comment 19

[Steven Yang [:styang]]

users may hit this issue easily. I would like to take it for 2.1S. thanks.

Comment 20

[GitHub Autolander Bot]

Attachment: [gaia] dominickuo:v2.1-bug-1107446 > mozilla-b2g:v2.1s

Comment 21

[Vincent Liu[:vliu]]

Attachment: pull request for v2.1s branch

Comment 22

[Vincent Liu[:vliu]]

https://github.com/mozilla-b2g/gaia/commit/406f72699d59f07c63b758f1fc75f23dd463bacc

Comment 23

[Shally Li（leave from Mozilla）]

Hi William,

    Per Comment 22,this bug has been fixed on v2.1S. But this bug has been failed verified on latest v2.1S. Could you help with this bug ?

Thank you very much.

----------------------------------------------------------------------------------
See attachments: verify_v2.1s.MP4 and logcat_2259.txt
Reproduce rate: 3/15

Repro STR:
Premise:Device have more than 50 songs.
1.Launch Music.
2.Switch to Songs page.
3.Slide down page quickly.
**Sometimes the song tab will jump to other tab.Sometimes the song will be played automatically.

Device: v2.1S build(Fail)
Build ID               20150422001201
Gaia Revision          406f72699d59f07c63b758f1fc75f23dd463bacc
Gaia Date              2015-04-21 09:14:40
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g34_v2_1s/rev/692450fb8560
Gecko Version          34.0
Device Name            scx15_sp7715ea
Firmware(Release)      4.4.2
Firmware(Incremental)  122
Firmware Date          Thu Feb  5 12:42:58 CST 2015

Comment 24

[Shally Li（leave from Mozilla）]

Attachment: verify_v2.1s.mp4

Comment 25

[Shally Li（leave from Mozilla）]

Attachment: logcat_2259.txt

Comment 26

[William Hsu [:whsu]]

Hi, Shally,

Thanks for your help!

I think it is different issue.

Device switches to another page just because device recognizes the touch event as "Tap" (you hardly press screen). See, the demo video show that the page switched to play page, right?

:)

Comment 27

[William Hsu [:whsu]]

Oh~! Wait! Thanks for Mike's reminder!

The video shows the defect. (Timestamp - 1:09)

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Hi, Dominic,

May I have your help?
We still can reproduce this bug. This patch doesn't seem to fix this problem.
Any suggestion?

Comment 28

[Dominic Kuo [:dkuo]]

(In reply to Shally from comment #23)
> See attachments: verify_v2.1s.MP4 and logcat_2259.txt
> Reproduce rate: 3/15

After watching the video, I doubt the reproduce rate is 3/15. I saw the tester scrolled nearly 300 times then got only 1 time successfully reproduced it, so the new reproduce rate should be 1/300, that means my patch improved this bug but didn't solve it completely.

(In reply to William Hsu [:whsu] from comment #27)
> Hi, Dominic,
> 
> May I have your help?
> We still can reproduce this bug. This patch doesn't seem to fix this problem.
> Any suggestion?

I need to re-investigate it and will need the gecko devs help, as Jim and I commented in comment 17 and comment 18, we should let gecko dev know about this if it's really a gecko issue.

Comment 29

[Dominic Kuo [:dkuo]]

While I am re-testing this bug, I think it's better to request qa for help, so adding qawanted and see if it's still reproducible with the latest build, because until now, we couldn't reproduce on my side and the Taipei qa team.

Comment 30

[Dominic Kuo [:dkuo]]

(In reply to Dominic Kuo [:dkuo] from comment #29)
> While I am re-testing this bug, I think it's better to request qa for help,
> so adding qawanted and see if it's still reproducible with the latest build,
> because until now, we couldn't reproduce on my side and the Taipei qa team.

Oops, I am commenting on a wrong bug! please ignore this, thanks.

Comment 31

[Josh Cheng [:josh]]

Dear Dominic,
Do you have any update for this issue?
Thanks!

Comment 32

[Dominic Kuo [:dkuo]]

(In reply to Josh Cheng [:josh] from comment #31)
> Dear Dominic,
> Do you have any update for this issue?
> Thanks!

Hey Josh, I didn't have much time to investigate this, but I tried to reproduce it just now, with 100 songs in about 10 mins then scrolled nearly 1000 times, I couldn't reproduce it with the commit in comment 22.

Can we request for qawanted to verify again? or needinfo someone who works on gecko though I really don't know who to ask...

Comment 33

[Pi Wei Cheng [:piwei] (inactive)]

Note:
QAnalysts don't have a Dolphin device, and this so far has only landed on Dolphin, so we can't verify this on Flame. Someone else who has a Dolphin will need to verify this.

Comment 34

[William Hsu [:whsu]]

(In reply to Dominic Kuo [:dkuo] from comment #32)
> (In reply to Josh Cheng [:josh] from comment #31)
> > Dear Dominic,
> > Do you have any update for this issue?
> > Thanks!
> 
> Hey Josh, I didn't have much time to investigate this, but I tried to
> reproduce it just now, with 100 songs in about 10 mins then scrolled nearly
> 1000 times, I couldn't reproduce it with the commit in comment 22.
> 
> Can we request for qawanted to verify again? or needinfo someone who works
> on gecko though I really don't know who to ask...

Hi, Dominic,

May I have a question?
Did you use Dolphin to reproduce this bug?

Comment 35

[Dominic Kuo [:dkuo]]

No, I used Flame to reproduce it. If this issue depends on the hardware then I think we really need someone from gecko/gonk to investigate.

Josh, would you please find some gecko/gonk dev to investigate this? (if my patch in comment 22 does not help here) thanks!

Comment 36

[Josh Cheng [:josh]]

Hi Lancy,
Can you verify whether the bug still exist on Flame 2.1 or Dolphin 2.1S?
Thanks

Comment 37

[Lancy(Leave from Mozilla)]

Attachment: Verify3_Flame2.1_Fail.mp4

Hi Josh,

This issue still exist on latest Flame 2.1 build and Dolphin2.1S build.

See attachments: Verify3_Flame2.1_Fail.mp4 and Verify3_log_Flame2.1.txt, Verify3_Dolphin 2.1S_Fail.mp4 and Verify3_logcat_Dolphin2.1S.txt.

Flame 2.1 Occurrence time:06:58
Flame 2.1 Reproduce rate: 1/30

Dolphin2.1S Occurrence time:22:36
Dolphin2.1S Reproduce rate:1/35

Repro STR:
Premise:Device have more than 50 songs.
1.Launch Music.
2.Switch to Songs page.
3.Slide page up and down quickly.
**Sometimes the song tab will jump to other tab.

Device: Dolphin2.1S 512m build(Fail)
Build ID               20150511001203
Gaia Revision          e4c6fee5a08daddf2d9dc90be7c1516e0f371f9d
Gaia Date              2015-05-06 21:48:08
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g34_v2_1s/rev/c1903564bbe6
Gecko Version          34.0
Device Name            scx15_sp7715ea
Firmware(Release)      4.4.2
Firmware(Incremental)  122
Firmware Date          Thu Feb  5 12:42:58 CST 2015

Device Flame 2.1 build (Fail)
Build ID               20150511001203
Gaia Revision          3e7bd686ecd852f4dfa4605b45f558e6bd34f02a
Gaia Date              2015-05-07 15:12:34
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/de4ebfdc7db6
Gecko Version          34.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150511.033924
Firmware Date          Mon May 11 03:39:35 EDT 2015
Bootloader             L1TC000118D0

Comment 38

[Lancy(Leave from Mozilla)]

Attachment: Verify3_log_Flame2.1.txt

Comment 39

[Lancy(Leave from Mozilla)]

Attachment: Verify3_Dolphin 2.1S_Fail.mp4

Comment 40

[Lancy(Leave from Mozilla)]

Attachment: Verify3_logcat_Dolphin2.1S.txt

Comment 41

[Steven Yang [:styang]]

Hi Chens, would you mind checking this issue? thanks.

Comment 42

[Josh Cheng [:josh]]

Hi Lancy,
Per comment 37, did you repo this with USB attached? 
If you did, can you try again with no USB connected because it might just be noise? 
Thanks!

Comment 43

[Steven Yang [:styang]]

I don't insist to fix it in 2.1S if team don't have time on this.

Comment 44

[Lancy(Leave from Mozilla)]

(In reply to Josh Cheng [:josh] from comment #42)
> Hi Lancy,
> Per comment 37, did you repo this with USB attached? 
> If you did, can you try again with no USB connected because it might just be
> noise? 
> Thanks!

Hi Josh,
I have verified the issue again without USB cable inserted, this issue cannot be reproduced on latest Dolphin 2.1S 512m build, but can be reproduced on latest Flame 2.1 build.

See attachments:Verify4_Dolphin 2.1S_Pass.mp4, Verify4_Flame2.1_Fail.mp4 and Verify4_Flame2.1_log.txt.
Flame 2.1 occurrence time:20:33.
Flame 2.1 reproducing rate:1/20.
Dolphin 2.1S reproducing rate:0/200

Device :Dolphin 2.1S 512m build (Verified Pass)
Build ID               20150512001201
Gaia Revision          e4c6fee5a08daddf2d9dc90be7c1516e0f371f9d
Gaia Date              2015-05-06 21:48:08
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g34_v2_1s/rev/6e6c3a969789
Gecko Version          34.0
Device Name            scx15_sp7715ea
Firmware(Release)      4.4.2
Firmware(Incremental)  122
Firmware Date          Thu Feb  5 12:42:58 CST 2015

Device :Flame 2.1 build (Verified Fail)
Build ID               20150512161201
Gaia Revision          c80865cb0bf73f1b97defbc646083b404feb3ac4
Gaia Date              2015-05-12 06:26:43
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/2fd3ef3fc14a
Gecko Version          34.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150512.194519
Firmware Date          Tue May 12 19:45:28 EDT 2015
Bootloader             L1TC000118D0

Comment 45

[Lancy(Leave from Mozilla)]

Attachment: Verify4_Flame2.1_Fail.mp4

Comment 46

[Lancy(Leave from Mozilla)]

Attachment: Verify4_Flame2.1_log.txt

Comment 47

[Lancy(Leave from Mozilla)]

Attachment: Verify4_Dolphin 2.1S_Pass.mp4