Video.js: Live ui HLS stream crashes in chrome when running in the background

Created on 3 Jul 2019  路  7Comments  路  Source: videojs/video.js

Description

Leaving the player running a live HLS stream (with liveUI enabled) in Chrome in a tab that is not the currently focused tab, causes the page to crash after about an hour. The javascript memory used in this time increases by over 400%. This doesn't seem to happen when the tab is focused.
This can be seen using the LiveUI sandbox test in the Video.js git repo.

Steps to reproduce

  1. clone the repository, and move into that directory
  2. run npm install or npm ci to install all necessary packages
  3. run npm start to start the local server
  4. open http://localhost:9999/sandbox/liveui.html in Chrome
  5. remove focus from that tab, either by minimising the window or opening another tab
  6. after 1 hour open/focus the LiveUI tab

Results

Expected

Page is responsive and has continued playing

Actual

Presented with Chrome's 'Page Unresponsive' modal. Unable to use the page.

Error output

Additional Information

Please include any additional information necessary here. Including the following:

versions

videojs

7.5.4

browsers

tested on:
Chrome 73.0.3683.103, 74.0.3729.131, 75.0.3770.100

OSes

Windows 10 on Surface Go and Lenovo IdeaPad U430 touch

plugins

None
2019-07-01 (1)

confirmed

All 7 comments

馃憢 Thanks for opening your first issue here! 馃憢

If you're reporting a 馃悶 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.
To help make it easier for us to investigate your issue, please follow the contributing guidelines.

Thanks! We are aware and it's likely related to #5937, we have time scheduled for addressing this, this quarter.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Having same issue (v7.5.5), can confirm.

After looking into the issue, this is definitely a duplicate of https://github.com/videojs/video.js/issues/5937

PR https://github.com/videojs/video.js/pull/6621 will resolve this as well.

@matbcvo try upgrading to 7.6.6 (but not to 7.7.x as there was a regression) for the time being

@gkatsev the RAF fix for #5937 slated for the 7.9.0 should close this issue as well.

Video.js 7.9.0 is out as a pre-release, closing as fixed.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

gfviegas picture gfviegas  路  3Comments

kocoten1992 picture kocoten1992  路  4Comments

borm picture borm  路  3Comments

cshah123 picture cshah123  路  4Comments

onigetoc picture onigetoc  路  4Comments