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

Side by Side Diff: webrtc/media/base/videoframe.h

Issue 2088953002: Add cricket::VideoFrame::frame_id() and set it to RTP timestamp. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 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
1 /* 1 /*
2 * Copyright (c) 2004 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2004 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 16 matching lines...) Expand all
27 // Basic accessors. 27 // Basic accessors.
28 // Note this is the width and height without rotation applied. 28 // Note this is the width and height without rotation applied.
29 virtual int width() const = 0; 29 virtual int width() const = 0;
30 virtual int height() const = 0; 30 virtual int height() const = 0;
31 31
32 // Returns the underlying video frame buffer. This function is ok to call 32 // Returns the underlying video frame buffer. This function is ok to call
33 // multiple times, but the returned object will refer to the same memory. 33 // multiple times, but the returned object will refer to the same memory.
34 virtual const rtc::scoped_refptr<webrtc::VideoFrameBuffer>& 34 virtual const rtc::scoped_refptr<webrtc::VideoFrameBuffer>&
35 video_frame_buffer() const = 0; 35 video_frame_buffer() const = 0;
36 36
37 // Frame ID. Normally RTP timestamp when the frame was received using RTP.
38 virtual uint32_t frame_id() const = 0;
stefan-webrtc 2016/07/28 08:54:49 Maybe add "transport" to the name to make it more
Sergey Ulanov 2016/07/29 00:10:22 Done.
39
37 // System monotonic clock, same timebase as rtc::TimeMicros(). 40 // System monotonic clock, same timebase as rtc::TimeMicros().
38 virtual int64_t timestamp_us() const = 0; 41 virtual int64_t timestamp_us() const = 0;
39 virtual void set_timestamp_us(int64_t time_us) = 0; 42 virtual void set_timestamp_us(int64_t time_us) = 0;
40 43
41 // Deprecated methods, for backwards compatibility. 44 // Deprecated methods, for backwards compatibility.
42 // TODO(nisse): Delete when usage in Chrome and other applications 45 // TODO(nisse): Delete when usage in Chrome and other applications
43 // have been replaced. 46 // have been replaced.
44 virtual int64_t GetTimeStamp() const { 47 virtual int64_t GetTimeStamp() const {
45 return rtc::kNumNanosecsPerMicrosec * timestamp_us(); 48 return rtc::kNumNanosecsPerMicrosec * timestamp_us();
46 } 49 }
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 virtual VideoFrame* CreateEmptyFrame(int w, 84 virtual VideoFrame* CreateEmptyFrame(int w,
82 int h, 85 int h,
83 int64_t timestamp_us) const = 0; 86 int64_t timestamp_us) const = 0;
84 87
85 virtual void set_rotation(webrtc::VideoRotation rotation) = 0; 88 virtual void set_rotation(webrtc::VideoRotation rotation) = 0;
86 }; 89 };
87 90
88 } // namespace cricket 91 } // namespace cricket
89 92
90 #endif // WEBRTC_MEDIA_BASE_VIDEOFRAME_H_ 93 #endif // WEBRTC_MEDIA_BASE_VIDEOFRAME_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698