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

Side by Side Diff: webrtc/video_receive_stream.h

Issue 2523773003: Keep all codec parameters in VideoReceiveStream::Decoder (Closed)
Patch Set: Created 4 years 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
(...skipping 27 matching lines...) Expand all
38 VideoDecoder* decoder = nullptr; 38 VideoDecoder* decoder = nullptr;
39 39
40 // Received RTP packets with this payload type will be sent to this decoder 40 // Received RTP packets with this payload type will be sent to this decoder
41 // instance. 41 // instance.
42 int payload_type = 0; 42 int payload_type = 0;
43 43
44 // Name of the decoded payload (such as VP8). Maps back to the depacketizer 44 // Name of the decoded payload (such as VP8). Maps back to the depacketizer
45 // used to unpack incoming packets. 45 // used to unpack incoming packets.
46 std::string payload_name; 46 std::string payload_name;
47 47
48 DecoderSpecificSettings decoder_specific; 48 std::map<std::string, std::string> codec_params;
mflodman 2016/11/23 13:56:17 Can you add a comment describing the map.
sprang_webrtc 2016/11/24 14:59:46 And preferably add some common keys as constants s
magjed_webrtc 2016/11/24 17:11:56 Done.
magjed_webrtc 2016/11/24 17:11:56 Done.
49 }; 49 };
50 50
51 struct Stats { 51 struct Stats {
52 std::string ToString(int64_t time_ms) const; 52 std::string ToString(int64_t time_ms) const;
53 53
54 int network_frame_rate = 0; 54 int network_frame_rate = 0;
55 int decode_frame_rate = 0; 55 int decode_frame_rate = 0;
56 int render_frame_rate = 0; 56 int render_frame_rate = 0;
57 57
58 // Decoder stats. 58 // Decoder stats.
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 EnableEncodedFrameRecording(rtc::kInvalidPlatformFileValue, 0); 215 EnableEncodedFrameRecording(rtc::kInvalidPlatformFileValue, 0);
216 } 216 }
217 217
218 protected: 218 protected:
219 virtual ~VideoReceiveStream() {} 219 virtual ~VideoReceiveStream() {}
220 }; 220 };
221 221
222 } // namespace webrtc 222 } // namespace webrtc
223 223
224 #endif // WEBRTC_VIDEO_RECEIVE_STREAM_H_ 224 #endif // WEBRTC_VIDEO_RECEIVE_STREAM_H_
OLDNEW
« webrtc/media/engine/webrtcvideoengine2_unittest.cc ('K') | « webrtc/video/video_receive_stream.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698