909228 - [B2G][Helix][Shutdown][HanCheng]Can't shutdown the device when playing video.

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Vendcom, defect, P2)

Description

[lecky]

Attachment: shutdown-logcat.log

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; aff-kingsoft-ciba; .NET4.0C; .NET4.0E)

Steps to reproduce:

1. Play a video.
2. Press power key to shutdown the device.


Actual results:

2. When the device is playing a video, it can't be powered off.


Expected results:

2. The device can be powered off when playing a video.

Comment 1

[lecky]

Hi,

My video is in internal SD card.

Reproduce step:
1. Do not insert SD card.
2. Record a video in camera app.
3. Play the video.
4. Press power key to power off the device.

Comment 2

[lecky]

Hi Wayne,

Is anyone work on this?

Comment 3

[Wayne Chang [:wchang]]

Hi Tim,

Should this be checked starting with system app? Can you suggest someone suitable?

Thanks

Leaving myself on ni? to revisit.

Comment 4

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

(In reply to lecky from comment #0)
> Created attachment 795304 [details]
> shutdown-logcat.log
> 
> Steps to reproduce:
> 
> 1. Play a video.
> 2. Press power key to shutdown the device.

shutdown -> screen off, or shut down from the sleep menu?

> 
> 
> Actual results:
> 
> 2. When the device is playing a video, it can't be powered off.

Please state what happened instead of what did not happen. This statement does not provide additional information.

> 
> 
> Expected results:
> 
> 2. The device can be powered off when playing a video.

Comment 5

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

Can QA verify this bug?

Comment 6

[William Hsu [:whsu]]

I cannot reproduce this bug on following build.
* Test build: Mozilla - V1.1.0 hd
  - BuildID: 20130912042203
  - Gaia:  "40a1cf0e29c633a879513b261af0340b69231f7b"
  - Gecko: "710e8b760ccb"
  - Gecko version: 18.0


Test Scenarios:
1. Entering sleep mode
   * Test steps:
    1. Launching the Camera app
    2. Record a video
    3. Pause the stop button to stop recording
    4. Tap the video icon on the top of the screen to play the video
    5. Press the power key to enter sleep mode
   * Result:
    - The device entered the sleep mode. (Pass)
    

2. Power off device
   * Test steps:
    1. Launching the Camera app
    2. Record a video
    3. Pause the stop button to stop recording
    4. Tap the video icon on the top of the screen to play the video
    5. Long press the power key
    6 After FxOS pops up the shutdown menu, press the "Power off" option
   * Result:
    - The device was shut down.(Pass)
    
3. Restart the device
   * Test steps:
    1. Launching the Camera app
    2. Record a video
    3. Pause the stop button to stop recording
    4. Tap the video icon on the top of the screen to play the video
    5. Press the power key to enter sleep mode
    6 After FxOS pops up the shutdown menu, press the "Restart" option
   * Result:
    - The device was restarted.(Pass)
    
By the way, I play the video from Gallery and Video app.
The device still can be restarted & shut down.

I will test it on partner build to see whether I can reproduce it.



--------- Needinfo myself ---------

Comment 7

[William Hsu [:whsu]]

Hi, all,

OOPS~ I "can" reproduce this case on partner build!
While user is playing the video the device cannot be restarted.

* Test build: 20130827
Attaching the log

Any thought?

Comment 8

[William Hsu [:whsu]]

Attachment: Video Log - 20130916

Comment 9

[William Hsu [:whsu]]

Hi, Tim,

I can reproduce this bug on partner build but I cannot reproduce this bug on Mozilla build.
Do we need to fix it?
If so, who can help on this problem?
Thank you!

Comment 10

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

POVB?

Comment 11

[lecky]

Hi Wayne :
 Please help us to push to merge this case into V1.1 HD. It blocked the Helix Device .

Comment 12

[Wayne Chang [:wchang]]

Lecky,

This problem is only seen on your build. there is no problem on Mozilla build. We have no actionable item here and there is nothing to merge.

Please follow up with your team.

Comment 13

[lecky]

Hi Wayne,

OK, I will check the different between our build and Mozilla build.

Comment 14

[lecky]

Dear Tim,

On our V1.1HD, in gaia\apps\system\js\sleep_menu.js, in function startPowerOff, after execute the code "var logoLoader = new LogoLoader(CustomLogoPath.poweroff)", the logoLoader.onload will be called after the video is finished. So it means that the device will power off after the video finished. 

Our issue is that our device can't power off immediately when playing the video. If the video is too long, it will wait for a long time to power off our device.

Could you have a look at it and check if it also exist on the master?

Thanks.

Comment 15

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

lecky, the only to know that is simply try to build the master branch with the same customization & video.

(In reply to lecky from comment #14)
> Our issue is that our device can't power off immediately when playing the
> video. If the video is too long, it will wait for a long time to power off
> our device.

Isn't that exactly describe the desire effect? The device should always play the entire shutdown video before poweroff.

Comment 16

[lecky]

Dear Tim,

On our device, normally, if we press the power button to turn off the device, it will play shutdown animation before power off.

Now our issue is:
If our device is playing another video(not the shutdown animation), if we press the power button to turn off the device, it will wait for to finish the playing video(step 1), and then go to play the shutdown animation(step 2), and power off in the end(step 3).

In the step 1, if the video is too long, it will wait for a long time before power off. if so, our customer will not accept it.

Thanks.

Comment 17

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

OK. This is not a PVOB bug but a bug there the shutdown video must wait for another video to finish.

I suggest we kill all the apps when the user selects "power off" from the menu.
Alive, do you think that is doable?

Comment 18

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

We could try "fire a power-off event to window manager to kill all apps and wait for the terminated events to be fired, and then do the power-off animiation."

Comment 19

[lecky]

Does this issue can be solved on V1.1?

Thanks.

Comment 20

[Evelyn Hung]

nominate hd?(or leo?) to get attention per comment 19, but from my perspective, it's not a blocker if the user can manually pause the video and turns power off.

Comment 21

[Wayne Chang [:wchang]]

Not blocking here as low user impact if pausing video allow switch off.

Comment 22

[KM Lee [:rexboy] (inactivate after Sep. 22)]

It's a known issue that we can't play two videos at the same time.
I just made some test, it didn't happen when we don't have custom poweroff animation.
(since the default one is just a css animation)

So I just came up a workaround: If we are playing some video, just don't play the custom
poweroff animation and use the default css poweroff animation, then things should go well.

We need to confirm that 1) video tag have such an event to distinguish this situation and
2) we all agree on this workaround though.

