Add sender controlled playout delay limits

webrtc/common_types.h

Index: webrtc/common_types.h
diff --git a/webrtc/common_types.h b/webrtc/common_types.h
index 587b90d753ce3f3eb092dfd01edb8b1b68303719..e133e0be9c600a19ae9c797bee57db73b3d38f2a 100644
--- a/webrtc/common_types.h
+++ b/webrtc/common_types.h
@@ -751,6 +751,21 @@ struct PacketTime {
                        // If unknown, this value will be set to zero.
 };
 
+// Minimum and maximum playout delay values from capture to render.
+// These are best effort values.
+//
+// A value < 0 indicates no change from previous valid value.
+//
+// min = max = 0 indicates that the receiver should try and render
+// frame as soon as possible.
+//
+// min = x, max = y indicates that the receiver is free to adapt
+// in the range (x, y) based on network jitter.
+struct PlayoutDelay {

[danilchap, 2016/05/26 09:41:58]

May be you want to add a default constructor and initialize values to -1/-1
Currently explicit default constructor of RTPHeaderExtension leaves this two
values uninitialized.

[Sergey Ulanov, 2016/05/30 07:22:56]

Or just add default initalizers '= -1' in the field declarations.

[Irfan, 2016/06/01 08:38:33]

I have added a comment here, but this eventually ends up in a union that gets
created and used in places.  We have to let the owner control initialization.

http://www.stroustrup.com/C++11FAQ.html#unions

I think RTP header structures deserve a clean up moving away from a union.

+  int min_ms;
+  int max_ms;
+};
+
 struct RTPHeaderExtension {
   RTPHeaderExtension();
 
@@ -772,6 +787,8 @@ struct RTPHeaderExtension {
   // ts_126114v120700p.pdf
   bool hasVideoRotation;
   uint8_t videoRotation;
+
+  PlayoutDelay playout_delay;
 };
 
 struct RTPHeader {