Skip to content

Commit

Permalink
[chrome.displaySource] Fix display source to use the current VEA API.
Browse files Browse the repository at this point in the history
Accept the capturer timestamp as a parameter in BitstreamBufferReady()
also in display source extension as per new VideoEncodeAccelerator API.

BUG=242107,350106

Review-Url: https://codereview.chromium.org/2033533003
Cr-Commit-Position: refs/heads/master@{#397378}
  • Loading branch information
eehakkin authored and Commit bot committed Jun 2, 2016
1 parent 7e922c0 commit 356a35f
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,8 @@ class WiFiDisplayVideoEncoderVEA final

void BitstreamBufferReady(int32_t bitstream_buffer_id,
size_t payload_size,
bool key_frame) override;
bool key_frame,
base::TimeDelta timestamp) override;
void NotifyError(media::VideoEncodeAccelerator::Error error) override;

scoped_refptr<WiFiDisplayVideoEncoder> InitOnMediaThread(
Expand Down Expand Up @@ -183,7 +184,8 @@ void WiFiDisplayVideoEncoderVEA::InsertFrameOnMediaThread(
void WiFiDisplayVideoEncoderVEA::BitstreamBufferReady(
int32_t bitstream_buffer_id,
size_t payload_size,
bool key_frame) {
bool key_frame,
base::TimeDelta timestamp) {
if (bitstream_buffer_id >= static_cast<int>(output_buffers_.size())) {
DVLOG(1) << "WiFiDisplayVideoEncoderVEA::BitstreamBufferReady()"
<< ": invalid bitstream_buffer_id=" << bitstream_buffer_id;
Expand Down

0 comments on commit 356a35f

Please sign in to comment.