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

webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc

Index: webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc
diff --git a/webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc b/webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc
index 5ec674f16a691d82903c55566c354e30da25eedb..6236caa9c3d5c80d7825ebf6cc8228a14a525e5c 100644
--- a/webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc
+++ b/webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc
@@ -11,6 +11,7 @@
 #include <stdio.h>
 
 #include "testing/gtest/include/gtest/gtest.h"
+#include "webrtc/base/checks.h"
 #include "webrtc/base/scoped_ptr.h"
 #include "webrtc/common_video/libyuv/include/webrtc_libyuv.h"
 #include "webrtc/modules/video_coding/codecs/vp8/include/vp8.h"
@@ -78,7 +79,11 @@ class Vp8UnitTestDecodeCompleteCallback : public webrtc::DecodedImageCallback {
  public:
   explicit Vp8UnitTestDecodeCompleteCallback(VideoFrame* frame)
       : decoded_frame_(frame), decode_complete(false) {}
-  int Decoded(webrtc::VideoFrame& frame);
+  int32_t Decoded(webrtc::VideoFrame& frame) override;
+  int32_t Decoded(VideoFrame& frame, int64_t decode_time_ms) override {

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

Why does one have webrtc::VideoFrame and the other VideoFrame?

[perkj_webrtc, 2015/11/10 12:50:16]

Done.

+    RTC_NOTREACHED();
+    return Decoded(frame);

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

return -1 here too, and in the other places we don't support this method.

[perkj_webrtc, 2015/11/10 12:50:16]

Done.

+  }
   bool DecodeComplete();
 
  private: