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

Fork update #3

Merged
merged 155 commits into from
Oct 7, 2019
Merged

Fork update #3

merged 155 commits into from
Oct 7, 2019

Conversation

desidiver
Copy link

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

  • changes have been done against master branch, and PR does not conflict
  • new unit / functional tests have been added (whenever applicable)
  • API or design changes are documented in API.md

itsjamie and others added 30 commits February 2, 2019 08:45
* 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
…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
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
…cript

* Playlist-loader rename.

* Playlist Loader conversion to TypeScript.

* Change responseType to string

* Typo fix for destory -> destroy.

* Change networkDetails type to "unknown".
* 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.
tjenkinson and others added 26 commits August 29, 2019 20:26
because it's not needed locally, and safer without it
This reverts commit 382bda2.

I was getting confused with --no-sandbox
…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
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`
@desidiver desidiver requested a review from gar7and October 7, 2019 16:39
Copy link

@gar7and gar7and left a 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

@desidiver desidiver merged commit aabb5ae into master Oct 7, 2019
@desidiver desidiver deleted the fork-update branch October 7, 2019 16:57
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.