Comment 23

[Beatriz Rodríguez [:brg]]

>(In reply to KM Lee[:rexboy] from comment #22)
> It's a known issue that we can't play two videos at the same time.
> I just made some test, it didn't happen when we don't have custom poweroff
> animation.
> (since the default one is just a css animation)
> 
> So I just came up a workaround: If we are playing some video, just don't
> play the custom
> poweroff animation and use the default css poweroff animation, then things
> should go well.
> 
This workaround won't be accepted by certification. Sorry. Custom power off should be played always. However this issue was not discovered during our IOT process because there is not specific test case!

We do not see this issue as a blocker but can we include the comment 18 suggestion for 1.3 or at least in product backlog? Ni Peter (FxOS product team) so he knows who can take care of it.

Comment 24

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

Additionally, when this bug is fixed, I would love to have integration tests in the tree to protect the behavior.

Comment 25

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

I have an alternative here:
Before we start the shutdown animation(video), send the current displayed app to background.
This would make it paused and this way is easier and faster than kill all apps.

Comment 26

[Peter Dolanjski [:pdol]]

Bruce, can you track this in the appropriate backlog for fixing beyond 1.2?

Comment 27

[Bruce Huang [:bhuang] (inactive)]

Adding to backlog

Comment 28

[BMO Automation]

Firefox OS is not being worked on