1190831 - [Tablet] Youtube app is not triggered when trying to play a youtube video

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[u421692]

Attachment: logs

Device: Sony Xperia Z2 (Android 5.1)
Build: Nightly 42.0a1 (2015-08-03)

1. Open youtube.com
2. Tap on a video you chose to play
3. Tap to play the video

Expected result:
If youtube app in installed and enabled, it should open and play the video
If youtube app in not installed or is disabled, it should open an embedded video in Firefox

Actual result:
Android "open with" dialog is displayed, youtube app is not present on that list. 
If you tap on the helper app icon, youtube app is triggered and video plays without any issues.

I was only able to reproduce this on tablets, might be a UA problem? 

Please see attached logs:

08-04 16:09:32.805: W/ActivityManager(861): getTasks: caller 10280 does not hold GET_TASKS; limiting output
08-04 16:09:32.830: I/ActivityManager(861): START u0 {act=android.intent.action.VIEW dat=rtsp://r6---sn-4g57kued.googlevideo.com/ClULENy73wIaTAmEIZTMqRL2_BMYDSANFC17ucBVMOCoAUIJbXYtZ29vZ2xlSARSBXdhdGNoYLjQz-Kywd7fVYIBBWh0bWw1igELS1hQd1doVE93dWMM/1DAB8820C532141461B277AB03E58562D94CBE81.87CDB8B57EE34F3E1092BFE52D29973FA9E33521/yt5/1/video.3gp flg=0x4000000 cmp=android/com.android.internal.app.ResolverActivity (has extras)} from uid 10204 on display 0

Maybe it tries to open a 3gp file? in that case bug 1179532 ?

Comment 1

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

NI self to check if this is my fault (since I changed the intent handling code recently).

Comment 2

[Mike Taylor [:miketaylr]]

In case this is somehow related to UA string changes, -- Google is working on some changes for us so we don't need UA overrides. From one of their engineers,

> For the tablet-UA-with-version-number, the default stream will be an MP4 container format 
> and H264 video coding, which is supported.

So that means in-browser video should work, at least on Tablet. I'll check if we can get this now and write a patch to tweak the override there.

> In the meantime, I'll update m.youtube.com to have identical behavior with and without the 
> the version number in the UA, namely to redirect to the native app.  It will take a little 
> more work to serve the correct streams for the phone-UA-with-version-number, but at least 
> you won't be forced to override the UA until that is fixed.

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Mihai, does this affect 40 (and if not, 41)?

If this is regressed due to the intent handling changes, I'd expect it to be a regression from bug 851693 or one of its follow-ups, which landed in 41.

Comment 4

[u421692]

This issue does note reproduce anymore, tested on Sony Xperia Z2 (Android 5.1), on all branches. Maybe it has something to do with the Google changes(see comment 2).
Tested with Youtube app enabled and disabled and in both cases it works fine.
When youtube app is enabled, video is launched in the youtube app.
When youtube app is disabled, video is launched embed in the browser.

Will close this for now.