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

Side by Side Diff: webrtc/video/audio_receive_stream.cc

Issue 1226123005: Define Stream base classes (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Code review follow-up 3 Created 5 years, 5 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/audio_receive_stream.h ('k') | webrtc/video/end_to_end_tests.cc » ('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 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 } else { 60 } else {
61 RTC_NOTREACHED() << "Unsupported RTP extension."; 61 RTC_NOTREACHED() << "Unsupported RTP extension.";
62 } 62 }
63 } 63 }
64 } 64 }
65 65
66 webrtc::AudioReceiveStream::Stats AudioReceiveStream::GetStats() const { 66 webrtc::AudioReceiveStream::Stats AudioReceiveStream::GetStats() const {
67 return webrtc::AudioReceiveStream::Stats(); 67 return webrtc::AudioReceiveStream::Stats();
68 } 68 }
69 69
70 void AudioReceiveStream::Start() {
71 }
72
73 void AudioReceiveStream::Stop() {
74 }
75
76 void AudioReceiveStream::SignalNetworkState(NetworkState state) {
77 }
78
70 bool AudioReceiveStream::DeliverRtcp(const uint8_t* packet, size_t length) { 79 bool AudioReceiveStream::DeliverRtcp(const uint8_t* packet, size_t length) {
71 return false; 80 return false;
72 } 81 }
73 82
74 bool AudioReceiveStream::DeliverRtp(const uint8_t* packet, size_t length) { 83 bool AudioReceiveStream::DeliverRtp(const uint8_t* packet, size_t length) {
75 RTPHeader header; 84 RTPHeader header;
76 if (!rtp_header_parser_->Parse(packet, length, &header)) { 85 if (!rtp_header_parser_->Parse(packet, length, &header)) {
77 return false; 86 return false;
78 } 87 }
79 88
80 // Only forward if the parsed header has absolute sender time. RTP time stamps 89 // Only forward if the parsed header has absolute sender time. RTP time stamps
81 // may have different rates for audio and video and shouldn't be mixed. 90 // may have different rates for audio and video and shouldn't be mixed.
82 if (header.extension.hasAbsoluteSendTime) { 91 if (header.extension.hasAbsoluteSendTime) {
83 int64_t arrival_time_ms = TickTime::MillisecondTimestamp(); 92 int64_t arrival_time_ms = TickTime::MillisecondTimestamp();
84 size_t payload_size = length - header.headerLength; 93 size_t payload_size = length - header.headerLength;
85 remote_bitrate_estimator_->IncomingPacket(arrival_time_ms, payload_size, 94 remote_bitrate_estimator_->IncomingPacket(arrival_time_ms, payload_size,
86 header, false); 95 header, false);
87 } 96 }
88 return true; 97 return true;
89 } 98 }
90 } // namespace internal 99 } // namespace internal
91 } // namespace webrtc 100 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/video/audio_receive_stream.h ('k') | webrtc/video/end_to_end_tests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698