Collecting encode_time_ms for each frame

webrtc/video_send_stream.h

Index: webrtc/video_send_stream.h
diff --git a/webrtc/video_send_stream.h b/webrtc/video_send_stream.h
index 04f4fea31c1d0356bf9c540dda2eae211d14b288..a694e366d604e2a32d767d48cbc7dd1801309b42 100644
--- a/webrtc/video_send_stream.h
+++ b/webrtc/video_send_stream.h
@@ -26,6 +26,13 @@ namespace webrtc {
 class LoadObserver;
 class VideoEncoder;
 
+class EncodingTimeObserver {
+ public:
+  virtual ~EncodingTimeObserver() {}
+
+  virtual void OnEncodedFrame(const VideoFrame& frame, int encode_time_ms) = 0;

[mflodman, 2015/10/06 11:11:27]

Not confuse this with EncodedFrameObserver::EncodedFrameCallback, I'd prefer to:
1. Replace the 'const VideoFrame& frame' argument with the capture ntp time
only, since that is what's being used by the 'EncodingTimeObserver'
implementation.
2. Rename 'OnEncodedFrame' to 'OnReportEncodedTime', or something similar.

[ivica, 2015/10/06 12:04:32]

Makes sense, this was really confusing. Renamed.

[mflodman, 2015/10/06 12:10:00]

Great!

+};
+
 // Class to deliver captured frame to the video send stream.
 class VideoCaptureInput {
  public:
@@ -152,6 +159,11 @@ class VideoSendStream : public SendStream {
     // below the minimum configured bitrate. If this variable is false, the
     // stream may send at a rate higher than the estimated available bitrate.
     bool suspend_below_min_bitrate = false;
+
+    // Called for each encoded frame. Passes the total time spent on encoding.
+    // TODO(ivica): Consolidate with post_encode_callback:
+    // https://code.google.com/p/webrtc/issues/detail?id=5042
+    EncodingTimeObserver* encoding_time_observer = nullptr;
   };
 
   // Gets interface used to insert captured frames. Valid as long as the