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

Side by Side Diff: webrtc/call/rtp_stream_receiver_controller.h

Issue 2886993005: Introduce RtpStreamReceiver and RtpStreamReceiverControllerInterface. (Closed)
Patch Set: Don't pass on packets if the ssrc isn't found in receive_rtp_config_. Created 3 years, 6 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
OLDNEW
(Empty)
1 /*
2 * Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
3 *
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
6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree.
9 */
10 #ifndef WEBRTC_CALL_RTP_STREAM_RECEIVER_CONTROLLER_H_
11 #define WEBRTC_CALL_RTP_STREAM_RECEIVER_CONTROLLER_H_
12
13 #include <memory>
14
15 #include "webrtc/base/array_view.h"
16 #include "webrtc/base/criticalsection.h"
17
18 #include "webrtc/call/rtp_demuxer.h"
19 #include "webrtc/call/rtp_stream_receiver_controller_interface.h"
20
21 namespace webrtc {
22
23 class ReceiveSideCongestionController;
24 class RtpPacketReceived;
25
26 // This class represents the RTP receive parsing and demuxing, for a
27 // single RTP session.
28 // TODO(nisse): Add RTCP processing, we should aim to terminate RTCP
29 // and not leave any RTCP processing to individual receive streams.
30 // TODO(nisse): Extract per-packet processing, including parsing and
31 // demuxing, into a separate class.
32 class RtpStreamReceiverController
33 : public RtpStreamReceiverControllerInterface {
34 public:
35 // Implements RtpStreamReceiverControllerInterface.
36 std::unique_ptr<RtpStreamReceiver> CreateReceiver(
37 uint32_t ssrc,
38 RtpPacketSinkInterface* sink) override;
39
40 // Thread-safe wrappers for the corresponding RtpDemuxer methods.
41 void AddSink(uint32_t ssrc, RtpPacketSinkInterface* sink) override;
42 size_t RemoveSink(const RtpPacketSinkInterface* sink) override;
43
44 // TODO(nisse): Not yet responsible for parsing.
45 bool OnRtpPacket(const RtpPacketReceived& packet);
46
47 private:
48 class Receiver : public RtpStreamReceiver {
49 public:
50 Receiver(RtpStreamReceiverController* controller,
51 uint32_t ssrc,
52 RtpPacketSinkInterface* sink);
53
54 ~Receiver() override;
55
56 private:
57 RtpStreamReceiverController* controller_;
danilchap 2017/06/13 15:58:32 may be * const controller_; * const sink_ to stres
nisse-webrtc 2017/06/14 06:31:06 Done.
58 RtpPacketSinkInterface* sink_;
59 };
60
61 rtc::CriticalSection lock_;
62 RtpDemuxer demuxer_ GUARDED_BY(&lock_);
63 };
64
65 } // namespace webrtc
66
67 #endif // WEBRTC_CALL_RTP_STREAM_RECEIVER_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698