Make sure to maintain errors on source tags after initialization #1575
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| // 1) Check if the source is new (if not, we want to keep the original so playback isn't interrupted) | ||
| // 2) Check to see if the network state of the tag was failed at init, and if so, reset the source | ||
| // anyway so the error gets fired. | ||
| if ((source && this.el_.currentSrc !== source.src) || (player.tag && player.tag.initNetworkState_ === 3)) { |
There was a problem hiding this comment.
Is there any way source could be undefined but player.tag.initNetworkState_ === 3? That doesn't seem possible but we still may want to check for source in the second part of this.
jgubman
added a commit
to jgubman/video.js
that referenced
this pull request
Oct 29, 2014
* 'stable' of https://github.com/videojs/video.js: (22 commits) Added line to changelog for 4.10.1 Release 4.10.1 Removed alert... Release 4.10.0 @heff turned on the custom html controls for touch devices. closes videojs#1617 updated options to include sans-children, and included self-hosted font blurb added CORS information to tracks guide. Fixes videojs#837 doc updates + readme quick start updated options guide to include components @heff Added the ability to set options for child components directly in the parent options. closes videojs#1599 @DevGavin added a Simplified Chinese translation. closes videojs#1593 @mmcc fixed an issue with the VolumeButton assuming it was vertical by default. closes videojs#1592 @heff enhanced the event listener API to allow for auto-cleanup of listeners on other componenets and elements. closes videojs#1588 @mmcc fixed an issue where errors on source tags could get missed. closes videojs#1575 Added doc for remaining time and removed onWaitEnd doc maybe actually check for keyLocation Update languages.md @heff updated the poster to use CSS styles to display; fixed the poster not showing if not originally set. closes videojs#1568 Release 4.9.1 Bumped to videojs-swf v4.5.1 to fix a data sanitization issue. closes videojs#1587 ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously if you included an empty source url, such as
<source src="" type="video/mp4">, the player would just show the loading spinner with no errors. This should take care of that (and close #869 again).