Change "video" to "media" in error messages.

[sirlancelot]

Fixes #2407

[pam]

Tests failed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: BUSTED

Commit: 08ae1cd
Build details: https://travis-ci.org/pam/video.js/builds/72629754

(Please note that this is a fully automated comment.)

[sirlancelot]

Looks like the failed test has nothing to do with this change in particular: Build Log Line 566

[gkatsev]

@pam retry

[gkatsev]

Yep, test failure is saucelabs being flaky

[pam]

Tests failed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: BUSTED

Commit: 08ae1cd
Build details: https://travis-ci.org/pam/video.js/builds/72633146

(Please note that this is a fully automated comment.)

[mister-ben]

All of the files in lang would also need to be updated. They use the English error as keys.

[heff]

We should probably just pull this in and let the translations come in as they can. We could go back and ping the original translators too.

[pam]

Tests failed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: BUSTED

Commit: 9eab598
Build details: https://travis-ci.org/pam/video.js/builds/72733512

(Please note that this is a fully automated comment.)

[dmlap]

@heff I believe mister-ben is pointing out that the translations won't work at all because the English keys won't match. It makes sense to let the values get updated independently but are you sure we should pull this in without updating the keys in the translations?

[gkatsev]

All the keys were updated in the language files.

[sirlancelot]

I updated the keys in 9eab598 as seen right above @pam's comment. Unfortunately, I don't speak the 25 languages to update their values ;)

[dmlap]

Oops. That's what I get for trying to do a code review from my phone. Nevermind my comment. LGTM.