1066871 - Video not being displayed

Status: RESOLVED WONTFIX

(Marketplace Graveyard :: Consumer Pages, defect, P3)

Description

[Andy McKay]

This app has a valid video (webm) on it:

https://marketplace-dev.allizom.org/app/compound-interest/

Clicking on the previews on scrolling along you get:

https://www.dropbox.com/s/qph96exzoynu48z/Screenshot%202014-09-12%2015.31.38.png?dl=0

As an aside, there's actually a warning sign in that page that you can't see.

The webm is there, for example: wget https://marketplace-dev-cdn.allizom.org/img/uploads/previews/full/152/152048.webm?modified=141056065 will get it for you.

Is this a matter of fireplace figuring out its a video and converting to a video tag appropriately?

Comment 1

[Davor Spasovski [:spasovski]]

The cause of this is a failure at https://github.com/mozilla/fireplace/blob/master/src/media/js/lightbox.js#L91 but it would need some more investigation. This is actually a fun bug.

Comment 2

[Andy McKay]

https://github.com/mozilla/zamboni/pull/2798

Comment 4

[Valentina Virlics [:vvalentina], Ecosystem QA]

The videos are still having appearance problems in app details page on MP-stage and MP-dev FF39(Win 7). I think this issue is also related with bug 1147624 (https://bugzilla.mozilla.org/show_bug.cgi?id=1147624). 
Please see screenshots: http://screencast.com/t/h9LrHUlEHu (MP-stage) and the app mentioned at comment #0 http://screencast.com/t/eHnvZrx6ZC

Comment 5

[Andy McKay]

Due to the db refresh, the app mentioned at comment 0 will no longer work. I tried it on stage and it didn't work.

Comment 6

[Wil Clouser [:clouserw]]

I added a video to https://marketplace.allizom.org/app/test-app-bat1429/ to test this.  On the dev hub the preview for it shows up, but I get the same failures Valentina gets on the consumer side.

Comment 7

[Davor Spasovski [:spasovski]]

https://github.com/mozilla/fireplace/pull/1217

Comment 8

[Valentina Virlics [:vvalentina], Ecosystem QA]

Verified as fixed on MP-dev on FF41(Win 7).
Postfix screencast: http://screencast.com/t/6uBf1riU
Closing bug.

Comment 9

[Valentina Virlics [:vvalentina], Ecosystem QA]

Because is almost the same issue, but different platform, I'll reopen this, due to its reproduction on MP-dev on FF OS 2.0. 

Note: the video cannot be seen in listing pages or app details page.

Please see the screenshot: http://screencast.com/t/qR8i47DwEw

Comment 10

[Davor Spasovski [:spasovski]]

works fine for me:

http://i.imgur.com/FERmcLF.png

Comment 11

[krupa raj[:krupa]]

STR:

1. Go to the apps details page (videoapp) on your FirefoxOS phone
2. Click on the video to play it
3. Close the lightbox
4. Notice the video listing now

expected behavior:
the video still has the play ">" on it

actual behavior:
It is a blank screen if the video has been played before.

Comment 12

[Valentina Virlics [:vvalentina], Ecosystem QA]

(In reply to Davor Spasovski [:spasovski] from comment #10)
> works fine for me:
> 
> http://i.imgur.com/FERmcLF.png

I was talking about MP-dev on FF OS 2.0, and I can still reproduce the initial issue.

Comment 13

[Davor Spasovski [:spasovski]]

It's a bit limited on what we can do here...the following commit should make sure we have the first frame and time data for each video (so that we don't get a transparent element as in comment 9). The video being tested has a fully white fade-in opening so that frame tends to be all-white at the moment as seen in: http://i.imgur.com/Y7pKIBE.png

https://github.com/mozilla/fireplace/commit/5b220e6

The only thing further we can try is to shimmy in a background image via CSS to let the user know a video will exist there but let's see if this gets us where we want to be.

Comment 14

[Valentina Virlics [:vvalentina], Ecosystem QA]

Verified on MP-dev FF OS 2.0(Flame) and I'm seeing 2 cases:

1. App details page: not able to see the video in the screenshots sections, but if I click the section an arrow is displayed in the lightbox and a "loanding rectangle" in the backscreen section (http://screencast.com/t/3LqgihVx4ihT)

2. Apps listing pages: i.e. Test App(videoapp)from New page, Toggle Apps Preview in expand mode, a white rectangle is displayed, if tapped an arrow is loaded in the lightbox and the video is playing in the backscreen section; if the arrow is also tapped, it disappears  and the lightbox remains open. (http://screencast.com/t/t7bHXa311)

Comment 15

[Victor Carciu]

Issue from comment #14 is reproducible also on Android, latest Nightly .

Comment 16

[Valentina Virlics [:vvalentina], Ecosystem QA]

Verified on FF43(Win 7) on MP-stage and the newly submitted videos are not working at all.
I used the same video like before; please see the screencast: http://screencast.com/t/JxmUfg6ebA6

Comment 17

[Valentina Virlics [:vvalentina], Ecosystem QA]

Another issue regarding videos is now seen during the submission process.
Even if I use the same video like before, and is a valid *.webm file, it cannot be saved.
Please see the screencast: http://screencast.com/t/E2pzzfWN

Please, do tell if I need to log a separate issue.

Comment 18

[David Durst [:ddurst]]

Possibly related to bug 1209106. CC ashort for confirmation

Comment 19

[Allen Short [:ashort]]

yes, the submission problem is bug 1209106.

Comment 20

[Davor Spasovski [:spasovski]]

.