Closed Bug 631873 Opened 13 years ago Closed 10 years ago

Firefox Mobile for Android should skip loading unsupported source in multiple <source> tags

Categories

(Web Compatibility :: Site Reports, defect, P2)

ARM
Android

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: petercpg, Unassigned)

References

()

Details

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b12pre) Gecko/20110205 Firefox/4.0b12pre
Build Identifier: Mozilla/5.0 (Android; Linux armv7l; rv:2.0b11pre) Gecko/20110126 Firefox/4.0b11pre Fennec/4.0b4

On Android, when Firefox is loading a html5 video page with multiple formats, it loads in the sequence of <source>, but if it doesn't support any of the sources, Firefox won't ignore that source.

Reproducible: Always

Steps to Reproduce:
1. Open http://test.netix.com.tw/video/index.html in Firefox
2. Tap the play button
Actual Results:  
Firefox Mobile couldn't play the video; it shows loading MP4 source after tapping the video area.

Expected Results:  
It should skip the MP4 source and load OGV or WebM.

When loading the testcase, Firefox 4.0b10 plays the WebM source and Fennec 4.0b5pre plays the OGV source correctly on Windows.

When playing different html files with single <source> in <video>, Fennec plays WebM and OGV on Android.
OS: Linux → Android
Hardware: Other → ARM
Summary: Firefox Mobile for Android should skip unspported source in multiple <source> tags → Firefox Mobile for Android should skip loading unsupported source in multiple <source> tags
Confirmed on Mozilla/5.0 (Android; Linux armv71; rv:2.0b12pre) Gecko/20110206 Firefox/4.0b12pre Fennec/4.0b5pre

Console:
Error in parsing value for 'background'.  Declaration dropped.
http://test.netix.com.tw/video/video-js/video-js.css
65
Status: UNCONFIRMED → NEW
Ever confirmed: true
tracking-fennec: --- → ?
Priority: -- → P2
Assignee: nobody → doug.turner
tracking-fennec: ? → 2.0+
The bug is with video.js, basically the user agent parsing is wrong for Mozilla Firefox on Android.


      if (VideoJS.isAndroid()) {
        if (ext == "mp4" || ext == "m4v") { return "maybe"; }


we correctly match isAndroid(), but we do not support either of those formats.

I tried to figure out how to create a bug on the videojs site, but couldn't figure it out.

tantek - you're quoted on the site.  Can you see what we can to do get Firefox on Android supported?
Assignee: doug.turner → english-us
tracking-fennec: 2.0+ → ---
Component: General → English US
Product: Fennec → Tech Evangelism
QA Contact: general → english-us
Version: Trunk → unspecified
Peter, is Firefox loading the mp4 source? If so that's a separate bug
Doug, I've emailed one of the VideoJS guys about your comment (with link to here).
blassey, take a look at the video.js code.  the video element uses video.js (which changes the default behavior).  If you remember the js file, it works as expected.

tantek, cool!  let me know if they need any help.

Peter, thanks for the bug report!  We are going to follow up with the authors of the video js lib and see if we can get it fixed.
blassey, s/remember/remove
VideoJS changelog shows no activity in 2011: https://github.com/zencoder/video-js
Is there any new planned VideoJS version, at least to fix this bug?
Why it's work on emulator (https://ftp.mozilla.org/pub/mozilla.org/mobile/releases/5.0/linux-i686/fennec-5.0.en-US.linux-i686.tar.bz2) and not in a real device like Google Nexus S ?
VideoJs looks for strings in the useragent. In this case, when it sees "Android" it forces the video to use h.264. Fennec only has Android in its UA string when it is running on an Android device (on Linux it should have "Linux").
@ Wesley Johnston: Thanks you. 

With https://addons.mozilla.org/fr/mobile/addon/phony/ and choose Desktop firefox, it's work :-)
A videojs 3.0 Beta is released:
https://github.com/zencoder/video-js
The home page of http://videojs.com/ change. Version 3.0 is here?

This page works now on Firefox mobile 9 on Android 4.0.3. It's lag a little ;-) (similar on FF 10 beta)
With Aurora (FF 11 on desktop), we have got full-screen bouton and it's work too.

\o/
According to comment #13 this is fixed
Assignee: english-us → nobody
Status: NEW → RESOLVED
Closed: 10 years ago
Component: English US → Mobile
Resolution: --- → FIXED
Product: Tech Evangelism → Web Compatibility
Component: Mobile → Site Reports
You need to log in before you can comment on or make changes to this bug.