diff options
author | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-13 22:43:41 +0000 |
---|---|---|
committer | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-13 22:43:41 +0000 |
commit | 8b6d3d07dc51f52f3079f300262c4e3c58645ee5 (patch) | |
tree | 3e1bcc9f51197d784222cbc35874882559a990b9 /remoting/client/rectangle_update_decoder.cc | |
parent | f6fad5a315506ff9d8fc49743917df8b201591c8 (diff) | |
download | chromium_src-8b6d3d07dc51f52f3079f300262c4e3c58645ee5.zip chromium_src-8b6d3d07dc51f52f3079f300262c4e3c58645ee5.tar.gz chromium_src-8b6d3d07dc51f52f3079f300262c4e3c58645ee5.tar.bz2 |
Simplify VideoEncoder interface.
This should also avoid assert in the linked bug.
BUG=284775
R=wez@chromium.org
Review URL: https://codereview.chromium.org/23477059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223152 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/client/rectangle_update_decoder.cc')
-rw-r--r-- | remoting/client/rectangle_update_decoder.cc | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/remoting/client/rectangle_update_decoder.cc b/remoting/client/rectangle_update_decoder.cc index 741357d..106c993 100644 --- a/remoting/client/rectangle_update_decoder.cc +++ b/remoting/client/rectangle_update_decoder.cc @@ -255,36 +255,28 @@ void RectangleUpdateDecoder::ProcessVideoPacket(scoped_ptr<VideoPacket> packet, } // Measure the latency between the last packet being received and presented. - bool last_packet = (packet->flags() & VideoPacket::LAST_PACKET) != 0; - base::Time decode_start; - if (last_packet) - decode_start = base::Time::Now(); + base::Time decode_start = base::Time::Now(); - base::Closure decode_done = - base::Bind(&RectangleUpdateDecoder::OnPacketDone, this, - last_packet, decode_start, done); + base::Closure decode_done = base::Bind( + &RectangleUpdateDecoder::OnPacketDone, this, decode_start, done); decode_task_runner_->PostTask(FROM_HERE, base::Bind( &RectangleUpdateDecoder::DecodePacket, this, base::Passed(&packet), decode_done)); } -void RectangleUpdateDecoder::OnPacketDone(bool last_packet, - base::Time decode_start, +void RectangleUpdateDecoder::OnPacketDone(base::Time decode_start, const base::Closure& done) { if (!main_task_runner_->BelongsToCurrentThread()) { main_task_runner_->PostTask(FROM_HERE, base::Bind( &RectangleUpdateDecoder::OnPacketDone, this, - last_packet, decode_start, done)); + decode_start, done)); return; } - // Record the latency between the final packet being received and - // presented. - if (last_packet) { - stats_.video_decode_ms()->Record( - (base::Time::Now() - decode_start).InMilliseconds()); - } + // Record the latency between the packet being received and presented. + stats_.video_decode_ms()->Record( + (base::Time::Now() - decode_start).InMilliseconds()); done.Run(); } |