Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chromium Web Browser Support #1020

Merged
merged 1 commit into from
Jul 8, 2014
Merged

Conversation

ryokenau
Copy link
Contributor

I'm new to git/github, so hope what I'm doing is right...

This adds detection for the Chromium web browser, which only contains open-source video codecs (i.e. WebM and Theora). Unlike Chrome, it has no native support for MP4 files.

This is also related to a 3 year old (unresolved) issue, "Chrome vs Chromium video source confusion" #62.

Feel free to improve the code, as I'm not much of the "programmer type".

johndyer added a commit that referenced this pull request Jul 8, 2014
Chromium Web Browser Support
@johndyer johndyer merged commit b7f82b4 into mediaelement:master Jul 8, 2014
@programmin1
Copy link

Please revert this change - Chromium absolutely has MP4, mp3 support, at least in Ubuntu (where Chromium is quite popular).

Test case - open a site running WP 4.0 RC, in Ubuntu's Chromium. No audio/video except ogg will play.

@programmin1
Copy link

There has been discussion on a ticket for Wordpress where this was brought up - WP uses a patched version that doesn't have an exception the breaks Chromium: https://core.trac.wordpress.org/ticket/29110

Can we please have this added to the base MeJS as well?

ale-grosselle pushed a commit to ale-grosselle/mediaelement that referenced this pull request Nov 25, 2016
@Luke-SF
Copy link

Luke-SF commented Mar 30, 2017

Still not fixed in the version in latest Wordpress... any eta on when this can be removed?

@rafa8626
Copy link
Contributor

I added a section to use the latest version of the plugin with Wordpress while this happens. I haven't had an answer about this issue yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants