forked from video-dev/hls.js
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fork update #3
Merged
Merged
Fork update #3
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
* Upgrade to webpack 4, babel 7 * Use straight git link for webworkify dep * Use simpler git url * Re-add globalObject workaround * Bump karma dep versions * Dont preprocess non-test files * Use @babel/register over ts-node * Update demo `const` usage. * Use webpack debug config in Karma configuration rather than duplication.
* remove events/EventEmitter && move Observer to ts * disable node modules in webpack
…com image link to new domain and using https (video-dev#2127)
…code (misc) Removing IE8-specific (dead) vtt/cue code
* Reduce GC pressure from ID3._utf8ArrayToStr If the TextDecoder API exists, it is significantly faster for large payloads than the existing code. This work happens on the main thread, so large payloads may cause UI hiccups. On my Chrome70 profiler runs, most of the time was actually being spent in GC.
…ideo-dev#2117) * video-devGH-2082: Replacing usage of Array.prototype.find and Array.protoype.findIndex with alternate solutions * video-devGH-2082: Adding eslint rule for restricting usage of findIndex/find methods
Basic typescript of event handler.
Per feedback from John, the further defined response type caused issue in callers.
…rack.len (video-dev#2096) * mp4-remuxer / remuxAudio() : dont rely on track.len to compute mdatSize recompute it by looping through all samples and summing their individual size get rid of track.len as it should now be useless related to video-dev#2063
…js into playlist-loader-typescript
…cript * Playlist-loader rename. * Playlist Loader conversion to TypeScript. * Change responseType to string * Typo fix for destory -> destroy. * Change networkDetails type to "unknown".
…mport preprocess.
Fixed some typos
* Start and stop subtitle loading in sync with startLoad()/stopLoad() API calls * Compute sliding & merge subtitle playlists during live playback * Fix shared references between subtitle-track-controller and subtitle-stream-controller * Refactoring: * Extend subtitle-stream-controller from base-stream-controller * Move more shared logic into base-stream-controller * Move live reload interval calculation into a shared function & import into subtitle-track-controller * Remove the trackId from subtitle fragments, and refer to them by their level property * Several other minor refactorings (flipped conditionals, destructuring args, organizing functions) * Add a bunch of unit tests
Fix XSS issue on metrics page reported in video-dev#2161.
because it's not needed locally, and safer without it
…-timeout Increase before each timeout
This reverts commit 382bda2. I was getting confused with --no-sandbox
Update functional tests to ES6.
…in delta computation (video-dev#2354) Remove a misplaced scale normalization in a timestamp delta compuation in the mp4-remuxer when checking for sample gaps and overlapping.e
because there is so much that it causes the end of the logs to be truncated on netlify
so that errors aren't missed
…-bug Fix bad conditional in key loader causing the wrong key to be loaded
…or live stream without media attached Fixes video-dev#2333
Less verbose webpack output on CI, and attempt to build docs on travis
…ithout-media-on-flush-backbuffer Bugfix: Uncaught exception trying to flush back buffer when starting live stream
because this is quicker, but also validates that the package-lock is in sync with the package.json Without this anything in the package.json that is out of sync with the package-lock gets silently updated and the integrity is not checked
use `npm ci` instead of `npm install`
Fix typo in API.md
gar7and
approved these changes
Oct 7, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved, we need to resync our hls fork
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.
This PR will... update the fork with the latest master from hls.js main repository.
Why is this Pull Request needed? To track and merge latest changes from hls.js
Are there any points in the code the reviewer needs to double check?
Resolves issues:
Checklist