Refactor video capturing code in the render process

content/renderer/media/media_stream_video_source.h

Index: content/renderer/media/media_stream_video_source.h
diff --git a/content/renderer/media/media_stream_video_source.h b/content/renderer/media/media_stream_video_source.h
index 5c5289dd776dd5864bbdd8187cb98d9a38419cd8..235d5b3838b243e675e5dd85582ccef8ebb38662 100644
--- a/content/renderer/media/media_stream_video_source.h
+++ b/content/renderer/media/media_stream_video_source.h
@@ -82,7 +82,9 @@ class CONTENT_EXPORT MediaStreamVideoSource
   // Delivers |frame| to registered tracks according to their constraints.
   // Note: current implementation assumes |frame| be contiguous layout of image
   // planes and I420.
-  virtual void DeliverVideoFrame(const scoped_refptr<media::VideoFrame>& frame);
+  virtual void DeliverVideoFrame(const scoped_refptr<media::VideoFrame>& frame,
+                                 const media::VideoCaptureFormat& format,
+                                 const base::TimeTicks& timestamp);

[Ami GONE FROM CHROMIUM, 2014/04/24 21:04:31]

media::VideoFrame has a timestamp field.  Why is this timestamp param required
here in addition?
(requires a comment at least)

[Alpha Left Google, 2014/04/24 22:50:34]

Done.

[Alpha Left Google, 2014/04/24 22:50:34]

It is there to allow synchronization with other sources. Added a comment.

 
   // An implementation must fetch the formats that can currently be used by
   // the source and call OnSupportedFormats when done.