Commit bc23b6f4 authored by Juliusz Chroboczek's avatar Juliusz Chroboczek

Disable simulcast for screen sharing.

Simulcast doesn't seem to work well with screen sharing: only one
layer is sent, which has very low throughput since we send the low
layer first.  Disable simulcast for screen sharing.
parent 3e554543
...@@ -230,9 +230,8 @@ ...@@ -230,9 +230,8 @@
<select id="requestselect" class="select select-inline"> <select id="requestselect" class="select select-inline">
<option value="">nothing</option> <option value="">nothing</option>
<option value="audio">audio only</option> <option value="audio">audio only</option>
<option value="screenshare-low">screen share (low)</option> <option value="screenshare">screenshare only</option>
<option value="screenshare">screen share</option> <option value="everything-low">low quality</option>
<option value="everything-low">everything (low)</option>
<option value="everything" selected>everything</option> <option value="everything" selected>everything</option>
</select> </select>
</form> </form>
......
...@@ -607,9 +607,6 @@ function mapRequest(what) { ...@@ -607,9 +607,6 @@ function mapRequest(what) {
case 'audio': case 'audio':
return {'': ['audio']}; return {'': ['audio']};
break; break;
case 'screenshare-low':
return {screenshare: ['audio','video-low'], '': ['audio']};
break;
case 'screenshare': case 'screenshare':
return {screenshare: ['audio','video'], '': ['audio']}; return {screenshare: ['audio','video'], '': ['audio']};
break; break;
...@@ -1227,7 +1224,7 @@ function setUpStream(c, stream) { ...@@ -1227,7 +1224,7 @@ function setUpStream(c, stream) {
let bps = getMaxVideoThroughput(); let bps = getMaxVideoThroughput();
// Firefox doesn't like us setting the RID if we're not // Firefox doesn't like us setting the RID if we're not
// simulcasting. // simulcasting.
if(simulcast) { if(simulcast && c.label !== 'screenshare') {
encodings.push({ encodings.push({
rid: 'h', rid: 'h',
maxBitrate: bps || unlimitedRate, maxBitrate: bps || unlimitedRate,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment