Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(474)

Side by Side Diff: content/renderer/pepper/video_encoder_shim.cc

Issue 1996453003: RTC Video Encoder: Use capturer timestamp (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/pepper/video_encoder_shim.h" 5 #include "content/renderer/pepper/video_encoder_shim.h"
6 6
7 #include <inttypes.h> 7 #include <inttypes.h>
8 8
9 #include <deque> 9 #include <deque>
10 10
(...skipping 469 matching lines...) Expand 10 before | Expand all | Expand 10 after
480 output_buffer_size); 480 output_buffer_size);
481 } 481 }
482 482
483 void VideoEncoderShim::OnBitstreamBufferReady( 483 void VideoEncoderShim::OnBitstreamBufferReady(
484 scoped_refptr<media::VideoFrame> frame, 484 scoped_refptr<media::VideoFrame> frame,
485 int32_t bitstream_buffer_id, 485 int32_t bitstream_buffer_id,
486 size_t payload_size, 486 size_t payload_size,
487 bool key_frame) { 487 bool key_frame) {
488 DCHECK(RenderThreadImpl::current()); 488 DCHECK(RenderThreadImpl::current());
489 489
490 host_->BitstreamBufferReady(bitstream_buffer_id, payload_size, key_frame); 490 // TODO(llandwerlin): Pass real timestamp.
491 host_->BitstreamBufferReady(bitstream_buffer_id, payload_size, key_frame,
492 base::TimeDelta());
Pawel Osciak 2016/05/23 07:15:52 Could we use frame.timestamp() here?
shenghao 2016/05/24 10:50:19 Done.
491 } 493 }
492 494
493 void VideoEncoderShim::OnNotifyError( 495 void VideoEncoderShim::OnNotifyError(
494 media::VideoEncodeAccelerator::Error error) { 496 media::VideoEncodeAccelerator::Error error) {
495 DCHECK(RenderThreadImpl::current()); 497 DCHECK(RenderThreadImpl::current());
496 498
497 host_->NotifyError(error); 499 host_->NotifyError(error);
498 } 500 }
499 501
500 } // namespace content 502 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698