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

media/base/decoder_buffer_unittest.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 "base/string_util.h"
 #include "media/base/decoder_buffer.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace media {
 
 TEST(DecoderBufferTest, Constructors) {
   scoped_refptr<DecoderBuffer> buffer(new DecoderBuffer(0));
   EXPECT_TRUE(buffer->GetData());
   EXPECT_EQ(0, buffer->GetDataSize());
   EXPECT_FALSE(buffer->IsEndOfStream());
 
   const int kTestSize = 10;
   scoped_refptr<DecoderBuffer> buffer3(new DecoderBuffer(kTestSize));
   ASSERT_TRUE(buffer3);
   EXPECT_EQ(kTestSize, buffer3->GetDataSize());
 }
 
 TEST(DecoderBufferTest, CreateEOSBuffer) {
   scoped_refptr<DecoderBuffer> buffer(DecoderBuffer::CreateEOSBuffer());
   EXPECT_TRUE(buffer->IsEndOfStream());
-  EXPECT_FALSE(buffer->GetData());
-  EXPECT_EQ(0, buffer->GetDataSize());
 }
 
 TEST(DecoderBufferTest, CopyFrom) {
   const uint8 kData[] = "hello";
   const int kDataSize = arraysize(kData);
   scoped_refptr<DecoderBuffer> buffer2(DecoderBuffer::CopyFrom(
       reinterpret_cast<const uint8*>(&kData), kDataSize));
   ASSERT_TRUE(buffer2);
   EXPECT_NE(kData, buffer2->GetData());
   EXPECT_EQ(buffer2->GetDataSize(), kDataSize);
@@ skipping 44 matching lines @@
   ASSERT_EQ(0, memcmp(read_only_data, kData, kDataSize));
   EXPECT_FALSE(buffer->IsEndOfStream());
 }
 
 TEST(DecoderBufferTest, GetDecryptConfig) {
   scoped_refptr<DecoderBuffer> buffer(new DecoderBuffer(0));
   EXPECT_FALSE(buffer->GetDecryptConfig());
 }
 
 }  // namespace media