1133677 - Video is not resumed when touching the screen in the video area in youtube

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Thomas]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.19 Safari/537.36

Steps to reproduce:

[PROCEDURE]
(PRSE ON, SD Card plugged, WIFI & BT On , PRSE connected to internet)

- Go to applications, Launch Firefox
=> Firefox is launched
- Go to Youtube
- Launch a video 
=> Video is played
- Touch the screen in the video area to pause the video (Please check the video attached)
=> Video is paused
- Touch again the screen to resume the video streaming

[BUG]
=> Video is not resumed (Please check the video attached)
=> video is played for less than a second then paused again

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

This sounds like a bug in the YouTube app. We do not have any control over Google's design.

Comment 2

[Karen Rudnitski [:kar]]

Can we get confirmation that this is something we can't control? Would it be a webcompat topic then?

Comment 3

[Karen Rudnitski [:kar]]

Here is a video and logs:
https://www.dropbox.com/sh/tfa8mop5l8b5efu/AAABm4ru2rlyO0TrYXJnvDwRa?dl=0

Comment 4

[Roman R.]

I have this problem and more on Windows 8.

Comment 5

[Kevin Brosnan [Ex-Mozilla]]

This works for me, http://people.mozilla.org/~kbrosnan/tmp/1133677/youtube.mp4

Roman you should file a different bug. This is regarding Firefox for Android. It is not safe to assume that the issue you are seeing on Windows 8 is the same.

Comment 6

[u421692]

Same as bug 1185149 and bug 1160889 ?

Comment 7

[[e-mail]]

This is the one issue that is stopping us from being accepted by one potential partner.

I just tried it myself in Nightly, and it continues to happen. Sony Xperia Z1 running Lollipop

YouTube mobile website
YouTube app: uninstalled

This does not happen if you request the desktop site, but that's beside the issue.

Comment 8

[Mike Taylor [:miketaylr]]

I can't reproduce this right now, but I can't get any videos to play in Nightly.

I am getting mp4 segments served to me, but I just see a double spinner in the video.

Release, Beta and Aurora all pass off the video to the app.

Sample video/mp4 segment: <https://r1---sn-bg5oqxjvh-jg2e.googlevideo.com/videoplayback?keepalive=yes&pcm2cms=yes&itag=133&signature=B61AC6466E2C67EB0FCEEBDA981FAACAB743E173.C152EBA81F226F4BB28E865BEDCCB92E273ACCE9&requiressl=yes&key=yt6&expire=1445039102&gir=yes&ip=216.188.255.151&mm=31&mn=sn-bg5oqxjvh-jg2e&ms=au&mt=1445017456&sparams=clen%2Cdur%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cpcm2cms%2Cpl%2Crequiressl%2Csource%2Cupn%2Cexpire&mv=m&id=o-AAho6x-Dv9wrxRFUNVlZp2BBZAwlCbPDk5OicSFW8aj8&mime=video%2Fmp4&pl=23&sver=3&upn=07NBiZMstco&ipbits=0&lmt=1444896955677944&fexp=9407155%2C9407169%2C9408710%2C9412755%2C9416673%2C9416838%2C9418805%2C9420934%2C9421509%2C9421561%2C9422138%2C9422681&source=youtube&initcwndbps=2322500&dur=256.456&clen=7825312&cpn=bN3CGXJbFL9wCdO_&alr=yes&ratebypass=yes&c=MWEB&cver=html5&range=7501418-7825311&rn=37&rbuf=245579>

Snorp, can you take a quick look at Youtube in-browser video?

Comment 9

[Mike Taylor [:miketaylr]]

Francisco, can you do a quick sanity check on Nightly to let me know if video is even playing for you? Google was working on a fix for getting us in-browser HTML5 video a few weeks back: https://bugzilla.mozilla.org/show_bug.cgi?id=1174784#c18

Comment 10

[Mike Taylor [:miketaylr]]

Hallvord also sees the same problem that I do. According to https://github.com/webcompat/web-bugs/issues/1755, this has been happening since about 2 weeks ago.

Comment 11

[James Willcox (:snorp) ([e-mail]) (he/him)]

