Tighten up media::DecoderBuffer API contract for end of stream buffers.

media/base/decoder_buffer.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/decoder_buffer.h"
 
 #include "base/logging.h"
 #include "media/base/decrypt_config.h"
 
 namespace media {
@@ skipping 30 matching lines @@
   CHECK(data);
   return make_scoped_refptr(new DecoderBuffer(data, data_size));
 }
 
 // static
 scoped_refptr<DecoderBuffer> DecoderBuffer::CreateEOSBuffer() {
   return make_scoped_refptr(new DecoderBuffer(NULL, 0));
 }
 
 base::TimeDelta DecoderBuffer::GetTimestamp() const {
+  DCHECK(!IsEndOfStream());
   return timestamp_;
 }
 
 void DecoderBuffer::SetTimestamp(const base::TimeDelta& timestamp) {
+  DCHECK(!IsEndOfStream());
   timestamp_ = timestamp;
 }
 
 base::TimeDelta DecoderBuffer::GetDuration() const {
+  DCHECK(!IsEndOfStream());
   return duration_;
 }
 
 void DecoderBuffer::SetDuration(const base::TimeDelta& duration) {
+  DCHECK(!IsEndOfStream());
   duration_ = duration;
 }
 
 const uint8* DecoderBuffer::GetData() const {
+  DCHECK(!IsEndOfStream());
   return data_.get();
 }
 
 uint8* DecoderBuffer::GetWritableData() const {
+  DCHECK(!IsEndOfStream());
   return data_.get();
 }
 
 int DecoderBuffer::GetDataSize() const {
+  DCHECK(!IsEndOfStream());
   return size_;
 }
 
 const DecryptConfig* DecoderBuffer::GetDecryptConfig() const {
+  DCHECK(!IsEndOfStream());
   return decrypt_config_.get();
 }
 
 void DecoderBuffer::SetDecryptConfig(scoped_ptr<DecryptConfig> decrypt_config) {
+  DCHECK(!IsEndOfStream());
   decrypt_config_ = decrypt_config.Pass();
 }
 
 bool DecoderBuffer::IsEndOfStream() const {
   return data_ == NULL;
 }
 
+std::string DecoderBuffer::AsHumanReadableString() {
+  if (IsEndOfStream()) {
+    return "end of stream";
+  }
+
+  std::ostringstream s;
+  s << "timestamp: " << timestamp_.InMicroseconds()
+    << " duration: " << duration_.InMicroseconds()
+    << " size: " << size_
+    << " encrypted: " << (decrypt_config_ != NULL);
+  return s.str();
+}
+
 }  // namespace media