RTC Video Encoder: Use capturer timestamp

media/gpu/vaapi_video_encode_accelerator.cc

Index: media/gpu/vaapi_video_encode_accelerator.cc
diff --git a/media/gpu/vaapi_video_encode_accelerator.cc b/media/gpu/vaapi_video_encode_accelerator.cc
index 5eea88e5335b78e457ff784a2789c0e7f850b3cd..0498407d0b4c7a66684209fe350ad35cb2b33d8e 100644
--- a/media/gpu/vaapi_video_encode_accelerator.cc
+++ b/media/gpu/vaapi_video_encode_accelerator.cc
@@ -555,8 +555,10 @@ void VaapiVideoEncodeAccelerator::TryToReturnBitstreamBuffer() {
             << " id: " << buffer->id << " size: " << data_size;
 
   child_task_runner_->PostTask(
-      FROM_HERE, base::Bind(&Client::BitstreamBufferReady, client_, buffer->id,
-                            data_size, encode_job->keyframe));
+      FROM_HERE,
+      base::Bind(&Client::BitstreamBufferReady, client_, buffer->id, data_size,
+                 encode_job->keyframe,
+                 base::TimeDelta::FromMicroseconds(encode_job->timestamp)));
 }
 
 void VaapiVideoEncodeAccelerator::Encode(
@@ -584,6 +586,9 @@ bool VaapiVideoEncodeAccelerator::PrepareNextJob() {
     return false;
   }
 
+  current_encode_job_->timestamp =
+      encoder_input_queue_.front()->frame->InMicroseconds();

[Pawel Osciak, 2016/05/26 07:39:23]

I would prefer not accessing encoder_input_queue_ from here for cleaner code (no
dependency between pop() in EncodeFrameTask() and this front()). Perhaps we
could pass timestamp as an argument to this method.

Also, I think you meant frame->timestamp().InMicroseconds()?

But I'd prefer if you just stored base::TimeDelta, i.e. frame->timestamp() in
EncodeJob, to be more portable.

[shenghao, 2016/05/26 10:38:20]

Done.

+
   current_encode_job_->input_surface = new VASurface(
       available_va_surface_ids_.back(), coded_size_,
       vaapi_wrapper_->va_surface_format(), va_surface_release_cb_);