824156 - [B2G] [Video] Video will be stopped from playing when alarm goes off.

Status: RESOLVED WORKSFORME

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

Description

[Misty Byrd]

Perquisites:
1.Have a video available to play from the video app that is at least 3 minutes in length.
2. Have an alarm set to sound with in the next 2 mins during playing the video

Steps to Reproduce: When playing video from the  video player app, and an alarm set up previously goes off the video is stopped. See test case  #5848  Step 5. When reach the alarm time
Then hear the alarm and the video will be mute




            Steps to Reproduce:
1) Install Build 201212170202 on an Unagi Device
2) Select Video app
3) Verify a working video of at least 3 minutes is available
4) Select Clock app
5) Verify an alarm will go off within the time length of video
6) Play video 
7) Wait for alarm to sound


Actual Results:
When alarm sounds the video currently playing is stopped and must select to resume playing

Expected Results:
 hear the alarm and the video will be mute

Repro frequency:
Repros 10/10 times

Comment 1

[[e-mail], [e-mail]]

It looks like an issue of Video App. I'm not sure the expected results which is wanted by UX. We need UX to confirm the behavior for Video app. 

Josh, 
Could you help to figure out the owner who designs for Video app.

And I think Video App will need to handle 'mozinterruptbegin' or 'mozvisibilitychange' event when alarm goes off during video playback.

Comment 2

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

AFAIK, the video app is designed to 'Stops when falling into background'. Since attention screen makes it fall into background, stopping the video is an expected behavior.

Comment 3

[Josh Carpenter [:jcarpenter]]

(In reply to Alive Kuo [:alive] from comment #2)
> AFAIK, the video app is designed to 'Stops when falling into background'.
> Since attention screen makes it fall into background, stopping the video is
> an expected behavior.

Ideally we should pause the active video, not stop it. The user must then manually un-pause the video once the app is back in the foreground.

Comment 4

[Misty Byrd]

I am changing the status on this bug. I see conflicting responses. I will keep it open until a definitive answer is found.

Comment 5

[Misty Byrd]

This issue repros in build #20121231070201

Comment 6

[Misty Byrd]

This issue repros in Ungia, build 20130104070203 v.1.0, the video is stopped when alarm sounds

Comment 7

[amiller]

UCID: clock-000
https://moztrap.mozilla.org/results/case/61290/

Comment 8

[Darren]

Issue occurs in Unagi build 20130115070201 with December 5th kernel.

Comment 9

[Angela Hubenya]

the video is still being stopped and returns user to Video Gallery when the alarm goes off. Unagi, build 20130125070201

Comment 10

[Angela Hubenya]

the video is still being stopped and returns user to Video Gallery when the alarm goes off. Unagi, build 20130130070201
UCID: clock-000
TC#: 5848 
https://moztrap.mozilla.org/runtests/run/712/env/296/?pagenumber=1&pagesize=20&sortfield=order&sortdirection=asc&filter-id=5848

Comment 11

[Jeni]

Video turns off when alarm goes off.  Once alarm has been accepted or snoozed it brings you back to the listing of videos instead of video resuming where left off.  
Build ID 20130221070203
Kernel Date: Dec 5
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/effe37a77f18

Comment 12

[Jeni]

Issue repros
 Unagi Build ID: 20130225070200
Kernel Date: Dec 5
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/3a5a27992a75
Gaia: 5691a16fff8e1403c75ed9d6f3a443b7e58198c6
Video turns off when alarm goes off. Once alarm has been accepted or snoozed it brings you back to the listing of videos instead of video resuming where left off.

Comment 13

[Nick Davidson]

in reference of comment 12 additional information: correct test case link: https://moztrap.mozilla.org/manage/cases/?pagenumber=1&pagesize=20&sortfield=created_on&sortdirection=desc&filter-id=5848

Comment 14

[Russ Nicoletti [:russn]]

When repeating the test on a buri, I find that the video resumes playing when the alarm is dismissed. I'm assuming the change in behavior is intentional. John, can you comment on this bug?  

Below is the build I tested:

Gaia      63cc8ffd6f2acdce14eb8285e8822ebe62fcab41
Gecko     http://hg.mozilla.org/releases/mozilla-aurora/rev/37ec3c7fb511
BuildID   20140120004002
Version   28.0a2

Comment 15

[John Hu [:johnhu][:johu][:醬糊小弟]]

Russ,

No, that's not intentional.

We pause the video playing when we receive a visibility change event. But we don't resume it but just restore it when we go back to video app.

I keep needinfo flag to wait me to check the code in this version.

Comment 16

[John Hu [:johnhu][:johu][:醬糊小弟]]

I had checked the code. We don't have any change to 1.3 since a month ago. The code is still as expected. I never did any tests on this. But I believe that it's not caused by video app.

Comment 17

[John Hu [:johnhu][:johu][:醬糊小弟]]

I will install a build into my device and test this scenario.

Comment 18

[John Hu [:johnhu][:johu][:醬糊小弟]]

Russ,

I cannot reproduce what you said in comment 14 with Inari v1.3 20140120004002. It works as expected and the video is still paused when the alarm goes off. But there is an exception that System app sends visibilitychange when the alarm window shows for 3 seconds. Within these 3 secs, if user close the alarm window, the video app still plays the video.

Comment 19

[John Hu [:johnhu][:johu][:醬糊小弟]]

Russ,

Do we still have this issue? I feel this is related to window manager of system app. It controls the visibility of each apps that affects video app a lot.

Comment 20

[Russ Nicoletti [:russn]]

I don't see any issue. The video is paused when the alarm goes off (for more than three seconds). From what I can tell it works the way Josh Carpenter [:jcarpenter] (comment #3) says it should.

Comment 21

[Russ Nicoletti [:russn]]

Adding needinfo for John to decide how to proceed with this bug.

Comment 22

[John Hu [:johnhu][:johu][:醬糊小弟]]

Russ,

Ok, cool. We all can confirm this. I make this bug as WORKSFORME.

If somebody can reproduce it, please reopen it.