Add DecodedImageCallback::Decoded() function with custom decode time value.

webrtc/video_decoder.h

Index: webrtc/video_decoder.h
diff --git a/webrtc/video_decoder.h b/webrtc/video_decoder.h
index 2822677868db4f3aa531e1e7bd4bf3c2dc6ff176..51c0f4467944afd632ff70ec30e7518da2642c3f 100644
--- a/webrtc/video_decoder.h
+++ b/webrtc/video_decoder.h
@@ -29,6 +29,15 @@ class DecodedImageCallback {
   virtual ~DecodedImageCallback() {}
 
   virtual int32_t Decoded(VideoFrame& decodedImage) = 0;
+  // Provides an alternative interface that allows the decoder to specify the
+  // decode time excluding waiting time for any previous pending frame to
+  // return. This is necessary for breaking positive feedback in the delay
+  // estimation when the decoder has a single output buffer.
+  virtual int32_t Decoded(VideoFrame& decodedImage, int64_t decode_time_ms) {
+    // The default implementation ignores custom decode time value.
+    return Decoded(decodedImage);
+  }

[stefan-webrtc, 2015/11/10 10:32:00]

Are there many places where you have to make this change if you don't provide a
default implementation? I would have preferred that if it's reasonable to avoid.

Maybe we can follow-up with removing this default impl at least? In that case
add a TODO.

+
   virtual int32_t ReceivedDecodedReferenceFrame(const uint64_t pictureId) {
     return -1;
   }