diff options
author | Joas Schilling <213943+nickvergessen@users.noreply.github.com> | 2021-08-27 16:56:09 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-27 16:56:09 +0300 |
commit | ead7451aaa03745c5b69f29fd180262a264e30ba (patch) | |
tree | ed838bc44a73f8af6fbea10eac548d2296536da0 /src | |
parent | 55442f98175e395ee6dd926240d5c9b1e14091c8 (diff) | |
parent | 7b4646e5d552b5647a54deaab5b16dc0c88aed7d (diff) |
Merge pull request #6153 from nextcloud/fix-laggy-high-resolution-videos-in-chromium
Fix laggy high resolution videos in Chromium
Diffstat (limited to 'src')
-rw-r--r-- | src/utils/webrtc/simplewebrtc/localmedia.js | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/utils/webrtc/simplewebrtc/localmedia.js b/src/utils/webrtc/simplewebrtc/localmedia.js index 3ab6485d2..5a293fa7a 100644 --- a/src/utils/webrtc/simplewebrtc/localmedia.js +++ b/src/utils/webrtc/simplewebrtc/localmedia.js @@ -152,6 +152,12 @@ LocalMedia.prototype.isLocalMediaActive = function() { * resolution, so if the camera does not have such resolution it will still * return the highest resolution available without failing. * + * A high frame rate needs to be requested too, as some cameras offer high + * resolution but with low frame rates, so Chromium could end providing a laggy + * high resolution video. If the frame rate is requested too then Chromium needs + * to balance all the constraints and thus provide a video without the highest + * resolution but with an acceptable frame rate. + * * @param {object} constraints the constraints to be adjusted */ LocalMedia.prototype._adjustVideoConstraintsForChromium = function(constraints) { @@ -177,6 +183,7 @@ LocalMedia.prototype._adjustVideoConstraintsForChromium = function(constraints) constraints.video.width = 1920 constraints.video.height = 1200 + constraints.video.frameRate = 60 } LocalMedia.prototype.start = function(mediaConstraints, cb, context) { |