Skip to content
This repository has been archived by the owner on Mar 22, 2022. It is now read-only.

Add split audio track source #377

Merged
merged 7 commits into from
May 26, 2020

Conversation

djee-ms
Copy link
Member

@djee-ms djee-ms commented May 25, 2020

Split the audio track source from the audio track, to allow sharing
across peer connections.

This change splits the former local audio track into 2 separate
components:

  • The audio track source, which generates the raw audio frames,
    generally from a local audio capture device (microphone) but not
    necessarily. At this time only device-based sources are supported
    though. The audio track source is not associated with any peer
    connection in particular, and can be shared amongst any number of
    local audio tracks, from the same peer connection or not.
  • The local audio track, which is associated with a particular peer
    connection, and which is a slim bridge between an audio track source
    and the audio transceiver of the peer connection.

This change adds support to the native and C# libraries only.

Split the audio track source from the audio track, to allow sharing
across peer connections.

This change splits the former local audio track into 2 separate
components:
- The audio track source, which generates the raw audio frames,
  generally from a local audio capture device (microphone) but not
  necessarily. At this time only device-based sources are supported
  though. The audio track source is not associated with any peer
  connection in particular, and can be shared amongst any number of
  local audio tracks, from the same peer connection or not.
- The local audio track, which is associated with a particular peer
  connection, and which is a slim bridge between an audio track source
  and the audio transceiver of the peer connection.
@djee-ms djee-ms added enhancement New feature or request breaking change This change breaks the current API and requires migration labels May 25, 2020
@djee-ms djee-ms requested a review from fibann May 25, 2020 12:11
@djee-ms djee-ms self-assigned this May 25, 2020
Comment on lines 44 to 45
// TODO(tommi): This method should be on the track and ideally volume should
// be applied in the track in a way that does not affect clones of the track.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove TODO (also below)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@djee-ms djee-ms merged commit 1d0c783 into microsoft:merge/split_source May 26, 2020
@djee-ms djee-ms deleted the split_source_audio branch May 26, 2020 09:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking change This change breaks the current API and requires migration enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants