RTC Video Encoder: Use capturer timestamp

media/video/video_encode_accelerator.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_VIDEO_VIDEO_ENCODE_ACCELERATOR_H_
 #define MEDIA_VIDEO_VIDEO_ENCODE_ACCELERATOR_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 60 matching lines @@
     //  encoder in bytes.
     virtual void RequireBitstreamBuffers(unsigned int input_count,
                                          const gfx::Size& input_coded_size,
                                          size_t output_buffer_size) = 0;
 
     // Callback to deliver encoded bitstream buffers.  Ownership of the buffer
     // is transferred back to the VEA::Client once this callback is made.
     // Parameters:
     //  |bitstream_buffer_id| is the id of the buffer that is ready.
     //  |payload_size| is the byte size of the used portion of the buffer.
     //  |key_frame| is true if this delivered frame is a keyframe.

[sandersd (OOO until July 31), 2016/05/19 20:27:12]

Document |timestamp|.

[shenghao, 2016/05/24 10:50:20]

Done.

     virtual void BitstreamBufferReady(int32_t bitstream_buffer_id,
                                       size_t payload_size,
-                                      bool key_frame) = 0;
+                                      bool key_frame,
+                                      base::TimeDelta timestamp) = 0;

[Pawel Osciak, 2016/05/23 07:15:53]

Perhaps const TimeDelta& here and in other places?

[shenghao, 2016/05/24 10:50:19]

All time classes are copyable, assignable, and occupy 64-bits per instance.
Thus, they can be efficiently passed by-value (as opposed to
by-reference)
https://code.google.com/p/chromium/codesearch#chromium/src/base/time/time.h&q...

 
     // Error notification callback. Note that errors in Initialize() will not be
     // reported here, but will instead be indicated by a false return value
     // there.
     virtual void NotifyError(Error error) = 0;
 
    protected:
     // Clients are not owned by VEA instances and should not be deleted through
     // these pointers.
     virtual ~Client() {}
@@ skipping 69 matching lines @@
 // std::unique_ptr<VideoEncodeAccelerator> uses "Destroy()" instead of trying to
 // use the destructor.
 template <>
 struct MEDIA_EXPORT default_delete<media::VideoEncodeAccelerator> {
   void operator()(media::VideoEncodeAccelerator* vea) const;
 };
 
 }  // namespace std
 
 #endif  // MEDIA_VIDEO_VIDEO_ENCODE_ACCELERATOR_H_