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

Side by Side Diff: webrtc/voice_engine/channel.h

Issue 2886813002: Delete RtpData::OnRecoveredPacket, use RecoveredPacketReceiver instead. (Closed)
Patch Set: Rebased. Created 3 years, 7 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 | « webrtc/video/rtp_stream_receiver.cc ('k') | no next file » | 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) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 298 matching lines...) Expand 10 before | Expand all | Expand 10 after
309 uint8_t payloadType, 309 uint8_t payloadType,
310 uint32_t timeStamp, 310 uint32_t timeStamp,
311 const uint8_t* payloadData, 311 const uint8_t* payloadData,
312 size_t payloadSize, 312 size_t payloadSize,
313 const RTPFragmentationHeader* fragmentation) override; 313 const RTPFragmentationHeader* fragmentation) override;
314 314
315 // From RtpData in the RTP/RTCP module 315 // From RtpData in the RTP/RTCP module
316 int32_t OnReceivedPayloadData(const uint8_t* payloadData, 316 int32_t OnReceivedPayloadData(const uint8_t* payloadData,
317 size_t payloadSize, 317 size_t payloadSize,
318 const WebRtcRTPHeader* rtpHeader) override; 318 const WebRtcRTPHeader* rtpHeader) override;
319 bool OnRecoveredPacket(const uint8_t* packet, size_t packet_length) override; 319
320 bool OnRecoveredPacket(const uint8_t* packet, size_t packet_length);
the sun 2017/05/29 14:00:31 IIUC this can be made private?
nisse-webrtc 2017/05/29 14:25:36 You're right. Done.
320 321
321 // From RtpFeedback in the RTP/RTCP module 322 // From RtpFeedback in the RTP/RTCP module
322 int32_t OnInitializeDecoder(int8_t payloadType, 323 int32_t OnInitializeDecoder(int8_t payloadType,
323 const char payloadName[RTP_PAYLOAD_NAME_SIZE], 324 const char payloadName[RTP_PAYLOAD_NAME_SIZE],
324 int frequency, 325 int frequency,
325 size_t channels, 326 size_t channels,
326 uint32_t rate) override; 327 uint32_t rate) override;
327 void OnIncomingSSRCChanged(uint32_t ssrc) override; 328 void OnIncomingSSRCChanged(uint32_t ssrc) override;
328 void OnIncomingCSRCChanged(uint32_t CSRC, bool added) override; 329 void OnIncomingCSRCChanged(uint32_t CSRC, bool added) override;
329 330
(...skipping 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
550 551
551 bool encoder_queue_is_active_ GUARDED_BY(encoder_queue_lock_) = false; 552 bool encoder_queue_is_active_ GUARDED_BY(encoder_queue_lock_) = false;
552 553
553 rtc::TaskQueue* encoder_queue_ = nullptr; 554 rtc::TaskQueue* encoder_queue_ = nullptr;
554 }; 555 };
555 556
556 } // namespace voe 557 } // namespace voe
557 } // namespace webrtc 558 } // namespace webrtc
558 559
559 #endif // WEBRTC_VOICE_ENGINE_CHANNEL_H_ 560 #endif // WEBRTC_VOICE_ENGINE_CHANNEL_H_
OLDNEW
« no previous file with comments | « webrtc/video/rtp_stream_receiver.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698