1020264 - [flame] Status bar is displayed after launching video app and you can't play a video until it dissapears

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect, P1)

Description

[Viorela Ioia [:viorela]]

Tapping on a video after launching the video app has no effect. The status bar still present and you have to wait 2-3 seconds until it dissapears

#Prerequisities:
Have a video saved in your phone

#STR:
1. Launch Video app
2. Tap the video item listed there

#Expected results:
The video is launched in full screen view

#Actual results:
The tap has no effect; The status bar is covering the video app and you can't play the video until it dissapears


Note: The status bar should dissapear once the video app is launched

I was able to reproduce this manually and also by running the automated tests
The issue is not reproducible on v1.4

Last working build: 
Gaia      b669dd2cc321f37cebc7081a79b968cac36b4200
Gecko     https://hg.mozilla.org/mozilla-central/rev/b85b57f05fda
BuildID   20140529160204
Version   32.0a1
ro.build.version.incremental=94
ro.build.date=Tue May 20 09:29:20 CST 2014

First failing build:
Gaia      26d8fcab9b61f46451600f39c51e0387ef3c4f88
Gecko     https://hg.mozilla.org/mozilla-central/rev/e6f113c83095
BuildID   20140530040207
Version   32.0a1
ro.build.version.incremental=94
ro.build.date=Tue May 20 09:29:20 CST 2014

Link to jenkins report:
http://selenium.qa.mtv2.mozilla.com:8080/view/B2G%20Flame/job/b2g.flame.mozilla-central.ui.non-smoketest/48/HTML_Report/

Comment 1

[Jason Smith [:jsmith]]

Can we get a window from our automated tests here?

Comment 2

[Viorela Ioia [:viorela]]

Attachment: screenshot.png

In the screenshot you can see that the text 'May 2014' from thumbnail header is not visible entirely. The status bar is black and it is covering the video app

Comment 3

[bhavana bajaj [:bajaj]]

Blocking as this is a bad regression.

Comment 4

[Harvey]

Leaving regressionwindow-wanted tag as per comment 1.

Issue appears in the builds before our first available B2G Inbound build. Regression window was done using Central.

Central Regression Window

Last working
Environmental Variables:
Device: Buri
BuildID: 20140203150906
Gaia: 53483b08ac780d19c085874006763bb1ef45c83d
Gecko: 23555d9a4a17
Version: 30.0a1
Firmware Version: v1.2-device.cfg

First Broken
Environmental Variables:
Device: Buri
BuildID: 20140203151506
Gaia: 75e9691f02b9d18585c18a5434beeff39ee7ea20
Gecko: c150845d077d
Version: 30.0a1
Firmware Version: v1.2-device.cfg

Last Working Gaia First Broken Gecko: Issue does NOT reproduce
Gaia: 53483b08ac780d19c085874006763bb1ef45c83d
Gecko: c150845d077d

First Broken Gaia Last Working Gecko: Issue DOES reproduce
Gaia: 75e9691f02b9d18585c18a5434beeff39ee7ea20
Gecko: 23555d9a4a17

Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/53483b08ac780d19c085874006763bb1ef45c83d...75e9691f02b9d18585c18a5434beeff39ee7ea20

Comment 5

[Jason Smith [:jsmith]]

The window in comment 4 definitely isn't right here. Automation is pointing this to being a recent regression on 6/4/2014, so the window should be present within the past few days.

I don't think you've reproduced the right bug here.

Comment 6

[Harvey]

The Regression Window in comment 4 was done with a status bar that blocked the video app for ~1 second, The below window was done with a status bar that blocked the video app for ~3 seconds.

B2G Inbound Regression Window

Last Working
Environmental Variables: 
Device: Flame 2.0 
Build ID: 20140528233003 
Gaia: eced604040a970e65dfb51fa83e1a18a9d00b7db 
Gecko: fe9a8825a3ae 
Version: 32.0a1 (2.0) 
Firmware Version: v10G-2

First Broken
Environmental Variables: 
Device: Flame 2.0 
Build ID: 20140529075121 
Gaia: d5dcbabd1ad07d5fd30a7875a9a80817bc93619c 
Gecko: f72106cb1769 
Version: 32.0a1 (2.0) 
Firmware Version: v10G-2

Last Working Gaia First Broken Gecko: Issue does NOT reproduce
Gaia: eced604040a970e65dfb51fa83e1a18a9d00b7db
Gecko: f72106cb1769

First Broken Gaia Last Working Gecko: Issue DOES reproduce
Gaia: d5dcbabd1ad07d5fd30a7875a9a80817bc93619c
Gecko: fe9a8825a3ae

Gaia Pushlog: 
https://github.com/mozilla-b2g/gaia/compare/eced604040a970e65dfb51fa83e1a18a9d00b7db...d5dcbabd1ad07d5fd30a7875a9a80817bc93619c

Comment 7

[Jason Smith [:jsmith]]

Maybe bug 950673 broke this?

Alive - What do you think?

Comment 8

[Gregor Wagner [:gwagner]]

(In reply to Jason Smith [:jsmith] from comment #7)
> Maybe bug 950673 broke this?
> 
> Alive - What do you think?

Forwarding to window-managment for diagnosis since 950673 is in window-management component.

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Will investigate.

Comment 10

[(inactive after 6/18) Alive Kuo [:alive]]

Could we have someone on this and steal it from me?

Comment 11

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

George, could you take this?

Comment 12

[(Inactive after June) George Duan [:gduan] [:喬智]]

Attachment: PR to master

Hi Alive,
The root cause is due to the timer of OPENING_TRANSITION_TIMEOUT is extended from 300 to 2500ms, so the appopened event delay the disappearing of status bar.

This patch has change the event from appopened to appopening in statusbar.js.

Could you kindly help me to check this patch?
Thanks.

Comment 13

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8441928 [details] [review]
PR to master

Better for having unit tests.

Comment 14

[(Inactive after June) George Duan [:gduan] [:喬智]]

test added.
Thanks Alive,

tbpl result( some of tests fail but I'm sure it's not related to this bug. )
https://tbpl.mozilla.org/?tree=Gaia-Try&rev=0e421edb1ca7e7a201c08d6d47cc90b0cfaa6c18

merged to master.
https://github.com/mozilla-b2g/gaia/commit/a20eaaf420f33b48e36565dc9b02b8034ac2d3d1

Comment 15

[Ryan VanderMeulen [:RyanVM]]

v2.0: https://github.com/mozilla-b2g/gaia/commit/7411dbbcc8257a03080d00b5b58a00edd814b003

Comment 16

[Yeojin Chung [:YeojinC]]

This issue is verified fixed on Flame 2.0 and 2.1.

Result: Status bar never appears when the Video app is launched. The video files play properly when selected from the list.

Device: Flame 2.0 (319mb, KK, Shallow Flash)
Build ID: 20141118000207
Gaia: 1ede2666f1e6c1b3fd3b282011caf0cbc59544b0
Gecko: 2bea026d4f86
Version: 32.0 (2.0)
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Device: Flame 2.1 (319mb, KK, Shallow Flash)
BuildID: 20141118001204
Gaia: 1b231b87aad384842dfc79614b2a9ca68a4b4ff3
Gecko: 95fbd7635152
Version: 34.0 (2.1) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0