Skip to content

Commit

Permalink
fix(JingleSession) Fix high fps screenshare not showing up.
Browse files Browse the repository at this point in the history
  • Loading branch information
jallamsetty1 committed Jan 13, 2025
1 parent 62ba126 commit 239e062
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
7 changes: 6 additions & 1 deletion modules/xmpp/JingleSessionPC.js
Original file line number Diff line number Diff line change
Expand Up @@ -562,7 +562,12 @@ export default class JingleSessionPC extends JingleSession {
const msid = $(source)
.find('>parameter[name="msid"]')
.attr('value');
const videoType = $(source).attr('videoType');
let videoType = $(source).attr('videoType');

// If the videoType is DESKTOP_HIGH_FPS for remote tracks, we should treat it as DESKTOP.
if (videoType === VideoType.DESKTOP_HIGH_FPS) {
videoType = VideoType.DESKTOP;
}

if (sourceDescription.has(sourceName)) {
sourceDescription.get(sourceName).ssrcList?.push(ssrc);
Expand Down
7 changes: 6 additions & 1 deletion service/RTC/VideoType.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,10 @@ export enum VideoType {
/**
* The desktop video type.
*/
DESKTOP = 'desktop'
DESKTOP = 'desktop',

/**
* The high fps desktop video type.
*/
DESKTOP_HIGH_FPS = 'desktop_high_fps'
}

0 comments on commit 239e062

Please sign in to comment.