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

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

Issue 2426563003: Moved transport.h from webrtc/ to webrtc/api, created build target and updated dependencies. (Closed)
Patch Set: 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 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
11 #ifndef WEBRTC_TRANSPORT_H_ 11 #ifndef WEBRTC_API_CALL_TRANSPORT_H_
12 #define WEBRTC_TRANSPORT_H_ 12 #define WEBRTC_API_CALL_TRANSPORT_H_
13 13
14 #include <stddef.h> 14 #include "webrtc/base/basictypes.h"
aleloi 2016/10/17 14:01:33 This file is not in any build target, but is inclu
ossu 2016/10/17 14:05:44 Are you sure you wouldn't be better off including
aleloi 2016/10/17 14:21:56 I think including the std headers is more simple a
the sun 2016/10/17 14:38:35 No, the files in api/ should only include files in
15
16 #include "webrtc/typedefs.h"
17 15
18 namespace webrtc { 16 namespace webrtc {
19 17
20 // TODO(holmer): Look into unifying this with the PacketOptions in 18 // TODO(holmer): Look into unifying this with the PacketOptions in
21 // asyncpacketsocket.h. 19 // asyncpacketsocket.h.
22 struct PacketOptions { 20 struct PacketOptions {
23 // A 16 bits positive id. Negative ids are invalid and should be interpreted 21 // A 16 bits positive id. Negative ids are invalid and should be interpreted
24 // as packet_id not being set. 22 // as packet_id not being set.
25 int packet_id = -1; 23 int packet_id = -1;
26 }; 24 };
27 25
28 class Transport { 26 class Transport {
29 public: 27 public:
30 virtual bool SendRtp(const uint8_t* packet, 28 virtual bool SendRtp(const uint8_t* packet,
31 size_t length, 29 size_t length,
32 const PacketOptions& options) = 0; 30 const PacketOptions& options) = 0;
33 virtual bool SendRtcp(const uint8_t* packet, size_t length) = 0; 31 virtual bool SendRtcp(const uint8_t* packet, size_t length) = 0;
34 32
35 protected: 33 protected:
36 virtual ~Transport() {} 34 virtual ~Transport() {}
37 }; 35 };
38 36
39 } // namespace webrtc 37 } // namespace webrtc
40 38
41 #endif // WEBRTC_TRANSPORT_H_ 39 #endif // WEBRTC_API_CALL_TRANSPORT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698