Added VP8 simulcast tests. Fixed analyzer to correctly infer timestamps.

webrtc/common_video/include/frame_callback.h

Index: webrtc/common_video/include/frame_callback.h
diff --git a/webrtc/common_video/include/frame_callback.h b/webrtc/common_video/include/frame_callback.h
index 81737c6b962e4dca60e56796a62d2f6955e519d2..ea8d18eb93255cf8946cc12baa9323b394ba0a4e 100644
--- a/webrtc/common_video/include/frame_callback.h
+++ b/webrtc/common_video/include/frame_callback.h
@@ -22,13 +22,21 @@ class VideoFrame;
 
 struct EncodedFrame {
  public:
-  EncodedFrame() : data_(NULL), length_(0), frame_type_(kEmptyFrame) {}
-  EncodedFrame(const uint8_t* data, size_t length, FrameType frame_type)
-    : data_(data), length_(length), frame_type_(frame_type) {}
+  EncodedFrame() : data_(NULL), length_(0), frame_type_(kEmptyFrame),

[sprang_webrtc, 2017/02/01 14:50:19]

nit: nullptr

[ilnik, 2017/02/01 15:12:50]

Done.

+                   encoded_width_(0), encoded_height_(0), timestamp_(0) {}
+  EncodedFrame(const uint8_t* data, size_t length, FrameType frame_type,
+               uint32_t encoded_width, uint32_t encoded_height,
+               uint32_t timestamp)
+    : data_(data), length_(length), frame_type_(frame_type),
+      encoded_width_(encoded_width), encoded_height_(encoded_height),
+      timestamp_(timestamp) { }
 
   const uint8_t* data_;
   const size_t length_;
   const FrameType frame_type_;
+  const uint32_t encoded_width_;
+  const uint32_t encoded_height_;
+  const int32_t timestamp_;

[sprang_webrtc, 2017/02/01 14:50:19]

Hm, constructor parameter is unsigned but member is signed?
Not sure which kind of timestamp this is, but I think we're moving towards using
ns timestamps with int64_t type wherever we can, so might be a good idea here
too.

[ilnik, 2017/02/01 15:12:50]

Done.

 };
 
 class I420FrameCallback {