Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Side by Side Diff: webrtc/api/call/audio_send_stream.h

Issue 2405183002: Moving WebRtcVoiceMediaChannel::SendSetCodec to AudioSendStream. (Closed)
Patch Set: final change Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | webrtc/audio/audio_send_stream.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 // Transport for outgoing packets. The transport is expected to exist for 76 // Transport for outgoing packets. The transport is expected to exist for
77 // the entire life of the AudioSendStream and is owned by the API client. 77 // the entire life of the AudioSendStream and is owned by the API client.
78 Transport* send_transport = nullptr; 78 Transport* send_transport = nullptr;
79 79
80 // Underlying VoiceEngine handle, used to map AudioSendStream to lower-level 80 // Underlying VoiceEngine handle, used to map AudioSendStream to lower-level
81 // components. 81 // components.
82 // TODO(solenberg): Remove when VoiceEngine channels are created outside 82 // TODO(solenberg): Remove when VoiceEngine channels are created outside
83 // of Call. 83 // of Call.
84 int voe_channel_id = -1; 84 int voe_channel_id = -1;
85 85
86 // Ownership of the encoder object is transferred to Call when the config is
87 // passed to Call::CreateAudioSendStream().
88 // TODO(solenberg): Implement, once we configure codecs through the new API.
89 // std::unique_ptr<AudioEncoder> encoder;
90 int cng_payload_type = -1; // pt, or -1 to disable Comfort Noise Generator.
91
92 // Bitrate limits used for variable audio bitrate streams. Set both to -1 to 86 // Bitrate limits used for variable audio bitrate streams. Set both to -1 to
93 // disable audio bitrate adaptation. 87 // disable audio bitrate adaptation.
94 // Note: This is still an experimental feature and not ready for real usage. 88 // Note: This is still an experimental feature and not ready for real usage.
95 int min_bitrate_kbps = -1; 89 int min_bitrate_kbps = -1;
96 int max_bitrate_kbps = -1; 90 int max_bitrate_kbps = -1;
91
92 struct SendCodecSpec {
93 SendCodecSpec() {
94 webrtc::CodecInst empty_inst = {0};
95 codec_inst = empty_inst;
96 codec_inst.pltype = -1;
97 }
98 bool operator==(const SendCodecSpec& rhs) const {
99 {
100 if (nack_enabled != rhs.nack_enabled) {
101 return false;
102 }
103 if (transport_cc_enabled != rhs.transport_cc_enabled) {
104 return false;
105 }
106 if (enable_codec_fec != rhs.enable_codec_fec) {
107 return false;
108 }
109 if (enable_opus_dtx != rhs.enable_opus_dtx) {
110 return false;
111 }
112 if (opus_max_playback_rate != rhs.opus_max_playback_rate) {
113 return false;
114 }
115 if (cng_payload_type != rhs.cng_payload_type) {
116 return false;
117 }
118 if (cng_plfreq != rhs.cng_plfreq) {
119 return false;
120 }
121 if (codec_inst != rhs.codec_inst) {
122 return false;
123 }
124 return true;
125 }
126 }
127 bool operator!=(const SendCodecSpec& rhs) const {
128 return !(*this == rhs);
129 }
130
131 bool nack_enabled = false;
132 bool transport_cc_enabled = false;
133 bool enable_codec_fec = false;
134 bool enable_opus_dtx = false;
135 int opus_max_playback_rate = 0;
136 int cng_payload_type = -1;
137 int cng_plfreq = -1;
138 webrtc::CodecInst codec_inst;
139 } send_codec_spec;
97 }; 140 };
98 141
99 // Starts stream activity. 142 // Starts stream activity.
100 // When a stream is active, it can receive, process and deliver packets. 143 // When a stream is active, it can receive, process and deliver packets.
101 virtual void Start() = 0; 144 virtual void Start() = 0;
102 // Stops stream activity. 145 // Stops stream activity.
103 // When a stream is stopped, it can't receive, process or deliver packets. 146 // When a stream is stopped, it can't receive, process or deliver packets.
104 virtual void Stop() = 0; 147 virtual void Stop() = 0;
105 148
106 // TODO(solenberg): Make payload_type a config property instead. 149 // TODO(solenberg): Make payload_type a config property instead.
107 virtual bool SendTelephoneEvent(int payload_type, int event, 150 virtual bool SendTelephoneEvent(int payload_type, int event,
108 int duration_ms) = 0; 151 int duration_ms) = 0;
109 152
110 virtual void SetMuted(bool muted) = 0; 153 virtual void SetMuted(bool muted) = 0;
111 154
112 virtual Stats GetStats() const = 0; 155 virtual Stats GetStats() const = 0;
113 156
114 protected: 157 protected:
115 virtual ~AudioSendStream() {} 158 virtual ~AudioSendStream() {}
116 }; 159 };
117 } // namespace webrtc 160 } // namespace webrtc
118 161
119 #endif // WEBRTC_API_CALL_AUDIO_SEND_STREAM_H_ 162 #endif // WEBRTC_API_CALL_AUDIO_SEND_STREAM_H_
OLDNEW
« no previous file with comments | « no previous file | webrtc/audio/audio_send_stream.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698