Yeah, video is busted on Nightly due to bug 1183788

Comment 12

[James Willcox (:snorp) ([e-mail]) (he/him)]

Err....bug 1211443

Comment 13

[Mike Taylor [:miketaylr]]

Thanks Snorp.

Comment 14

[Mike Taylor [:miketaylr]]

Confirmed once you disable the Youtube app. But I'd like to test this once Bug 1174784 is fixed (which depends on Bug 1211443 -- fix on inbound), because we get what looks like different players.

Comment 15

[[e-mail]]

Bug 1211443 fixed, Youtube videos working again in Nightly.

The "Video no resumed" issue of this Bug 1133677 continues.

Comment 16

[Mike Taylor [:miketaylr]]

Thanks for checking Francisco. Digging into this code is at the top of my TODO list today.

Comment 17

[Mike Taylor [:miketaylr]]

Francisco, can you verify one more thing for me?

go to about:config and set "general.useragent.updates.enabled" to false (this will disable the current UA override for YT).

After that, does the bug still reproduce? 

For me it works normally after that, which suggests we're not running into something event-y like Bug 1016480 but some kind of UA sniffing thing (and having the Android version in the UA string gets us down the right codepath).

Comment 18

[[e-mail]]

Hmm, what it does for me is that it's not possible to pause the video by tapping it anymore. Instead, it displays (when tapped) a more prominent pause button over the progress bar, which can be used to pause/resume without issues.

Comment 19

[[e-mail]]

oops, got trigger happy- without changing the "general.useragent.updates.enabled", you can pause/resume using the standard play button too. It's only when tapping the video that it doesn't work properly.

Comment 20

[Mike Taylor [:miketaylr]]

Attachment: Screenshot_2015-10-21-14-29-48.png

OK, if what you're seeing is the same as this screenshot (and not a fullscreen www.youtube.com player), then I think we're getting the right behavior with the fix from 1174784 and the UA override removed.

With the content playing in-line (and not in the Desktop-like fullscreen player) the only way to stop/start the video is by interacting with the native video controls. This is the same way that Chrome Mobile behaves (they just have nicer looking native controls IMO).

Comment 21

[Mike Taylor [:miketaylr]]

Attachment: old-youtube-with-app-disabled.png

Here's the screenshot of the "old" player that you get with the app disabled (which has the bug). We should get the player (without the bug) in the other screenshot once the Youtube fix is deployed 100%.

Comment 22

[[e-mail]]

Yes, even if I go fullscreen, the player looks like that, and it's a good solution.

I had several questions, but I ran some tests, and now I only have one:
When will this hit production? Until 44 or sooner?

Comment 23

[Mike Taylor [:miketaylr]]

(In reply to [e-mail] from comment #22)
> I had several questions, but I ran some tests, and now I only have one:
> When will this hit production? Until 44 or sooner?

I guess that's up to Youtube. Once we know the fix is deployed everywhere (supposedly this week? next?), we can land the patch in Bug 1217055. Approximately 24 hours after that hits moz-central it should be updated for our users (on 42 and up*).

*42 is the release that (re-)enabled the dynamic UA override stuff (cf. Bug 1178760).

Comment 24

[[e-mail]]

I tested this over the weekend after receiving a Beta (Fx42) update, and it works as advertised!

Comment 25

[Mike Taylor [:miketaylr]]

Yay!

Comment 26

[[e-mail]]

I tried this again, in both Release and Beta, prior to sending to the partner, and the behavior changed. On first tap, the controls appear, I pause by pressing the pause button, the controls remain, I press play, and playing does not resume, instead, the controls disappear, and tapping again on the video resumes the video... Tapping on the video while paused and with the controls visible resumes too.

Comment 27

[[e-mail]]

Any thoughts on comment 26 Mike? I just tried it again after the latest Beta upgrade, no change.

Comment 28

[Mike Taylor [:miketaylr]]

Hey Francisco, 

(very sorry for delay -- was traveling for a conf then sick).

Yep, I can confirm the behavior you're describing. I feel like we should close this bug and open a new one, given that there are new STR. I'll do that now and link it from here.