821513 - web intent of vnd.youtube is not playing in the video app

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Browser, defect, P2)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

1. launch browser
2. goto vnd.youtube:e_qHyj9-CWs

Expected: video app launches and plays the video from youtube
Actual: 
Does not play : 
12-13 15:15:25.338: E/GeckoConsole(1475): [JavaScript Error: "[Exception... "'Illegal value' when calling method: [nsIProtocolHandler::newChannel]"  nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"  location: "native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0"  data: no]"]

Comment 1

[Jason Smith [:jsmith]]

Why is this not expected behavior? You've given an invalid value to the android intent and NS_ERROR got thrown. I don't think there's anything wrong here.

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

vnd.youtube is a youtube specific intent that launches the youtube app on android.  vnd.youtube should launch the video app and play the video.

Comment 3

[Jason Smith [:jsmith]]

(In reply to Naoki Hirata :nhirata from comment #2)
> vnd.youtube is a youtube specific intent that launches the youtube app on
> android.  vnd.youtube should launch the video app and play the video.

Perhaps. But if the value given is wrong is the intent, it should fail, no?

Fabrice - What do you think?

Comment 4

[[:fabrice] Fabrice Desré]

Wait, is this android or b2g? On b2g we support the vnd.youtube scheme, since this is what we use for youtube support.

Comment 5

[Jason Smith [:jsmith]]

(In reply to Fabrice Desré [:fabrice] from comment #4)
> Wait, is this android or b2g? On b2g we support the vnd.youtube scheme,
> since this is what we use for youtube support.

b2g. Naoki is testing this through the browser directly by going to a vnd.youtube URL.

Comment 6

[Joe Cheng [:jcheng] (please needinfo)]

Triage: where can users get such address from? vnd.youtube:e_qHyj9-CWs
is this a common usage?

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

https://bugzilla.mozilla.org/show_bug.cgi?id=777633

Youtube itself.

Comment 8

[David Scravaglieri [:scravag]]

This address can't be used on a desktop browser. Please and renominate if needed.

Comment 9

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

B2G browser convert the string to http://vnd.youtube:e_qHyj9-CWs, so the problem is on Gaia Browser.

Comment 10

[Ben Francis [:benfrancis]]

Is this a v1 requirement?

Comment 11

[Jason Smith [:jsmith]]

(In reply to Ben Francis [:benfrancis] from comment #10)
> Is this a v1 requirement?

If it's a valid youtube vnd.youtube protocol, yes. See bug 777633 that goes into lots of detail on this one.

Comment 12

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

I think what Ben want to ask is that, is "being able to type the fake protocol into the browser awesome bar" a v1 requirement?

We could already play the YouTube video w/o problem when user press the YouTube site play button, with bug 777633.

Comment 13

[Jason Smith [:jsmith]]

(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) from comment #12)
> I think what Ben want to ask is that, is "being able to type the fake
> protocol into the browser awesome bar" a v1 requirement?
> 
> We could already play the YouTube video w/o problem when user press the
> YouTube site play button, with bug 777633.

Oh. Well in that case, yeah, I'd say this is an extreme nice to have at best.

Comment 14

[David Scravaglieri [:scravag]]

Test in the following link the "Test Video 1"

http://people.mozilla.com/~nhirata/html_tp/AndroidLinks.html

Comment 15

[Alex Keybl [:akeybl]]

This is in the Browser component (comment 9). If this impacts other apps/user scenarios, please re-nominate.

Comment 16

[Ben Francis [:benfrancis]]

I don't think we should build support for the fake vnd.youtube: protocol into the browser app to allow people to type it as a URL. I think we already support the intended use cases through bug 777633

Please reopen if this is a problem.

Comment 17

[Jason Smith [:jsmith]]

FWIW, YouTube in fact said they don't like the fact we are using the vnd protocol. We've got orders to move away from this model asap in the closest release possible (hopeful for 1.1).