diff options
author | lambroslambrou@chromium.org <lambroslambrou@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-23 18:53:52 +0000 |
---|---|---|
committer | lambroslambrou@chromium.org <lambroslambrou@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-23 18:53:52 +0000 |
commit | 41b93ad3c786a4c17b3e80c00b18bfa2cee81f92 (patch) | |
tree | cc47f5271bac818106fc273c6d2bf237fe28c241 /remoting/client/rectangle_update_decoder.cc | |
parent | ecb0b0fd4782c2b6b525aa186f7c0713ca27104d (diff) | |
download | chromium_src-41b93ad3c786a4c17b3e80c00b18bfa2cee81f92.zip chromium_src-41b93ad3c786a4c17b3e80c00b18bfa2cee81f92.tar.gz chromium_src-41b93ad3c786a4c17b3e80c00b18bfa2cee81f92.tar.bz2 |
Cleanup VideoDecoder interface.
This removes some un-needed methods and the DecodeResult enumeration.
Review URL: https://chromiumcodereview.appspot.com/23702056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224762 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/client/rectangle_update_decoder.cc')
-rw-r--r-- | remoting/client/rectangle_update_decoder.cc | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/remoting/client/rectangle_update_decoder.cc b/remoting/client/rectangle_update_decoder.cc index 106c993..3f21a99 100644 --- a/remoting/client/rectangle_update_decoder.cc +++ b/remoting/client/rectangle_update_decoder.cc @@ -92,14 +92,11 @@ void RectangleUpdateDecoder::DecodePacket(scoped_ptr<VideoPacket> packet, if (notify_size_or_dpi_change) consumer_->SetSourceSize(source_size_, source_dpi_); - if (!decoder_->IsReadyForData()) { - // TODO(ajwong): This whole thing should move into an invalid state. - LOG(ERROR) << "Decoder is unable to process data. Dropping packet."; - return; - } - - if (decoder_->DecodePacket(packet.get()) == VideoDecoder::DECODE_DONE) + if (decoder_->DecodePacket(*packet.get())) { SchedulePaint(); + } else { + LOG(ERROR) << "DecodePacket() failed."; + } } void RectangleUpdateDecoder::SchedulePaint() { |