1015431 - [B2G][Flame][Vertical Homescreen] Flash of open animated app menu is seen on locked device when user wakes it

Status: RESOLVED DUPLICATE of bug 1016483

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

Description

[Rachel Pribble [:rpribble]]

Description:
If the user locks the device while an app with animation playing is open and running, then presses the lock button again to wake the device, they will see a flash of the open app before the device moves to the lock screen (app Slice Fruits used in testing).

Repro Steps:
1) Updated Flame to BuildID: 20140523073020
2) Set UI to vertical scroll
3) Download an app from the marketplace with animation on the menu/home screen (Slice Fruits)
4) Lock the device
5) Press the lock button again
6) Observe flash of running app

Actual:
User sees a flash of the running app before being shown the lock screen.

Expected:
Content is not able to be seen if device is locked.

Environmental Variables:
Device: Flame v2.0 MOZ ril
BuildID: 20140523073020
Gaia: e83bd91c8f7c66f95d55cb741299e12bc8f7b429
Gecko: 681c5668d881
Version: 32.0a1
Firmware Version: v10G-2

Notes:

Repro frequency: 100%
See attached: Video

-------------------------------------------------

This issue also occurs on the Open_C v2.0 engineering build.

Open C 2.0 Eng
2.0 Environmental Variables:
Device: Open_C v2.0 MOZ ril
BuildID: 20140523073020
Gaia: e83bd91c8f7c66f95d55cb741299e12bc8f7b429
Gecko: 681c5668d881
Version: 32.0a1
Firmware Version: P821A10V1.0.0B06_LOG_DL

User sees a flash of the running app before being shown the lock screen.

Comment 1

[Rachel Pribble [:rpribble]]

Attachment: Logcat.txt

Comment 2

[Rachel Pribble [:rpribble]]

Attachment: Video.ogg

Comment 3

[Jason Smith [:jsmith]]

Does this reproduce on the old homescreen?

Comment 4

[Eric Kramer [:Kramer]]

This issue DOES repro on the old homescreen.

Environmental Variables:
Device: Flame v2.0 MOZ ril
BuildID: 20140523073020
Gaia: e83bd91c8f7c66f95d55cb741299e12bc8f7b429
Gecko: 681c5668d881
Version: 32.0a1
Firmware Version: v10G-2

Comment 5

[Jason Smith [:jsmith]]

Ok - can we check if this reproduces on 1.4?

Comment 6

[Eric Kramer [:Kramer]]

This issue does NOT repro on 1.4

1.4 Environmental Variables:
Device: Flame 1.4
BuildID: 20140529000204
Gaia: 7bc1c15c67661a0b8e35f18f15a9d03d1d2cfcd5
Gecko: 2181cac4d0fc
Version: 30.0
Firmware Version: v10G-2

Comment 7

[Kevin Grandon :kgrandon]

This is not specific to the homescreen, but still something we should absolutely fix. To help the engineers focus on their appropriate areas, I would like to remove blocking on Test plan bugs. Let me know if you have any problems of that.

At the same time I would like to ni? alive here in case he is aware of any recent window manager changes that may have caused this.

Comment 8

[Jason Smith [:jsmith]]

(In reply to Kevin Grandon :kgrandon from comment #7)
> This is not specific to the homescreen, but still something we should
> absolutely fix. To help the engineers focus on their appropriate areas, I
> would like to remove blocking on Test plan bugs. Let me know if you have any
> problems of that.
> 
> At the same time I would like to ni? alive here in case he is aware of any
> recent window manager changes that may have caused this.

That makes sense to me.

Comment 9

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

I suspect this is due to lockscreenWindow z-index fail.
Greg, are you able to work on?

Comment 10

[Greg Weng [:snowmantw][:gweng][:λ]]

In fact this happens after some recent bugs landed (after 5/10 I remembered), and I've solved it once in as-an-app bug. But I think it's good to have a clear regression.

I'll do a bisect after complete the current work.

Comment 11

[Rachel Pribble [:rpribble]]

This issue is now occurring with all apps on the newest build, regardless of if they have animated elements or not.

Environmental Variables:
Device: Flame v2.0 MOZ ril
BuildID: 20140530040207
Gaia: 26d8fcab9b61f46451600f39c51e0387ef3c4f88
Gecko: e6f113c83095
Version: 32.0a1
Firmware Version: v1.2-device.cfg

If user has an app open, then locks the device, then presses the unlock button to view the lock screen, they will see a flash of the app before the lock screen loads correctly.

Comment 12

[Greg Weng [:snowmantw][:gweng][:λ]]

I've bisect this is a regression caused by Bug 1009950. I would take a look if we can back it out or the author (Etienne) can fix it in another bug.

Comment 13

[Etienne Segonzac (:etienne)]

Looks like the fix for bug 1016483 also fixed this issue.
I still get a white flash some times when turning the screen on (we've had this since the lockscreen window introduction) but I never see the app.