Tighten up media::DecoderBuffer API contract for end of stream buffers (round 2).

media/base/data_buffer.h

 // 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.
 
 #ifndef MEDIA_BASE_DATA_BUFFER_H_
 #define MEDIA_BASE_DATA_BUFFER_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/time.h"
 #include "media/base/media_export.h"
 
 namespace media {
 
 // A simple buffer that takes ownership of the given data pointer or allocates
 // as necessary.
 //
 // Unlike DecoderBuffer, allocations are assumed to be allocated with the
 // default memory allocator (i.e., new uint8[]).
+//
+// NOTE: It is illegal to call any method when IsEndOfStream() is true.
 class MEDIA_EXPORT DataBuffer : public base::RefCountedThreadSafe<DataBuffer> {
  public:
   // Allocates buffer of size |buffer_size| >= 0.
   explicit DataBuffer(int buffer_size);
 
   // Assumes valid data of size |buffer_size|.
   DataBuffer(scoped_array<uint8> buffer, int buffer_size);
 
   // Create a DataBuffer whose |data_| is copied from |data|.
   //
@@ skipping 42 matching lines @@
   scoped_array<uint8> data_;
   int buffer_size_;
   int data_size_;
 
   DISALLOW_COPY_AND_ASSIGN(DataBuffer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DATA_BUFFER_H_