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

Unified Diff: webrtc/modules/video_coding/frame_object.h

Issue 2444193010: Delete FrameObject::size member. (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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/modules/video_coding/frame_buffer2.cc ('k') | webrtc/modules/video_coding/frame_object.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/video_coding/frame_object.h
diff --git a/webrtc/modules/video_coding/frame_object.h b/webrtc/modules/video_coding/frame_object.h
index a63ae996b8dff1331500954110a6f7dd9b3ea121..542797226e99434dc6e6e95b5c81599ddc47ef48 100644
--- a/webrtc/modules/video_coding/frame_object.h
+++ b/webrtc/modules/video_coding/frame_object.h
@@ -36,6 +36,7 @@ class FrameObject : public webrtc::VCMEncodedFrame {
// When this frame should be rendered.
virtual int64_t RenderTime() const = 0;
+ size_t size() { return _length; }
nisse-webrtc 2016/10/27 11:24:34 It should be _length here, not _size, right?
philipel 2016/10/27 11:27:15 Thats right!
sprang_webrtc 2016/10/27 12:40:02 Ugh.. Any chance you could rename them size_ and c
// The tuple (|picture_id|, |spatial_layer|) uniquely identifies a frame
// object. For codec types that don't necessarily have picture ids they
@@ -47,8 +48,6 @@ class FrameObject : public webrtc::VCMEncodedFrame {
size_t num_references;
uint16_t references[kMaxFrameReferences];
bool inter_layer_predicted;
-
- size_t size;
};
class PacketBuffer;
« no previous file with comments | « webrtc/modules/video_coding/frame_buffer2.cc ('k') | webrtc/modules/video_coding/frame_object.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698