Closed
Bug 1302957
Opened 8 years ago
Closed 7 years ago
Vimeo video content not playable without Fullscreen API support
Categories
(Web Compatibility :: Site Reports, defect)
Tracking
(platform-rel +, firefox48 unaffected, firefox49 unaffected, firefox50 unaffected, firefox51+ wontfix, firefox52+ wontfix, firefox53 wontfix)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
platform-rel | --- | + |
firefox48 | --- | unaffected |
firefox49 | --- | unaffected |
firefox50 | --- | unaffected |
firefox51 | + | wontfix |
firefox52 | + | wontfix |
firefox53 | --- | wontfix |
People
(Reporter: JuliaC, Unassigned, NeedInfo)
References
()
Details
(Keywords: regression, Whiteboard: [needscontact][platform-rel-Vimeo])
[Affected versions]: - latest Nightly 51.0a1 (2016-09-14) [Affected platforms]: - Windows 10 x64 - Ubuntu 16.04 x86 - Mac OS X 10.11 [Steps to reproduce]: 1. Launch Firefox 2. Go to https://vimeo.com/ and open a random video (e.g. https://vimeo.com/181907337) 3. Bookmark the chosen video 4. Open the Bookmarks Menu, look for the previous created bookmark and open its Properties (context menu option) 5. Check the "Load this bookmark in the sidebar" option and click the "Save" button 6. Open again the Bookmarks Menu and click the bookmark in question [Expected result]: - The chosen video item is properly loaded in the sidebar and no rendering issues are encountered [Actual result]: - The chosen video item is not loaded if it is open in the sidebar [Regression range]: * Last good revision: eb90834c4f107ebdeb369702d66e5257ff352d36 * First bad revision: 5aabb19fcdfe384ced77636f8556d3488e37d51f * Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=eb90834c4f107ebdeb369702d66e5257ff352d36&tochange=5aabb19fcdfe384ced77636f8556d3488e37d51f * Looks like the following bug has the changes which introduced the regression: https://bugzilla.mozilla.org/show_bug.cgi?id=1299795
Comment 1•8 years ago
|
||
Follow up on previous comment: "* Looks like the following bug has the changes which introduced the regression: https://bugzilla.mozilla.org/show_bug.cgi?id=1299795", I'm needinfo'ing the owner of that bug. @xidorn - can you take a look at this? Should we move this from Audio/Video:Playback to DOM?
Component: Audio/Video → Audio/Video: Playback
Flags: needinfo?(xidorn+moz)
Comment 2•8 years ago
|
||
It seems Vimeo video doesn't work if the Fullscreen API is disabled. If you set pref "full-screen-api.enabled" to false, you'll find that Vimeo video would not work even it is not in sidebar. I think it is something Vimeo should fix, because there is no guarantee that Fullscreen API would always be available. Users may want to disable Fullscreen API for e.g. protecting their privacy. So moving to Tech Evangelism.
Component: Audio/Video: Playback → Desktop
Flags: needinfo?(xidorn+moz)
Product: Core → Tech Evangelism
Version: 51 Branch → Firefox 51
status-firefox48:
--- → unaffected
Comment 3•8 years ago
|
||
I think the issue is in `_initAssets` and `_initPlayer` @ https://f.vimeocdn.com/js_opt/utils/player_manager/player_manager.min.js?55ab7083 function _initJunk() { if (usePlayer) { doStuff() return; } if (useFlash) { doStuff() return } } usePlayer looks like: usePlayer = fullscreenSupport || IE10 && !flashSupport || windowsPhone; And fullscreenSupport like so: fullscreenSupport = document.fullscreenEnabled || document.webkitFullscreenEnabled || document.mozFullScreenEnabled || document.msFullscreenEnabled || videoFullscreenSupport; So there's an early return and the player is never set up because `usePlayer` is never true w/o fullscreen API support.
Summary: Vimeo video content is not loaded/rendered in the sidebar → Vimeo video content not playable without Fullscreen API support
Whiteboard: [needscontact]
Updated•8 years ago
|
Flags: needinfo?(astevenson)
Comment 4•8 years ago
|
||
Adam, do we have any Vimeo contacts?
Comment 5•8 years ago
|
||
We do. Andrew, could you help us on this one?
Flags: needinfo?(astevenson) → needinfo?(andrew)
(In reply to Mike Taylor [:miketaylr] from comment #4) > Adam, do we have any Vimeo contacts? Let's have a go via the discussion mailing list?
platform-rel: --- → ?
Whiteboard: [needscontact] → [needscontact][platform-rel-Vimeo]
Updated•8 years ago
|
platform-rel: ? → +
Comment 7•8 years ago
|
||
Benjamin, did you have any insight here? Sounds like may be something to do with falling back to Flash which we may not support for fullscreen in the sidebar.
status-firefox52:
--- → affected
tracking-firefox51:
--- → +
tracking-firefox52:
--- → +
Flags: needinfo?(benjamin)
Comment 8•8 years ago
|
||
(to follow up, Overholt started a thread on our vimeo ML -- there is a question for bsmedberg to answer regarding flash support)
Comment 9•8 years ago
|
||
Liz, I think comment 3 captures the problem here so I don't know what other information I can provide.
Flags: needinfo?(benjamin)
Updated•8 years ago
|
Updated•7 years ago
|
Comment 10•7 years ago
|
||
Too late for a fix for 53, as we are in the last week of the 53 beta cycle.
Comment 11•7 years ago
|
||
I'm unable to reproduce this issue in Firefox 55 for OSX and Windows 10. The video takes a second to actually play in Windows, but it does work. Without a screenshot I have nothing to compare to, but since the video is playing I think this is fixed. Please reopen if I'm missing something here. Thanks!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Assignee | ||
Updated•5 years ago
|
Product: Tech Evangelism → Web Compatibility
You need to log in
before you can comment on or make changes to this bug.
Description
•